<?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>84566</bug_id>
          
          <creation_ts>2012-04-22 23:32:24 -0700</creation_ts>
          <short_desc>Web Inspector: [chromium] show more useful information for throttled requests.</short_desc>
          <delta_ts>2012-05-01 05:28:11 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Web Inspector (Deprecated)</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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>
          
          <blocked>79666</blocked>
          <everconfirmed>0</everconfirmed>
          <reporter>yzshen</reporter>
          <assigned_to>yzshen</assigned_to>
          <cc>abarth</cc>
    
    <cc>apavlov</cc>
    
    <cc>bweinstein</cc>
    
    <cc>dglazkov</cc>
    
    <cc>fishd</cc>
    
    <cc>jamesr</cc>
    
    <cc>joepeck</cc>
    
    <cc>joi</cc>
    
    <cc>keishi</cc>
    
    <cc>kling</cc>
    
    <cc>loislo</cc>
    
    <cc>ossy</cc>
    
    <cc>pfeldman</cc>
    
    <cc>pmuellr</cc>
    
    <cc>rik</cc>
    
    <cc>szledan</cc>
    
    <cc>timothy</cc>
    
    <cc>tkent+wkapi</cc>
    
    <cc>webkit.review.bot</cc>
    
    <cc>yurys</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>607495</commentid>
    <comment_count>0</comment_count>
    <who name="">yzshen</who>
    <bug_when>2012-04-22 23:32:24 -0700</bug_when>
    <thetext>Currently, when a request is intentionally throttled by URLRequestThrottler, it shows up as a failure in the DevTools console and the network tab, without any specific information. It may cause confusion for users because it looks the same as other failures, such as the request is sent but no response is received.

The corresponding Chromium bug is http://code.google.com/p/chromium/issues/detail?id=109857.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>607496</commentid>
    <comment_count>1</comment_count>
    <who name="">yzshen</who>
    <bug_when>2012-04-22 23:32:41 -0700</bug_when>
    <thetext>I will upload a patch soon.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>608127</commentid>
    <comment_count>2</comment_count>
      <attachid>138444</attachid>
    <who name="">yzshen</who>
    <bug_when>2012-04-23 15:51:42 -0700</bug_when>
    <thetext>Created attachment 138444
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>608134</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-04-23 15:57:27 -0700</bug_when>
    <thetext>Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>608152</commentid>
    <comment_count>4</comment_count>
    <who name="">yzshen</who>
    <bug_when>2012-04-23 16:10:34 -0700</bug_when>
    <thetext>Hi, Inspector experts.

This patch shows the description for failed requests in the Status column of the Network tab.

Currently the Status column shows things like:
successful request: &quot;304 \n Not Modified&quot;
failed request: &quot;(failed)&quot;

This patch will show the following for failed requests:
&quot;(failed) \n &lt;localizedFailDescription&gt;&quot;

To me, this adds more useful information for failures (e.g., for requests rejected by the Chromium-specific feature of HTTP throttling). And it matches the successful cases, which have already had a human-friendly description.

Please let me know if this looks okay to you. I would be happy to make changes if you have better ideas. Thanks in advance!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>608879</commentid>
    <comment_count>5</comment_count>
      <attachid>138444</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-04-24 11:33:14 -0700</bug_when>
    <thetext>Comment on attachment 138444
Patch

Clearing flags on attachment: 138444

Committed r115086: &lt;http://trac.webkit.org/changeset/115086&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>608880</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-04-24 11:33:33 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>609556</commentid>
    <comment_count>7</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-04-25 02:03:29 -0700</bug_when>
    <thetext>It made a test fail on Qt platform:

--- /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/inspector/network-status-non-http-expected.txt 
+++ /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/layout-test-results/inspector/network-status-non-http-actual.txt 
@@ -1,6 +1,6 @@
 {
     0 : &quot;data:application/javascript: Success&quot;
     1 : &quot;network-test.js: Success&quot;
-    2 : &quot;non-existent-file.js: (failed)&quot;
+    2 : &quot;non-existent-file.js: (failed)Error opening /home/webkitbuildbot/slaves/release32bit-NRWT/buildslave/qt-linux-32-release-NRWT/build/LayoutTests/inspector/non-existent-file.js: No such file or directory&quot;
 }
 
