<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>107461</bug_id>
          
          <creation_ts>2013-01-21 10:40:19 -0800</creation_ts>
          <short_desc>[Qt] QtTestBrowser should provide option to enable/disable Javascript</short_desc>
          <delta_ts>2013-02-07 08:06:02 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Qt</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Vivek Galatage">vivekg</reporter>
          <assigned_to name="Vivek Galatage">vivek.vg</assigned_to>
          <cc>allan.jensen</cc>
    
    <cc>hausmann</cc>
    
    <cc>jturcotte</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>noam</cc>
    
    <cc>ossy</cc>
    
    <cc>vivek.vg</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>812258</commentid>
    <comment_count>0</comment_count>
    <who name="Vivek Galatage">vivekg</who>
    <bug_when>2013-01-21 10:40:19 -0800</bug_when>
    <thetext>QtTestBrowser should provide option to enable/disable Javascript.

Patch to follow.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>812259</commentid>
    <comment_count>1</comment_count>
      <attachid>183804</attachid>
    <who name="Vivek Galatage">vivek.vg</who>
    <bug_when>2013-01-21 10:44:46 -0800</bug_when>
    <thetext>Created attachment 183804
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>812596</commentid>
    <comment_count>2</comment_count>
      <attachid>183888</attachid>
    <who name="Vivek Galatage">vivek.vg</who>
    <bug_when>2013-01-21 21:53:25 -0800</bug_when>
    <thetext>Created attachment 183888
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>812655</commentid>
    <comment_count>3</comment_count>
    <who name="Noam Rosenthal">noam</who>
    <bug_when>2013-01-21 23:33:31 -0800</bug_when>
    <thetext>You can do this via web inspector; Do you really need this duplicated in the browser settings?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>812658</commentid>
    <comment_count>4</comment_count>
    <who name="Vivek Galatage">vivekg</who>
    <bug_when>2013-01-21 23:40:24 -0800</bug_when>
    <thetext>(In reply to comment #3)
&gt; You can do this via web inspector; Do you really need this duplicated in the browser settings?

yes I agree. But for a quick test for disabling it we need to open the inspector and then navigate it to the &quot;Inspector settings&quot; and then in the &quot;General&quot; tab to actually disable it. I thought for a test browser that would take a little longer hence introduced this quick menu item. 

Also one thing I am finding different is that on linux the inspector is not shown inside the QtTestBrowser whereas the same is shown on the mac.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>812864</commentid>
    <comment_count>5</comment_count>
    <who name="Jocelyn Turcotte">jturcotte</who>
    <bug_when>2013-01-22 03:17:39 -0800</bug_when>
    <thetext>(In reply to comment #4)
It feels like you would be the only one using this menu. QtTestBrowser is more a WebKit developer tool then a web developer tool.

Do you see a use case where other developers might want to use this menu?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>812867</commentid>
    <comment_count>6</comment_count>
    <who name="Vivek Galatage">vivekg</who>
    <bug_when>2013-01-22 03:23:25 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; (In reply to comment #4)
&gt; It feels like you would be the only one using this menu. QtTestBrowser is more a WebKit developer tool then a web developer tool.
&gt; 
&gt; Do you see a use case where other developers might want to use this menu?

I agree. The use case is something like: I am working on [1] and wanted a quick way to verify this bug. Hence I wanted it to quickly provide me an option to enable/disable the Javascript and hence prompted me in adding it to the QtTestBrowser.

[1] https://bugs.webkit.org/show_bug.cgi?id=103902</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>824159</commentid>
    <comment_count>7</comment_count>
    <who name="Vivek Galatage">vivekg</who>
    <bug_when>2013-02-04 02:06:12 -0800</bug_when>
    <thetext>(In reply to comment #5)
&gt; (In reply to comment #4)
&gt; It feels like you would be the only one using this menu. QtTestBrowser is more a WebKit developer tool then a web developer tool.
&gt; 
&gt; Do you see a use case where other developers might want to use this menu?

Another use case is that of browsers (almost all the browsers) providing the option to disable JavaScript execution altogether. So this menu would help simulate that behavior as well. In this case, the users usually disable the option via browser settings as such.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>827259</commentid>
    <comment_count>8</comment_count>
      <attachid>183888</attachid>
    <who name="Jocelyn Turcotte">jturcotte</who>
    <bug_when>2013-02-07 03:28:25 -0800</bug_when>
    <thetext>Comment on attachment 183888
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=183888&amp;action=review

&gt; Tools/QtTestBrowser/launcherwindow.cpp:501
&gt; +    toggleJavaScriptEnabled-&gt;setChecked(true);

This should read:
toggleJavaScriptEnabled-&gt;setChecked(settings-&gt;testAttribute(QWebSettings::JavascriptEnabled));

&gt; Tools/QtTestBrowser/launcherwindow.cpp:933
&gt; +    statusBar()-&gt;showMessage(QString(&quot;Javascript &quot;) + QString(enable ? &quot;enabled&quot; : &quot;disabled&quot;), 5000);

This isn&apos;t necessary, the checkbox already tells the user if JS is enabled or not.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>827294</commentid>
    <comment_count>9</comment_count>
      <attachid>183888</attachid>
    <who name="Vivek Galatage">vivekg</who>
    <bug_when>2013-02-07 04:13:20 -0800</bug_when>
    <thetext>Comment on attachment 183888
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=183888&amp;action=review

Thank you for your feedback.

&gt;&gt; Tools/QtTestBrowser/launcherwindow.cpp:501
&gt;&gt; +    toggleJavaScriptEnabled-&gt;setChecked(true);
&gt; 
&gt; This should read:
&gt; toggleJavaScriptEnabled-&gt;setChecked(settings-&gt;testAttribute(QWebSettings::JavascriptEnabled));

Sure, I will correct this.

&gt;&gt; Tools/QtTestBrowser/launcherwindow.cpp:933
&gt;&gt; +    statusBar()-&gt;showMessage(QString(&quot;Javascript &quot;) + QString(enable ? &quot;enabled&quot; : &quot;disabled&quot;), 5000);
&gt; 
&gt; This isn&apos;t necessary, the checkbox already tells the user if JS is enabled or not.

Agreed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>827296</commentid>
    <comment_count>10</comment_count>
      <attachid>187057</attachid>
    <who name="Vivek Galatage">vivek.vg</who>
    <bug_when>2013-02-07 04:14:49 -0800</bug_when>
    <thetext>Created attachment 187057
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>827524</commentid>
    <comment_count>11</comment_count>
      <attachid>187057</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-02-07 08:05:57 -0800</bug_when>
    <thetext>Comment on attachment 187057
Patch

Clearing flags on attachment: 187057

Committed r142125: &lt;http://trac.webkit.org/changeset/142125&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>827525</commentid>
    <comment_count>12</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2013-02-07 08:06:02 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>183804</attachid>
            <date>2013-01-21 10:44:46 -0800</date>
            <delta_ts>2013-01-21 21:53:18 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-107461-20130122001140.patch</filename>
            <type>text/plain</type>
            <size>3116</size>
            <attacher name="Vivek Galatage">vivek.vg</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQwMzI0CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggMTkxMTg4MWI0NmNiNWExODNhZjhmOTAxMmE5YzkyNDEz
NzIzNTk1ZC4uYTY1NDkyY2ZmMjkyMGFkYjdiODQzYTU0MTcwZjcwY2M3MDA4MTA2YSAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDIx
IEBACiAyMDEzLTAxLTIxICBWaXZlayBHYWxhdGFnZSAgPHZpdmVrLnZnQHNhbXN1bmcuY29tPgog
CisgICAgICAgIFtRdF0gUXRUZXN0QnJvd3NlciBzaG91bGQgcHJvdmlkZSBvcHRpb24gdG8gZW5h
YmxlL2Rpc2FibGUgSmF2YXNjcmlwdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTA3NDYxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgT3B0aW9uIHRvIGVuYWJsZS9kaXNhYmxlIEphdmFzY3JpcHQgd291bGQg
YmUgaGFuZHkgb3B0aW9uIHRvIHRlc3QKKyAgICAgICAgY2VydGFpbiBmdW5jdGlvbmFsaXRpZXMg
b2Ygd2ViIHBhZ2VzIHdpdGgvd2l0aG91dCBKYXZhc2NyaXB0LgorCisgICAgICAgICogUXRUZXN0
QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHA6CisgICAgICAgIChMYXVuY2hlcldpbmRvdzo6Y3Jl
YXRlQ2hyb21lKToKKyAgICAgICAgKExhdW5jaGVyV2luZG93Ojp0b2dnbGVKYXZhU2NyaXB0RW5h
YmxlZCk6CisgICAgICAgICogUXRUZXN0QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5oOgorICAgICAg
ICAoTGF1bmNoZXJXaW5kb3cpOgorCisyMDEzLTAxLTIxICBWaXZlayBHYWxhdGFnZSAgPHZpdmVr
LnZnQHNhbXN1bmcuY29tPgorCiAgICAgICAgIFF0VGVzdEJyb3dzZXIgc2hvdWxkIHByb3ZpZGUg
d2F5IHRvIGNsZWFyIHNlbGVjdGVkIGVsZW1lbnRzCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDc0MzcKIApkaWZmIC0tZ2l0IGEvVG9vbHMvUXRUZXN0
QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHAgYi9Ub29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVy
d2luZG93LmNwcAppbmRleCA4NTUxOTBiNzgxNjg3NThhOTE4MmMzN2MyOWYyYTVjODAwYjFiZGQ1
Li5mOTdmZmE4YzdmZjBlMzE2NDg5YmM2NDc2YjgxOGY4MDZjMzIwM2NhIDEwMDY0NAotLS0gYS9U
b29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVyd2luZG93LmNwcAorKysgYi9Ub29scy9RdFRlc3RC
cm93c2VyL2xhdW5jaGVyd2luZG93LmNwcApAQCAtNDk2LDYgKzQ5NiwxMCBAQCB2b2lkIExhdW5j
aGVyV2luZG93OjpjcmVhdGVDaHJvbWUoKQogI2VuZGlmCiAgICAgdG9nZ2xlU2Nyb2xsQW5pbWF0
b3ItPnNldENoZWNrZWQoZmFsc2UpOwogCisgICAgUUFjdGlvbiogdG9nZ2xlSmF2YVNjcmlwdGVF
bmFibGVkID0gc2V0dGluZ3NNZW51LT5hZGRBY3Rpb24oIkVuYWJsZSBKYXZhc2NyaXB0IiwgdGhp
cywgU0xPVCh0b2dnbGVKYXZhU2NyaXB0RW5hYmxlZChib29sKSkpOworICAgIHRvZ2dsZUphdmFT
Y3JpcHRlRW5hYmxlZC0+c2V0Q2hlY2thYmxlKHRydWUpOworICAgIHRvZ2dsZUphdmFTY3JpcHRl
RW5hYmxlZC0+c2V0Q2hlY2tlZCh0cnVlKTsKKwogICAgIFFBY3Rpb24qIHRvZ2dsZUludGVycnVw
dGluZ0phdmFTY3JpcHRlRW5hYmxlZCA9IHNldHRpbmdzTWVudS0+YWRkQWN0aW9uKCJFbmFibGUg
aW50ZXJydXB0aW5nIGpzIHNjcmlwdHMiLCB0aGlzLCBTTE9UKHRvZ2dsZUludGVycnVwdGluZ0ph
dmFTY3JpcHRFbmFibGVkKGJvb2wpKSk7CiAgICAgdG9nZ2xlSW50ZXJydXB0aW5nSmF2YVNjcmlw
dGVFbmFibGVkLT5zZXRDaGVja2FibGUodHJ1ZSk7CiAgICAgdG9nZ2xlSW50ZXJydXB0aW5nSmF2
YVNjcmlwdGVFbmFibGVkLT5zZXRDaGVja2VkKGZhbHNlKTsKQEAgLTkyMyw2ICs5MjcsMTIgQEAg
dm9pZCBMYXVuY2hlcldpbmRvdzo6dG9nZ2xlRnJhbWVGbGF0dGVuaW5nKGJvb2wgdG9nZ2xlKQog
ICAgIHBhZ2UoKS0+c2V0dGluZ3MoKS0+c2V0QXR0cmlidXRlKFFXZWJTZXR0aW5nczo6RnJhbWVG
bGF0dGVuaW5nRW5hYmxlZCwgdG9nZ2xlKTsKIH0KIAordm9pZCBMYXVuY2hlcldpbmRvdzo6dG9n
Z2xlSmF2YVNjcmlwdEVuYWJsZWQoYm9vbCBlbmFibGUpCit7CisgICAgcGFnZSgpLT5zZXR0aW5n
cygpLT5zZXRBdHRyaWJ1dGUoUVdlYlNldHRpbmdzOjpKYXZhc2NyaXB0RW5hYmxlZCwgZW5hYmxl
KTsKKyAgICBzdGF0dXNCYXIoKS0+c2hvd01lc3NhZ2UoUVN0cmluZygiSmF2YXNjcmlwdCAiKSAr
IFFTdHJpbmcoZW5hYmxlID8gImVuYWJsZWQiIDogImRpc2FibGVkIiksIDUwMDApOworfQorCiB2
b2lkIExhdW5jaGVyV2luZG93Ojp0b2dnbGVJbnRlcnJ1cHRpbmdKYXZhU2NyaXB0RW5hYmxlZChi
b29sIGVuYWJsZSkKIHsKICAgICBwYWdlKCktPnNldEludGVycnVwdGluZ0phdmFTY3JpcHRFbmFi
bGVkKGVuYWJsZSk7CmRpZmYgLS1naXQgYS9Ub29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVyd2lu
ZG93LmggYi9Ub29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVyd2luZG93LmgKaW5kZXggZmIzMTM3
ZGFjZmZjNzdlODcwNDc4ODFhMmE5N2ViOGMxZTU2ZjRhYi4uMTg1ZjIxYjc0MGUyNTA4OGFjZTBh
MGZmMzA3MDBjMWEwZDYyYjMxYiAxMDA2NDQKLS0tIGEvVG9vbHMvUXRUZXN0QnJvd3Nlci9sYXVu
Y2hlcndpbmRvdy5oCisrKyBiL1Rvb2xzL1F0VGVzdEJyb3dzZXIvbGF1bmNoZXJ3aW5kb3cuaApA
QCAtMTYwLDYgKzE2MCw3IEBAIHByb3RlY3RlZCBRX1NMT1RTOgogICAgIHZvaWQgdG9nZ2xlU3Bh
dGlhbE5hdmlnYXRpb24oYm9vbCBiKTsKICAgICB2b2lkIHRvZ2dsZUZ1bGxTY3JlZW5Nb2RlKGJv
b2wgZW5hYmxlKTsKICAgICB2b2lkIHRvZ2dsZUZyYW1lRmxhdHRlbmluZyhib29sIHRvZ2dsZSk7
CisgICAgdm9pZCB0b2dnbGVKYXZhU2NyaXB0RW5hYmxlZChib29sIGVuYWJsZSk7CiAgICAgdm9p
ZCB0b2dnbGVJbnRlcnJ1cHRpbmdKYXZhU2NyaXB0RW5hYmxlZChib29sIGVuYWJsZSk7CiAgICAg
dm9pZCB0b2dnbGVKYXZhc2NyaXB0Q2FuT3BlbldpbmRvd3MoYm9vbCBlbmFibGUpOwogICAgIHZv
aWQgdG9nZ2xlQXV0b0xvYWRJbWFnZXMoYm9vbCBlbmFibGUpOwo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>183888</attachid>
            <date>2013-01-21 21:53:25 -0800</date>
            <delta_ts>2013-02-07 04:14:44 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-107461-20130122112117.patch</filename>
            <type>text/plain</type>
            <size>3113</size>
            <attacher name="Vivek Galatage">vivek.vg</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQwMDkzCmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggZjNjNTAxZTNiMTg4ZjRiM2E4NjRmMTQ4ZTU4MDc3Nzhj
OGQxZWJkMi4uNmIzOWMyMzRhZmU4YmUxNWE3ZThmZmEwMjIwMjhmMjQxYzc0MjU0NCAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDIx
IEBACiAyMDEzLTAxLTIxICBWaXZlayBHYWxhdGFnZSAgPHZpdmVrLnZnQHNhbXN1bmcuY29tPgog
CisgICAgICAgIFtRdF0gUXRUZXN0QnJvd3NlciBzaG91bGQgcHJvdmlkZSBvcHRpb24gdG8gZW5h
YmxlL2Rpc2FibGUgSmF2YXNjcmlwdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTA3NDYxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgT3B0aW9uIHRvIGVuYWJsZS9kaXNhYmxlIEphdmFzY3JpcHQgd291bGQg
YmUgaGFuZHkgb3B0aW9uIHRvIHRlc3QKKyAgICAgICAgY2VydGFpbiBmdW5jdGlvbmFsaXRpZXMg
b2Ygd2ViIHBhZ2VzIHdpdGgvd2l0aG91dCBKYXZhc2NyaXB0LgorCisgICAgICAgICogUXRUZXN0
QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHA6CisgICAgICAgIChMYXVuY2hlcldpbmRvdzo6Y3Jl
YXRlQ2hyb21lKToKKyAgICAgICAgKExhdW5jaGVyV2luZG93Ojp0b2dnbGVKYXZhU2NyaXB0RW5h
YmxlZCk6CisgICAgICAgICogUXRUZXN0QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5oOgorICAgICAg
ICAoTGF1bmNoZXJXaW5kb3cpOgorCisyMDEzLTAxLTIxICBWaXZlayBHYWxhdGFnZSAgPHZpdmVr
LnZnQHNhbXN1bmcuY29tPgorCiAgICAgICAgIFF0VGVzdEJyb3dzZXIgc2hvdWxkIHByb3ZpZGUg
d2F5IHRvIGNsZWFyIHNlbGVjdGVkIGVsZW1lbnRzCiAgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMDc0MzcKIApkaWZmIC0tZ2l0IGEvVG9vbHMvUXRUZXN0
QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHAgYi9Ub29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVy
d2luZG93LmNwcAppbmRleCA4NTUxOTBiNzgxNjg3NThhOTE4MmMzN2MyOWYyYTVjODAwYjFiZGQ1
Li5hODllODRkOTBkMThmMzIwNzYyNWQ4NWE1ZWE3YjUzMzIxNDdlMGNlIDEwMDY0NAotLS0gYS9U
b29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVyd2luZG93LmNwcAorKysgYi9Ub29scy9RdFRlc3RC
cm93c2VyL2xhdW5jaGVyd2luZG93LmNwcApAQCAtNDk2LDYgKzQ5NiwxMCBAQCB2b2lkIExhdW5j
aGVyV2luZG93OjpjcmVhdGVDaHJvbWUoKQogI2VuZGlmCiAgICAgdG9nZ2xlU2Nyb2xsQW5pbWF0
b3ItPnNldENoZWNrZWQoZmFsc2UpOwogCisgICAgUUFjdGlvbiogdG9nZ2xlSmF2YVNjcmlwdEVu
YWJsZWQgPSBzZXR0aW5nc01lbnUtPmFkZEFjdGlvbigiRW5hYmxlIEphdmFzY3JpcHQiLCB0aGlz
LCBTTE9UKHRvZ2dsZUphdmFTY3JpcHRFbmFibGVkKGJvb2wpKSk7CisgICAgdG9nZ2xlSmF2YVNj
cmlwdEVuYWJsZWQtPnNldENoZWNrYWJsZSh0cnVlKTsKKyAgICB0b2dnbGVKYXZhU2NyaXB0RW5h
YmxlZC0+c2V0Q2hlY2tlZCh0cnVlKTsKKwogICAgIFFBY3Rpb24qIHRvZ2dsZUludGVycnVwdGlu
Z0phdmFTY3JpcHRlRW5hYmxlZCA9IHNldHRpbmdzTWVudS0+YWRkQWN0aW9uKCJFbmFibGUgaW50
ZXJydXB0aW5nIGpzIHNjcmlwdHMiLCB0aGlzLCBTTE9UKHRvZ2dsZUludGVycnVwdGluZ0phdmFT
Y3JpcHRFbmFibGVkKGJvb2wpKSk7CiAgICAgdG9nZ2xlSW50ZXJydXB0aW5nSmF2YVNjcmlwdGVF
bmFibGVkLT5zZXRDaGVja2FibGUodHJ1ZSk7CiAgICAgdG9nZ2xlSW50ZXJydXB0aW5nSmF2YVNj
cmlwdGVFbmFibGVkLT5zZXRDaGVja2VkKGZhbHNlKTsKQEAgLTkyMyw2ICs5MjcsMTIgQEAgdm9p
ZCBMYXVuY2hlcldpbmRvdzo6dG9nZ2xlRnJhbWVGbGF0dGVuaW5nKGJvb2wgdG9nZ2xlKQogICAg
IHBhZ2UoKS0+c2V0dGluZ3MoKS0+c2V0QXR0cmlidXRlKFFXZWJTZXR0aW5nczo6RnJhbWVGbGF0
dGVuaW5nRW5hYmxlZCwgdG9nZ2xlKTsKIH0KIAordm9pZCBMYXVuY2hlcldpbmRvdzo6dG9nZ2xl
SmF2YVNjcmlwdEVuYWJsZWQoYm9vbCBlbmFibGUpCit7CisgICAgcGFnZSgpLT5zZXR0aW5ncygp
LT5zZXRBdHRyaWJ1dGUoUVdlYlNldHRpbmdzOjpKYXZhc2NyaXB0RW5hYmxlZCwgZW5hYmxlKTsK
KyAgICBzdGF0dXNCYXIoKS0+c2hvd01lc3NhZ2UoUVN0cmluZygiSmF2YXNjcmlwdCAiKSArIFFT
dHJpbmcoZW5hYmxlID8gImVuYWJsZWQiIDogImRpc2FibGVkIiksIDUwMDApOworfQorCiB2b2lk
IExhdW5jaGVyV2luZG93Ojp0b2dnbGVJbnRlcnJ1cHRpbmdKYXZhU2NyaXB0RW5hYmxlZChib29s
IGVuYWJsZSkKIHsKICAgICBwYWdlKCktPnNldEludGVycnVwdGluZ0phdmFTY3JpcHRFbmFibGVk
KGVuYWJsZSk7CmRpZmYgLS1naXQgYS9Ub29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVyd2luZG93
LmggYi9Ub29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVyd2luZG93LmgKaW5kZXggZmIzMTM3ZGFj
ZmZjNzdlODcwNDc4ODFhMmE5N2ViOGMxZTU2ZjRhYi4uMTg1ZjIxYjc0MGUyNTA4OGFjZTBhMGZm
MzA3MDBjMWEwZDYyYjMxYiAxMDA2NDQKLS0tIGEvVG9vbHMvUXRUZXN0QnJvd3Nlci9sYXVuY2hl
cndpbmRvdy5oCisrKyBiL1Rvb2xzL1F0VGVzdEJyb3dzZXIvbGF1bmNoZXJ3aW5kb3cuaApAQCAt
MTYwLDYgKzE2MCw3IEBAIHByb3RlY3RlZCBRX1NMT1RTOgogICAgIHZvaWQgdG9nZ2xlU3BhdGlh
bE5hdmlnYXRpb24oYm9vbCBiKTsKICAgICB2b2lkIHRvZ2dsZUZ1bGxTY3JlZW5Nb2RlKGJvb2wg
ZW5hYmxlKTsKICAgICB2b2lkIHRvZ2dsZUZyYW1lRmxhdHRlbmluZyhib29sIHRvZ2dsZSk7Cisg
ICAgdm9pZCB0b2dnbGVKYXZhU2NyaXB0RW5hYmxlZChib29sIGVuYWJsZSk7CiAgICAgdm9pZCB0
b2dnbGVJbnRlcnJ1cHRpbmdKYXZhU2NyaXB0RW5hYmxlZChib29sIGVuYWJsZSk7CiAgICAgdm9p
ZCB0b2dnbGVKYXZhc2NyaXB0Q2FuT3BlbldpbmRvd3MoYm9vbCBlbmFibGUpOwogICAgIHZvaWQg
dG9nZ2xlQXV0b0xvYWRJbWFnZXMoYm9vbCBlbmFibGUpOwo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>187057</attachid>
            <date>2013-02-07 04:14:49 -0800</date>
            <delta_ts>2013-02-07 08:05:57 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-107461-20130207174226.patch</filename>
            <type>text/plain</type>
            <size>3020</size>
            <attacher name="Vivek Galatage">vivek.vg</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTQxOTYzCmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggZGQ1MzJhMmQ2MWEyZWU4NTU3YTgxNDUzZWZmMDJkY2U0
MjJlOGRhMy4uMWFhMGY3Zjk4MzZmZmFjY2U5NzcyZTk4NWFjMzExZWY0MzA4YzRkMCAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE5
IEBACisyMDEzLTAyLTA3ICBWaXZlayBHYWxhdGFnZSAgPHZpdmVrLnZnQHNhbXN1bmcuY29tPgor
CisgICAgICAgIFtRdF0gUXRUZXN0QnJvd3NlciBzaG91bGQgcHJvdmlkZSBvcHRpb24gdG8gZW5h
YmxlL2Rpc2FibGUgSmF2YXNjcmlwdAorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z
aG93X2J1Zy5jZ2k/aWQ9MTA3NDYxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BT
ISkuCisKKyAgICAgICAgT3B0aW9uIHRvIGVuYWJsZS9kaXNhYmxlIEphdmFzY3JpcHQgd291bGQg
YmUgaGFuZHkgb3B0aW9uIHRvIHRlc3QKKyAgICAgICAgY2VydGFpbiBmdW5jdGlvbmFsaXRpZXMg
b2Ygd2ViIHBhZ2VzIHdpdGgvd2l0aG91dCBKYXZhc2NyaXB0LgorCisgICAgICAgICogUXRUZXN0
QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHA6CisgICAgICAgIChMYXVuY2hlcldpbmRvdzo6Y3Jl
YXRlQ2hyb21lKToKKyAgICAgICAgKExhdW5jaGVyV2luZG93Ojp0b2dnbGVKYXZhU2NyaXB0RW5h
YmxlZCk6CisgICAgICAgICogUXRUZXN0QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5oOgorICAgICAg
ICAoTGF1bmNoZXJXaW5kb3cpOgorCiAyMDEzLTAyLTA1ICBHeXV5b3VuZyBLaW0gIDxneXV5b3Vu
Zy5raW1Ac2Ftc3VuZy5jb20+CiAKICAgICAgICAgW0NNYWtlXSBEb24ndCB3YXJuIHVudXNlZCBj
bWFrZSB2YXJpYWJsZXMgd2hpY2ggYXJlbid0IHVzZWQgYnkgY21ha2UgcG9ydHMKZGlmZiAtLWdp
dCBhL1Rvb2xzL1F0VGVzdEJyb3dzZXIvbGF1bmNoZXJ3aW5kb3cuY3BwIGIvVG9vbHMvUXRUZXN0
QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHAKaW5kZXggODU1MTkwYjc4MTY4NzU4YTkxODJjMzdj
MjlmMmE1YzgwMGIxYmRkNS4uYjhlMDgwYjVkNWMzMWYyNWRiMjBmODE4MDFhMTFhMWIzNGRhNTEz
YSAxMDA2NDQKLS0tIGEvVG9vbHMvUXRUZXN0QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHAKKysr
IGIvVG9vbHMvUXRUZXN0QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5jcHAKQEAgLTQ5Niw2ICs0OTYs
MTAgQEAgdm9pZCBMYXVuY2hlcldpbmRvdzo6Y3JlYXRlQ2hyb21lKCkKICNlbmRpZgogICAgIHRv
Z2dsZVNjcm9sbEFuaW1hdG9yLT5zZXRDaGVja2VkKGZhbHNlKTsKIAorICAgIFFBY3Rpb24qIHRv
Z2dsZUphdmFTY3JpcHRFbmFibGVkID0gc2V0dGluZ3NNZW51LT5hZGRBY3Rpb24oIkVuYWJsZSBK
YXZhc2NyaXB0IiwgdGhpcywgU0xPVCh0b2dnbGVKYXZhU2NyaXB0RW5hYmxlZChib29sKSkpOwor
ICAgIHRvZ2dsZUphdmFTY3JpcHRFbmFibGVkLT5zZXRDaGVja2FibGUodHJ1ZSk7CisgICAgdG9n
Z2xlSmF2YVNjcmlwdEVuYWJsZWQtPnNldENoZWNrZWQoc2V0dGluZ3MtPnRlc3RBdHRyaWJ1dGUo
UVdlYlNldHRpbmdzOjpKYXZhc2NyaXB0RW5hYmxlZCkpOworCiAgICAgUUFjdGlvbiogdG9nZ2xl
SW50ZXJydXB0aW5nSmF2YVNjcmlwdGVFbmFibGVkID0gc2V0dGluZ3NNZW51LT5hZGRBY3Rpb24o
IkVuYWJsZSBpbnRlcnJ1cHRpbmcganMgc2NyaXB0cyIsIHRoaXMsIFNMT1QodG9nZ2xlSW50ZXJy
dXB0aW5nSmF2YVNjcmlwdEVuYWJsZWQoYm9vbCkpKTsKICAgICB0b2dnbGVJbnRlcnJ1cHRpbmdK
YXZhU2NyaXB0ZUVuYWJsZWQtPnNldENoZWNrYWJsZSh0cnVlKTsKICAgICB0b2dnbGVJbnRlcnJ1
cHRpbmdKYXZhU2NyaXB0ZUVuYWJsZWQtPnNldENoZWNrZWQoZmFsc2UpOwpAQCAtOTIzLDYgKzky
NywxMSBAQCB2b2lkIExhdW5jaGVyV2luZG93Ojp0b2dnbGVGcmFtZUZsYXR0ZW5pbmcoYm9vbCB0
b2dnbGUpCiAgICAgcGFnZSgpLT5zZXR0aW5ncygpLT5zZXRBdHRyaWJ1dGUoUVdlYlNldHRpbmdz
OjpGcmFtZUZsYXR0ZW5pbmdFbmFibGVkLCB0b2dnbGUpOwogfQogCit2b2lkIExhdW5jaGVyV2lu
ZG93Ojp0b2dnbGVKYXZhU2NyaXB0RW5hYmxlZChib29sIGVuYWJsZSkKK3sKKyAgICBwYWdlKCkt
PnNldHRpbmdzKCktPnNldEF0dHJpYnV0ZShRV2ViU2V0dGluZ3M6OkphdmFzY3JpcHRFbmFibGVk
LCBlbmFibGUpOworfQorCiB2b2lkIExhdW5jaGVyV2luZG93Ojp0b2dnbGVJbnRlcnJ1cHRpbmdK
YXZhU2NyaXB0RW5hYmxlZChib29sIGVuYWJsZSkKIHsKICAgICBwYWdlKCktPnNldEludGVycnVw
dGluZ0phdmFTY3JpcHRFbmFibGVkKGVuYWJsZSk7CmRpZmYgLS1naXQgYS9Ub29scy9RdFRlc3RC
cm93c2VyL2xhdW5jaGVyd2luZG93LmggYi9Ub29scy9RdFRlc3RCcm93c2VyL2xhdW5jaGVyd2lu
ZG93LmgKaW5kZXggZmIzMTM3ZGFjZmZjNzdlODcwNDc4ODFhMmE5N2ViOGMxZTU2ZjRhYi4uMTg1
ZjIxYjc0MGUyNTA4OGFjZTBhMGZmMzA3MDBjMWEwZDYyYjMxYiAxMDA2NDQKLS0tIGEvVG9vbHMv
UXRUZXN0QnJvd3Nlci9sYXVuY2hlcndpbmRvdy5oCisrKyBiL1Rvb2xzL1F0VGVzdEJyb3dzZXIv
bGF1bmNoZXJ3aW5kb3cuaApAQCAtMTYwLDYgKzE2MCw3IEBAIHByb3RlY3RlZCBRX1NMT1RTOgog
ICAgIHZvaWQgdG9nZ2xlU3BhdGlhbE5hdmlnYXRpb24oYm9vbCBiKTsKICAgICB2b2lkIHRvZ2ds
ZUZ1bGxTY3JlZW5Nb2RlKGJvb2wgZW5hYmxlKTsKICAgICB2b2lkIHRvZ2dsZUZyYW1lRmxhdHRl
bmluZyhib29sIHRvZ2dsZSk7CisgICAgdm9pZCB0b2dnbGVKYXZhU2NyaXB0RW5hYmxlZChib29s
IGVuYWJsZSk7CiAgICAgdm9pZCB0b2dnbGVJbnRlcnJ1cHRpbmdKYXZhU2NyaXB0RW5hYmxlZChi
b29sIGVuYWJsZSk7CiAgICAgdm9pZCB0b2dnbGVKYXZhc2NyaXB0Q2FuT3BlbldpbmRvd3MoYm9v
bCBlbmFibGUpOwogICAgIHZvaWQgdG9nZ2xlQXV0b0xvYWRJbWFnZXMoYm9vbCBlbmFibGUpOwo=
</data>

          </attachment>
      

    </bug>

</bugzilla>