Could you check it, please?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>609666</commentid>
    <comment_count>8</comment_count>
    <who name="Szilard Ledan">szledan</who>
    <bug_when>2012-04-25 05:29:50 -0700</bug_when>
    <thetext>This patch inspector/network-status-non-http.html fails on qt. This test has been skipped until it is fixed. See http://trac.webkit.org/changeset/115188
Please unskip it with the proper fix.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>609838</commentid>
    <comment_count>9</comment_count>
    <who name="">yzshen</who>
    <bug_when>2012-04-25 09:40:18 -0700</bug_when>
    <thetext>Sorry for the inconvenience.
I will fix it as soon as possible.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>610272</commentid>
    <comment_count>10</comment_count>
      <attachid>138916</attachid>
    <who name="">yzshen</who>
    <bug_when>2012-04-25 18:21:57 -0700</bug_when>
    <thetext>Created attachment 138916
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>610276</commentid>
    <comment_count>11</comment_count>
      <attachid>138916</attachid>
    <who name="">yzshen</who>
    <bug_when>2012-04-25 18:23:54 -0700</bug_when>
    <thetext>Comment on attachment 138916
Patch

This patch fixes the test regression. Please take a look.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>610347</commentid>
    <comment_count>12</comment_count>
      <attachid>138916</attachid>
    <who name="Yury Semikhatsky">yurys</who>
    <bug_when>2012-04-25 22:04:07 -0700</bug_when>
    <thetext>Comment on attachment 138916
Patch

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

&gt; LayoutTests/inspector/network-status-non-http.html:24
&gt; +                if (outputStatus.indexOf(&quot;(failed)&quot;) == 0)

Please use .startsWith for prefix checks.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>610694</commentid>
    <comment_count>13</comment_count>
    <who name="">yzshen</who>
    <bug_when>2012-04-26 10:03:31 -0700</bug_when>
    <thetext>(In reply to comment #12)
&gt; (From update of attachment 138916 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=138916&amp;action=review
&gt; 
&gt; &gt; LayoutTests/inspector/network-status-non-http.html:24
&gt; &gt; +                if (outputStatus.indexOf(&quot;(failed)&quot;) == 0)
&gt; 
&gt; Please use .startsWith for prefix checks.

Javascript string doesn&apos;t have .startsWith method, does it?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>612192</commentid>
    <comment_count>14</comment_count>
      <attachid>138916</attachid>
    <who name="Pavel Feldman">pfeldman</who>
    <bug_when>2012-04-28 00:31:09 -0700</bug_when>
    <thetext>Comment on attachment 138916
Patch

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

&gt;&gt;&gt; LayoutTests/inspector/network-status-non-http.html:24
&gt;&gt;&gt; +                if (outputStatus.indexOf(&quot;(failed)&quot;) == 0)
&gt;&gt; 
&gt;&gt; Please use .startsWith for prefix checks.
&gt; 
&gt; Javascript string doesn&apos;t have .startsWith method, does it?

We added it to the String&apos;s prototype in utilities.js</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>612626</commentid>
    <comment_count>15</comment_count>
      <attachid>138916</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-04-30 07:44:59 -0700</bug_when>
    <thetext>Comment on attachment 138916
Patch

Rejecting attachment 138916 from commit-queue.

Failed to run &quot;[&apos;/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch&apos;, &apos;--status-host=queues.webkit.org&apos;, &apos;-...&quot; exit_code: 2

Last 500 characters of output:
ueue/

Parsed 3 diffs from patch file(s).
patching file LayoutTests/ChangeLog
Hunk #1 succeeded at 1 with fuzz 3.
patching file LayoutTests/inspector/network-status-non-http.html
patching file LayoutTests/platform/qt/Skipped
Hunk #1 FAILED at 2616.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/qt/Skipped.rej

Failed to run &quot;[u&apos;/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply&apos;, u&apos;--force&apos;, u&apos;--reviewer&apos;, u&apos;Pavel Feld...&quot; exit_code: 1 cwd: /mnt/git/webkit-commit-queue/

Full output: http://queues.webkit.org/results/12597070</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>612882</commentid>
    <comment_count>16</comment_count>
    <who name="">yzshen</who>
    <bug_when>2012-04-30 12:55:35 -0700</bug_when>
    <thetext>(In reply to comment #14)
&gt; (From update of attachment 138916 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=138916&amp;action=review
&gt; 
&gt; &gt;&gt;&gt; LayoutTests/inspector/network-status-non-http.html:24
&gt; &gt;&gt;&gt; +                if (outputStatus.indexOf(&quot;(failed)&quot;) == 0)
&gt; &gt;&gt; 
&gt; &gt;&gt; Please use .startsWith for prefix checks.
&gt; &gt; 
&gt; &gt; Javascript string doesn&apos;t have .startsWith method, does it?
&gt; 
&gt; We added it to the String&apos;s prototype in utilities.js

(Sorry I am not familiar with this part of codebase.)
Is it allowed to directly refer to utilities.js from the LayoutTests folder?
Or we have other ways to access startsWith?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>613348</commentid>
    <comment_count>17</comment_count>
    <who name="Andreas Kling">kling</who>
    <bug_when>2012-05-01 05:28:11 -0700</bug_when>
    <thetext>Committed r115720: &lt;http://trac.webkit.org/changeset/115720&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>138444</attachid>
            <date>2012-04-23 15:51:42 -0700</date>
            <delta_ts>2012-04-25 18:21:52 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-84566-20120423155141.patch</filename>
            <type>text/plain</type>
            <size>4873</size>
            <attacher>yzshen</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTE0ODYyCmRpZmYgLS1naXQgYS9Tb3VyY2UvUGxhdGZvcm0v
Q2hhbmdlTG9nIGIvU291cmNlL1BsYXRmb3JtL0NoYW5nZUxvZwppbmRleCBiNjg4YTE0MTUwNzNl
NTA2NGY1YzcxNmY3MGU4MTUwNDc2MDRlY2Y4Li5kMTM1OWZjY2E4YWI3MzRlYzNkYTdmYWRhOWVm
MWNkNzJlMTVhYmE4IDEwMDY0NAotLS0gYS9Tb3VyY2UvUGxhdGZvcm0vQ2hhbmdlTG9nCisrKyBi
L1NvdXJjZS9QbGF0Zm9ybS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAxMi0wNC0yMyAg
WXV6aHUgU2hlbiAgPHl6c2hlbkBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgW2Nocm9taXVtXSBB
ZGQgYSBkZXNjcmlwdGlvbiBmaWVsZCBpbiBXZWJVUkxFcnJvciBhbmQgc2hvdyBmYWlsdXJlIGRl
c2NyaXB0aW9uIGluIHRoZSBpbnNwZWN0b3IgbmV0d29yayB0YWIuCisgICAgICAgIGh0dHBzOi8v
YnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04NDU2NgorCisgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogY2hyb21pdW0vcHVibGljL1dlYlVSTEVy
cm9yLmg6CisgICAgICAgIChXZWJVUkxFcnJvcik6CisKIDIwMTItMDQtMjAgIERhbmEgSmFuc2Vu
cyAgPGRhbmFrakBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgW2Nocm9taXVtXSBXZWJGaWx0ZXJP
cGVyYXRpb25zIEFQSSBkb2VzIG5vdCBjb21waWxlIG9yIGR5bmFtaWNhbGx5IGxpbmsKZGlmZiAt
LWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxv
ZwppbmRleCAwYjVlZGY0ZTY4MDA3YThjNTU3MDBhNDE1NTNmOGIyZTA2YjA0ZjM4Li43NTJmNjE5
M2RkZGY2ZTA2Nzg3NTk4YWVlYzNlMTY0MmFlYzBkNDAwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEs
MTYgQEAKKzIwMTItMDQtMjMgIFl1emh1IFNoZW4gIDx5enNoZW5AY2hyb21pdW0ub3JnPgorCisg
ICAgICAgIFtjaHJvbWl1bV0gQWRkIGEgZGVzY3JpcHRpb24gZmllbGQgaW4gV2ViVVJMRXJyb3Ig
YW5kIHNob3cgZmFpbHVyZSBkZXNjcmlwdGlvbiBpbiB0aGUgaW5zcGVjdG9yIG5ldHdvcmsgdGFi
LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODQ1NjYK
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICAqIGluc3Bl
Y3Rvci9mcm9udC1lbmQvTmV0d29ya1BhbmVsLmpzOgorICAgICAgICAoV2ViSW5zcGVjdG9yLk5l
dHdvcmtEYXRhR3JpZE5vZGUucHJvdG90eXBlLl9yZWZyZXNoU3RhdHVzQ2VsbCk6CisgICAgICAg
ICogcGxhdGZvcm0vY2hyb21pdW0vc3VwcG9ydC9XZWJVUkxFcnJvci5jcHA6CisgICAgICAgIChX
ZWJLaXQ6OldlYlVSTEVycm9yOjpvcGVyYXRvcj0pOgorICAgICAgICAoV2ViS2l0OjpXZWJVUkxF
cnJvcjo6b3BlcmF0b3IgUmVzb3VyY2VFcnJvcik6CisKIDIwMTItMDQtMjIgIFNoYXduIFNpbmdo
ICA8c2hhd25zaW5naEBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgW2Nocm9taXVtXSBEYW1hZ2Ug
VHJhY2tlciBuZWVkcyB0byB1c2UgQ0NNYXRoVXRpbCB0cmFuc2Zvcm1zCmRpZmYgLS1naXQgYS9T
b3VyY2UvUGxhdGZvcm0vY2hyb21pdW0vcHVibGljL1dlYlVSTEVycm9yLmggYi9Tb3VyY2UvUGxh
dGZvcm0vY2hyb21pdW0vcHVibGljL1dlYlVSTEVycm9yLmgKaW5kZXggODYyZGJlYWEzYjNjMGYy
OGUyNjk0ZjhkOGI2ZmIxOGEyZDFlODUwZi4uYjZlYjg5MzU3NDRlMzIxYWRkM2UzNWZmZDYxODc4
NGU5YTBiZDhmOSAxMDA2NDQKLS0tIGEvU291cmNlL1BsYXRmb3JtL2Nocm9taXVtL3B1YmxpYy9X
ZWJVUkxFcnJvci5oCisrKyBiL1NvdXJjZS9QbGF0Zm9ybS9jaHJvbWl1bS9wdWJsaWMvV2ViVVJM
RXJyb3IuaApAQCAtNTksNiArNTksOSBAQCBzdHJ1Y3QgV2ViVVJMRXJyb3IgewogICAgIC8vIFRo
ZSB1cmwgdGhhdCBmYWlsZWQgdG8gbG9hZC4KICAgICBXZWJVUkwgdW5yZWFjaGFibGVVUkw7CiAK
KyAgICAvLyBBIGRlc2NyaXB0aW9uIGZvciB0aGUgZXJyb3IuCisgICAgV2ViU3RyaW5nIGxvY2Fs
aXplZERlc2NyaXB0aW9uOworCiAgICAgV2ViVVJMRXJyb3IoKSA6IHJlYXNvbigwKSwgaXNDYW5j
ZWxsYXRpb24oZmFsc2UpIHsgfQogCiAjaWYgZGVmaW5lZChXRUJLSVRfSU1QTEVNRU5UQVRJT04p
CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL05ldHdvcmtQ
YW5lbC5qcyBiL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvTmV0d29ya1BhbmVs
LmpzCmluZGV4IDNiZWI1N2Y5MjkyMGNiYmZiZDA2ZDljNGMxYzczYWM5M2Y3ZjZhZWUuLmY3ZDRj
ZTBlY2I0ZjE5NGMwMDI1OTZiMzcxYmRiYTRhZTAzNmE1ZjUgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X
ZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvTmV0d29ya1BhbmVsLmpzCisrKyBiL1NvdXJjZS9X
ZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvTmV0d29ya1BhbmVsLmpzCkBAIC0xODU4LDEwICsx
ODU4LDE0IEBAIFdlYkluc3BlY3Rvci5OZXR3b3JrRGF0YUdyaWROb2RlLnByb3RvdHlwZSA9IHsK
ICAgICAgICAgdGhpcy5fc3RhdHVzQ2VsbC5yZW1vdmVDaGlsZHJlbigpOwogCiAgICAgICAgIGlm
ICh0aGlzLl9yZXF1ZXN0LmZhaWxlZCkgewotICAgICAgICAgICAgaWYgKHRoaXMuX3JlcXVlc3Qu
Y2FuY2VsZWQpCi0gICAgICAgICAgICAgICAgdGhpcy5fc3RhdHVzQ2VsbC5zZXRUZXh0QW5kVGl0
bGUoV2ViSW5zcGVjdG9yLlVJU3RyaW5nKCIoY2FuY2VsZWQpIikpOwotICAgICAgICAgICAgZWxz
ZQotICAgICAgICAgICAgICAgIHRoaXMuX3N0YXR1c0NlbGwuc2V0VGV4dEFuZFRpdGxlKFdlYklu
c3BlY3Rvci5VSVN0cmluZygiKGZhaWxlZCkiKSk7CisgICAgICAgICAgICB2YXIgZmFpbFRleHQg
PSB0aGlzLl9yZXF1ZXN0LmNhbmNlbGVkID8gV2ViSW5zcGVjdG9yLlVJU3RyaW5nKCIoY2FuY2Vs
ZWQpIikgOiBXZWJJbnNwZWN0b3IuVUlTdHJpbmcoIihmYWlsZWQpIik7CisgICAgICAgICAgICBp
ZiAodGhpcy5fcmVxdWVzdC5sb2NhbGl6ZWRGYWlsRGVzY3JpcHRpb24pIHsKKyAgICAgICAgICAg
ICAgICB0aGlzLl9zdGF0dXNDZWxsLmFwcGVuZENoaWxkKGRvY3VtZW50LmNyZWF0ZVRleHROb2Rl
KGZhaWxUZXh0KSk7CisgICAgICAgICAgICAgICAgdGhpcy5fYXBwZW5kU3VidGl0bGUodGhpcy5f
c3RhdHVzQ2VsbCwgdGhpcy5fcmVxdWVzdC5sb2NhbGl6ZWRGYWlsRGVzY3JpcHRpb24pOworICAg
ICAgICAgICAgICAgIHRoaXMuX3N0YXR1c0NlbGwudGl0bGUgPSBmYWlsVGV4dCArICIgIiArIHRo
aXMuX3JlcXVlc3QubG9jYWxpemVkRmFpbERlc2NyaXB0aW9uOworICAgICAgICAgICAgfSBlbHNl
IHsKKyAgICAgICAgICAgICAgICB0aGlzLl9zdGF0dXNDZWxsLnNldFRleHRBbmRUaXRsZShmYWls
VGV4dCk7CisgICAgICAgICAgICB9CiAgICAgICAgICAgICB0aGlzLl9zdGF0dXNDZWxsLmFkZFN0
eWxlQ2xhc3MoIm5ldHdvcmstZGltLWNlbGwiKTsKICAgICAgICAgICAgIHRoaXMuZWxlbWVudC5h
ZGRTdHlsZUNsYXNzKCJuZXR3b3JrLWVycm9yLXJvdyIpOwogICAgICAgICAgICAgcmV0dXJuOwpk
aWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vY2hyb21pdW0vc3VwcG9ydC9XZWJV
UkxFcnJvci5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9jaHJvbWl1bS9zdXBwb3J0L1dl
YlVSTEVycm9yLmNwcAppbmRleCBlYzMwYjlmMTI5NzJjOWQyMGEzMDlhODY5ZDhlNWVmOTgwZmZj
OTU0Li4xMzQxZGVlZTY5YmUwMTlkZDc5ZTdjNzRlZjUxOTFmNTY3ODU0ZGI1IDEwMDY0NAotLS0g
YS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9jaHJvbWl1bS9zdXBwb3J0L1dlYlVSTEVycm9yLmNw
cAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9jaHJvbWl1bS9zdXBwb3J0L1dlYlVSTEVy
cm9yLmNwcApAQCAtNTMsNiArNTMsNyBAQCBXZWJVUkxFcnJvciYgV2ViVVJMRXJyb3I6Om9wZXJh
dG9yPShjb25zdCBSZXNvdXJjZUVycm9yJiBlcnJvcikKICAgICAgICAgcmVhc29uID0gZXJyb3Iu
ZXJyb3JDb2RlKCk7CiAgICAgICAgIHVucmVhY2hhYmxlVVJMID0gS1VSTChQYXJzZWRVUkxTdHJp
bmcsIGVycm9yLmZhaWxpbmdVUkwoKSk7CiAgICAgICAgIGlzQ2FuY2VsbGF0aW9uID0gZXJyb3Iu
aXNDYW5jZWxsYXRpb24oKTsKKyAgICAgICAgbG9jYWxpemVkRGVzY3JpcHRpb24gPSBlcnJvci5s
b2NhbGl6ZWREZXNjcmlwdGlvbigpOwogICAgIH0KICAgICByZXR1cm4gKnRoaXM7CiB9CkBAIC02
NCw3ICs2NSw4IEBAIFdlYlVSTEVycm9yOjpvcGVyYXRvciBSZXNvdXJjZUVycm9yKCkgY29uc3QK
ICAgICBDU3RyaW5nIHNwZWMgPSB1bnJlYWNoYWJsZVVSTC5zcGVjKCk7CiAgICAgUmVzb3VyY2VF
cnJvciByZXNvdXJjZUVycm9yID0gUmVzb3VyY2VFcnJvcihkb21haW4sIHJlYXNvbiwKICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIFN0cmluZzo6ZnJvbVVU
Rjgoc3BlYy5kYXRhKCksCi0gICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICBzcGVjLmxlbmd0aCgpKSwgU3RyaW5nKCkpOworICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgc3BlYy5sZW5ndGgoKSksCisgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBsb2NhbGl6ZWREZXNjcmlwdGlvbik7
CiAgICAgcmVzb3VyY2VFcnJvci5zZXRJc0NhbmNlbGxhdGlvbihpc0NhbmNlbGxhdGlvbik7CiAg
ICAgcmV0dXJuIHJlc291cmNlRXJyb3I7CiB9Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>138916</attachid>
            <date>2012-04-25 18:21:57 -0700</date>
            <delta_ts>2012-04-30 07:44:59 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-84566-20120425182156.patch</filename>
            <type>text/plain</type>
            <size>2487</size>
            <attacher>yzshen</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTE1MjMyCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggMjdkMzZhMzU2MjQ1MWRhMzRlOTZl
M2RlYzJjODAxNjcxODU5NWE1Ni4uMTU2MTlhYTM3MWZkNDEzN2YwZGFlZDZkZjdjZGYwMmNiZTNi
NzNjMyAxMDA2NDQKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEyLTA0LTI1ICBZdXpodSBTaGVuICA8eXpz
aGVuQGNocm9taXVtLm9yZz4KKworICAgICAgICBGaXggdGVzdCByZWdyZXNzaW9uIGludHJvZHVj
ZWQgYnkgcjExNTA4Ni4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcu
Y2dpP2lkPTg0NTY2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg
ICAgICAgKiBpbnNwZWN0b3IvbmV0d29yay1zdGF0dXMtbm9uLWh0dHAuaHRtbDoKKyAgICAgICAg
KiBwbGF0Zm9ybS9xdC9Ta2lwcGVkOgorCiAyMDEyLTA0LTI1ICBBbHBoYSBMYW0gIDxoY2xhbUBj
aHJvbWl1bS5vcmc+CiAKICAgICAgICAgVW5yZXZpZXdlZCB0ZXN0IGV4cGVjdGF0aW9ucyB1cGRh
dGUuCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9pbnNwZWN0b3IvbmV0d29yay1zdGF0dXMtbm9u
LWh0dHAuaHRtbCBiL0xheW91dFRlc3RzL2luc3BlY3Rvci9uZXR3b3JrLXN0YXR1cy1ub24taHR0
cC5odG1sCm9sZCBtb2RlIDEwMDY0NApuZXcgbW9kZSAxMDA3NTUKaW5kZXggZDVlNmRiOWZiNDFm
ZjhmY2M2Y2JjN2VlYWJhOTZmNmY2OTE4MDIxMi4uMTEyY2FiMWM1YzllOTFhNGQzMTYxMDkxMjc3
NmRjYjE5MTVhMDBjZQotLS0gYS9MYXlvdXRUZXN0cy9pbnNwZWN0b3IvbmV0d29yay1zdGF0dXMt
bm9uLWh0dHAuaHRtbAorKysgYi9MYXlvdXRUZXN0cy9pbnNwZWN0b3IvbmV0d29yay1zdGF0dXMt
bm9uLWh0dHAuaHRtbApAQCAtMTksNyArMTksMTEgQEAgZnVuY3Rpb24gdGVzdCgpIHsKICAgICAg
ICAgICAgIGlmIChyZXF1ZXN0c1RvV2F0Y2gudGVzdCh1cmwpKSB7CiAgICAgICAgICAgICAgICAg
dmFyIHVybEZyYWdtZW50ID0gL15kYXRhOi8udGVzdCh1cmwpID8gdXJsLnJlcGxhY2UoLywuKiQv
LCAiIikgOiB1cmwucmVwbGFjZSgvLipcLy8sICIiKTsKICAgICAgICAgICAgICAgICB2YXIgc3Rh
dHVzID0gZG9jdW1lbnQuZXZhbHVhdGUoImFuY2VzdG9yOjp0ci90ZFtwb3NpdGlvbigpPTNdIiwg
dXJscy5zbmFwc2hvdEl0ZW0oaSksIG51bGwsIFhQYXRoUmVzdWx0LlNUUklOR19UWVBFLCBudWxs
KTsKLSAgICAgICAgICAgICAgICBvdXRwdXRTdHJpbmdzLnB1c2godXJsRnJhZ21lbnQgKyAiOiAi
ICsgc3RhdHVzLnN0cmluZ1ZhbHVlKTsKKyAgICAgICAgICAgICAgICB2YXIgb3V0cHV0U3RhdHVz
ID0gc3RhdHVzLnN0cmluZ1ZhbHVlOworICAgICAgICAgICAgICAgIC8vIFRydW5jYXRlIHRoZSBz
dGF0dXMgc3RyaW5nIG9uIGZhaWx1cmUsIHNpbmNlIHRoZSBkZXNjcmlwdGlvbiBmb2xsb3dpbmcg
IihmYWlsZWQpIiBpcyBwbGF0Zm9ybS1kZXBlbmRlbnQuCisgICAgICAgICAgICAgICAgaWYgKG91
dHB1dFN0YXR1cy5pbmRleE9mKCIoZmFpbGVkKSIpID09IDApCisgICAgICAgICAgICAgICAgICAg
IG91dHB1dFN0YXR1cyA9ICIoZmFpbGVkKSI7CisgICAgICAgICAgICAgICAgb3V0cHV0U3RyaW5n
cy5wdXNoKHVybEZyYWdtZW50ICsgIjogIiArIG91dHB1dFN0YXR1cyk7CiAgICAgICAgICAgICB9
CiAgICAgICAgIH0KICAgICAgICAgb3V0cHV0U3RyaW5ncy5zb3J0KCk7CmRpZmYgLS1naXQgYS9M
YXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9Ta2lwcGVkIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQv
U2tpcHBlZAppbmRleCBlN2RkYWNhODFmMzBlYjI1YTI1MzljYjU4ODY1NjhlM2Y4NzQzNGViLi4y
ZjNjOWRiMTBhMjE1ZTU3MTNmYmVkMzc3MDgxNjY0ZWFjYzY2ZTg0IDEwMDY0NAotLS0gYS9MYXlv
dXRUZXN0cy9wbGF0Zm9ybS9xdC9Ta2lwcGVkCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0
L1NraXBwZWQKQEAgLTI2MTYsNyArMjYxNiwzIEBAIHBsdWdpbnMvbnBydW50aW1lL29iamVjdC1m
cm9tLWRlc3Ryb3llZC1wbHVnaW4taW4tc3ViZnJhbWUuaHRtbAogIyBbUXRdIE5ldyBpbnNwZWN0
b3IvZGVidWdnZXIvbGlua2lmaWVyLmh0bWwgaXMgZmxha2V5IGZyb20gdGhlIGJlZ2lubmluZyAo
cjExNTA2NCkuCiAjIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04NDg2
NgogaW5zcGVjdG9yL2RlYnVnZ2VyL2xpbmtpZmllci5odG1sCi0KLSMgW1F0XSBSRUdSRVNTSU9O
KHIxMTUwODYpOiBpbnNwZWN0b3IvbmV0d29yay1zdGF0dXMtbm9uLWh0dHAuaHRtbCBpcyBmYWls
aW5nCi0jIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04NDU2NgotaW5z
cGVjdG9yL25ldHdvcmstc3RhdHVzLW5vbi1odHRwLmh0bWwK
</data>
<flag name="review"
          id="144415"
          type_id="1"
          status="+"
          setter="pfeldman"
    />
    <flag name="commit-queue"
          id="144417"
          type_id="3"
          status="-"
          setter="webkit.review.bot"
    />
          </attachment>
      

    </bug>

</bugzilla>