<?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>49691</bug_id>
          
          <creation_ts>2010-11-17 15:22:03 -0800</creation_ts>
          <short_desc>[NRWT] Remove a &apos;uri&apos; member from TestInput class</short_desc>
          <delta_ts>2010-11-18 15:40:58 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</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="Hayato Ito">hayato</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dpranke</cc>
    
    <cc>hamaji</cc>
    
    <cc>tony</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>310560</commentid>
    <comment_count>0</comment_count>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-17 15:22:03 -0800</bug_when>
    <thetext>Remove uri member from TestInput class because uri member is not used anywhere.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>310561</commentid>
    <comment_count>1</comment_count>
      <attachid>74162</attachid>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-17 15:25:17 -0800</bug_when>
    <thetext>Created attachment 74162
remove-uri</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>310623</commentid>
    <comment_count>2</comment_count>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-17 17:08:19 -0800</bug_when>
    <thetext>After this patch, I&apos;ll make another patch which makes Driver.run_test() take only one parameter, TestInput, to simplify the interface of Driver.run_test().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>310630</commentid>
    <comment_count>3</comment_count>
      <attachid>74162</attachid>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-11-17 17:13:46 -0800</bug_when>
    <thetext>Comment on attachment 74162
remove-uri

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

LGTM otherwise.

&gt; WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:-175
&gt; -    test_input.uri = port.filename_to_uri(test_input.filename).strip()

When you do make run_test() take a single input, just stuff this down into the port/* implementations so that the ports that do need to convert the test name to the uri can do so, and the ports that don&apos;t (e.g., the test port, the dryrun port), don&apos;t need to convert back.

&gt; WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:175
&gt; +    uri = port.filename_to_uri(test_input.filename).strip()

Nit: You don&apos;t actually need the strip() . filename_to_uri() doesn&apos;t append any whitespace. I&apos;m not sure why this crept in at some point, and I&apos;ve removed it in the patches-that-have-not-yet-been-uploaded-for-review :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>310639</commentid>
    <comment_count>4</comment_count>
      <attachid>74183</attachid>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-17 17:35:27 -0800</bug_when>
    <thetext>Created attachment 74183
remove-strip</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>310643</commentid>
    <comment_count>5</comment_count>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-17 17:42:18 -0800</bug_when>
    <thetext>Thank you for the review.
I&apos;ve updated patch. &apos;strip()&apos; has been removed.

(In reply to comment #3)
&gt; (From update of attachment 74162 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=74162&amp;action=review
&gt; 
&gt; LGTM otherwise.
&gt; 
&gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:-175
&gt; &gt; -    test_input.uri = port.filename_to_uri(test_input.filename).strip()
&gt; 
&gt; When you do make run_test() take a single input, just stuff this down into the port/* implementations so that the ports that do need to convert the test name to the uri can do so, and the ports that don&apos;t (e.g., the test port, the dryrun port), don&apos;t need to convert back.

Thank you. I&apos;ll do it for each ports. I don&apos;t think that is difficult work. 

&gt; 
&gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py:175
&gt; &gt; +    uri = port.filename_to_uri(test_input.filename).strip()
&gt; 
&gt; Nit: You don&apos;t actually need the strip() . filename_to_uri() doesn&apos;t append any whitespace. I&apos;m not sure why this crept in at some point, and I&apos;ve removed it in the patches-that-have-not-yet-been-uploaded-for-review :)

Done. I&apos;ve removed &apos;strip()&apos; in the patch-that-has-been-uploaded. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>310649</commentid>
    <comment_count>6</comment_count>
      <attachid>74183</attachid>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-17 18:02:49 -0800</bug_when>
    <thetext>Comment on attachment 74183
remove-strip

Clearing flags on attachment: 74183

Committed r72267: &lt;http://trac.webkit.org/changeset/72267&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>310650</commentid>
    <comment_count>7</comment_count>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-17 18:02:58 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>311121</commentid>
    <comment_count>8</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2010-11-18 15:20:38 -0800</bug_when>
    <thetext>There have been some failures on WebKit Win after this patch landed.  Do you think they might be related?  Here&apos;s a sample crash:
http://build.chromium.org/p/chromium/builders/Webkit%20Win/builds/1788/steps/webkit_tests/logs/stdio

2010-11-18 13:46:53,076 20544 dump_render_tree_thread.py:441  DEBUGThread-1 http/tests/xmlhttprequest/access-control-basic-post-fail-non-simple-content-type.html passed
2010-11-18 13:46:53,201 20544 executive.py:357  DEBUG&quot;e:\b\build\slave\Webkit_Win\build\src\third_party\cygwin\bin\wdiff.exe --start-delete=##WDIFF_DEL## --end-delete=##WDIFF_END## --start-insert=##WDIFF_ADD## --end-insert=##WDIFF_END## e:\b\build\slave\Webkit_Win\build\src\webkit\Release\..\../../layout-test-results\platform/mac/editing/selection/25228-expected.txt e:\b\build\slave\Webkit_Win\build\src\webkit\Release\..\../../layout-test-results\platform/mac/editing/selection/25228-actual.txt&quot; took 0.06s
2010-11-18 13:46:53,233 20544 dump_render_tree_thread.py:441  DEBUGThread-1 http/tests/xmlhttprequest/access-control-basic-whitelist-request-headers.html passed
2010-11-18 13:46:53,390 20544 dump_render_tree_thread.py:441  DEBUGThread-1 http/tests/xmlhttprequest/access-control-basic-whitelist-response-headers.html passed
2010-11-18 13:46:53,437 20544 executive.py:357  DEBUG&quot;e:\b\build\slave\Webkit_Win\build\src\webkit\Release\image_diff.exe --diff c:\docume~1\chrome~2\locals~1\temp\tmpi4wul2\expected.png c:\docume~1\chrome~2\locals~1\temp\tmpi4wul2\actual.png e:\b\build\slave\Webkit_Win\build\src\webkit\Release\..\../../layout-test-results\platform/mac/editing/selection/25228-diff.png&quot; took 0.22s
2010-11-18 13:46:53,437 20544 dump_render_tree_thread.py:438  DEBUGThread-4 platform/mac/editing/selection/25228.html failed:
  Text diff mismatch
  Image mismatch
2010-11-18 13:46:56,358 20544 dump_render_tree_thread.py:340  ERRORThread-2 dying, exception raised
Traceback (most recent call last):
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\new-run-webkit-tests&quot;, line 38, in &lt;module&gt;
    sys.exit(run_webkit_tests.main())
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 1684, in main
    return run(port_obj, options, args)
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 1373, in run
    num_unexpected_results = test_runner.run(result_summary)
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 759, in run
    self._run_tests(self._test_files_list, result_summary))
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 639, in _run_tests
    result_summary)
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 330, in _covered_run
    self._run(test_runner=None, result_summary=None)
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 422, in _run
    result = self._run_test(test_input)
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 517, in _run_test
    self._driver)
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 176, in _run_single_test
    test_output = driver.run_test(uri, test_input.timeout, image_hash_to_driver)
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\port\chromium.py&quot;, line 509, in run_test
    &apos;&apos;.join(output), self._output_image(), actual_checksum,
  File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\port\chromium.py&quot;, line 452, in _output_image
    with open(png_path, &apos;rb&apos;) as image_file:
IOError: [Errno 13] Permission denied: &apos;e:\\b\\build\\slave\\Webkit_Win\\build\\src\\webkit\\Release\\..\\../../layout-test-results\\png_result1.png&apos;
No handlers could be found for logger &quot;webkitpy.common.system&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>311133</commentid>
    <comment_count>9</comment_count>
    <who name="Hayato Ito">hayato</who>
    <bug_when>2010-11-18 15:40:58 -0800</bug_when>
    <thetext>At first glance, I have no ideas why &apos;Permission Denied&apos; occurs on only Windows.
That might be related with my change. I&apos;ll investigate that.

(In reply to comment #8)
&gt; There have been some failures on WebKit Win after this patch landed.  Do you think they might be related?  Here&apos;s a sample crash:
&gt; http://build.chromium.org/p/chromium/builders/Webkit%20Win/builds/1788/steps/webkit_tests/logs/stdio
&gt; 
&gt; 2010-11-18 13:46:53,076 20544 dump_render_tree_thread.py:441  DEBUGThread-1 http/tests/xmlhttprequest/access-control-basic-post-fail-non-simple-content-type.html passed
&gt; 2010-11-18 13:46:53,201 20544 executive.py:357  DEBUG&quot;e:\b\build\slave\Webkit_Win\build\src\third_party\cygwin\bin\wdiff.exe --start-delete=##WDIFF_DEL## --end-delete=##WDIFF_END## --start-insert=##WDIFF_ADD## --end-insert=##WDIFF_END## e:\b\build\slave\Webkit_Win\build\src\webkit\Release\..\../../layout-test-results\platform/mac/editing/selection/25228-expected.txt e:\b\build\slave\Webkit_Win\build\src\webkit\Release\..\../../layout-test-results\platform/mac/editing/selection/25228-actual.txt&quot; took 0.06s
&gt; 2010-11-18 13:46:53,233 20544 dump_render_tree_thread.py:441  DEBUGThread-1 http/tests/xmlhttprequest/access-control-basic-whitelist-request-headers.html passed
&gt; 2010-11-18 13:46:53,390 20544 dump_render_tree_thread.py:441  DEBUGThread-1 http/tests/xmlhttprequest/access-control-basic-whitelist-response-headers.html passed
&gt; 2010-11-18 13:46:53,437 20544 executive.py:357  DEBUG&quot;e:\b\build\slave\Webkit_Win\build\src\webkit\Release\image_diff.exe --diff c:\docume~1\chrome~2\locals~1\temp\tmpi4wul2\expected.png c:\docume~1\chrome~2\locals~1\temp\tmpi4wul2\actual.png e:\b\build\slave\Webkit_Win\build\src\webkit\Release\..\../../layout-test-results\platform/mac/editing/selection/25228-diff.png&quot; took 0.22s
&gt; 2010-11-18 13:46:53,437 20544 dump_render_tree_thread.py:438  DEBUGThread-4 platform/mac/editing/selection/25228.html failed:
&gt;   Text diff mismatch
&gt;   Image mismatch
&gt; 2010-11-18 13:46:56,358 20544 dump_render_tree_thread.py:340  ERRORThread-2 dying, exception raised
&gt; Traceback (most recent call last):
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\new-run-webkit-tests&quot;, line 38, in &lt;module&gt;
&gt;     sys.exit(run_webkit_tests.main())
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 1684, in main
&gt;     return run(port_obj, options, args)
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 1373, in run
&gt;     num_unexpected_results = test_runner.run(result_summary)
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 759, in run
&gt;     self._run_tests(self._test_files_list, result_summary))
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\run_webkit_tests.py&quot;, line 639, in _run_tests
&gt;     result_summary)
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 330, in _covered_run
&gt;     self._run(test_runner=None, result_summary=None)
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 422, in _run
&gt;     result = self._run_test(test_input)
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 517, in _run_test
&gt;     self._driver)
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\layout_package\dump_render_tree_thread.py&quot;, line 176, in _run_single_test
&gt;     test_output = driver.run_test(uri, test_input.timeout, image_hash_to_driver)
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\port\chromium.py&quot;, line 509, in run_test
&gt;     &apos;&apos;.join(output), self._output_image(), actual_checksum,
&gt;   File &quot;e:\b\build\slave\Webkit_Win\build\src\third_party\WebKit\WebKitTools\Scripts\webkitpy\layout_tests\port\chromium.py&quot;, line 452, in _output_image
&gt;     with open(png_path, &apos;rb&apos;) as image_file:
&gt; IOError: [Errno 13] Permission denied: &apos;e:\\b\\build\\slave\\Webkit_Win\\build\\src\\webkit\\Release\\..\\../../layout-test-results\\png_result1.png&apos;
&gt; No handlers could be found for logger &quot;webkitpy.common.system&quot;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>74162</attachid>
            <date>2010-11-17 15:25:17 -0800</date>
            <delta_ts>2010-11-17 17:35:22 -0800</delta_ts>
            <desc>remove-uri</desc>
            <filename>bug-49691-20101118082515.patch</filename>
            <type>text/plain</type>
            <size>3906</size>
            <attacher name="Hayato Ito">hayato</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBjZGFjMWRhMzFiNzVmZTQyZTgzOWM3OWNiM2ZmOWE2YjAxODI3NDViLi4xYmExMTMw
YzA0MzViYWE0NWJkNTk0NGE2NWNmMGY3MjQxYzhjY2QzIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTYgQEAK
IDIwMTAtMTEtMTcgIEhheWF0byBJdG8gIDxoYXlhdG9AY2hyb21pdW0ub3JnPgogCisgICAgICAg
IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFJlbW92ZSBhIHVyaSBtZW1i
ZXIgZnJvbSBUZXN0SW5wdXQgY2xhc3MuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTQ5NjkxCisKKyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2xh
eW91dF90ZXN0cy9sYXlvdXRfcGFja2FnZS9kdW1wX3JlbmRlcl90cmVlX3RocmVhZC5weToKKyAg
ICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5fd2Via2l0X3Rlc3RzLnB5
OgorCisyMDEwLTExLTE3ICBIYXlhdG8gSXRvICA8aGF5YXRvQGNocm9taXVtLm9yZz4KKwogICAg
ICAgICBSZWZhY3RvciBUZXN0VHlwZUJhc2UuY29tcGFyZV9vdXRwdXQoKS4KIAogICAgICAgICBJ
bnRyb2R1Y2UgYSBUZXN0T3V0cHV0IGNsYXNzIGFuZCB1cGRhdGUgY29tcGFyZV9vdXRwdXQoKSBv
ZiBlYWNoIHRlc3QKZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5
b3V0X3Rlc3RzL2xheW91dF9wYWNrYWdlL2R1bXBfcmVuZGVyX3RyZWVfdGhyZWFkLnB5IGIvV2Vi
S2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvbGF5b3V0X3BhY2thZ2UvZHVt
cF9yZW5kZXJfdHJlZV90aHJlYWQucHkKaW5kZXggODE0Mjg2NzJiNjA1MTg3ZTIyMTExNzQ0YzA0
NDkwZDU1OTVlYTNlZi4uNzVlYjc1YTBiOGEzYTg4NWZhOTg4ZGJhNmZiNGY0MWY3NmVjZmRiYyAx
MDA2NDQKLS0tIGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvbGF5
b3V0X3BhY2thZ2UvZHVtcF9yZW5kZXJfdHJlZV90aHJlYWQucHkKKysrIGIvV2ViS2l0VG9vbHMv
U2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvbGF5b3V0X3BhY2thZ2UvZHVtcF9yZW5kZXJf
dHJlZV90aHJlYWQucHkKQEAgLTkwLDcgKzkwLDcgQEAgZGVmIF9wcm9jZXNzX291dHB1dChwb3J0
LCBvcHRpb25zLCB0ZXN0X2lucHV0LCB0ZXN0X3R5cGVzLCB0ZXN0X2FyZ3MsCiAgICAgICBwb3J0
OiBwb3J0LXNwZWNpZmljIGhvb2tzCiAgICAgICBvcHRpb25zOiBjb21tYW5kIGxpbmUgb3B0aW9u
cyBhcmd1bWVudCBmcm9tIG9wdHBhcnNlCiAgICAgICBwcm9jOiBhbiBhY3RpdmUgRHVtcFJlbmRl
clRyZWUgcHJvY2VzcwotICAgICAgdGVzdF9pbnB1dDogT2JqZWN0IGNvbnRhaW5pbmcgdGhlIHRl
c3QgZmlsZW5hbWUsIHVyaSBhbmQgdGltZW91dAorICAgICAgdGVzdF9pbnB1dDogT2JqZWN0IGNv
bnRhaW5pbmcgdGhlIHRlc3QgZmlsZW5hbWUgYW5kIHRpbWVvdXQKICAgICAgIHRlc3RfdHlwZXM6
IGxpc3Qgb2YgdGVzdCB0eXBlcyB0byBzdWJqZWN0IHRoZSBvdXRwdXQgdG8KICAgICAgIHRlc3Rf
YXJnczogYXJndW1lbnRzIHRvIGJlIHBhc3NlZCB0byBlYWNoIHRlc3QKICAgICAgIHRlc3Rfb3V0
cHV0OiBhIFRlc3RPdXRwdXQgb2JqZWN0IGNvbnRhaW5pbmcgdGhlIG91dHB1dCBvZiB0aGUgdGVz
dApAQCAtMTcyLDggKzE3Miw4IEBAIGRlZiBfcnVuX3NpbmdsZV90ZXN0KHBvcnQsIG9wdGlvbnMs
IHRlc3RfaW5wdXQsIHRlc3RfdHlwZXMsIHRlc3RfYXJncywgZHJpdmVyKToKICAgICAgICAgaW1h
Z2VfaGFzaF90b19kcml2ZXIgPSBwb3J0LmV4cGVjdGVkX2NoZWNrc3VtKHRlc3RfaW5wdXQuZmls
ZW5hbWUpCiAgICAgZWxzZToKICAgICAgICAgaW1hZ2VfaGFzaF90b19kcml2ZXIgPSBOb25lCi0g
ICAgdGVzdF9pbnB1dC51cmkgPSBwb3J0LmZpbGVuYW1lX3RvX3VyaSh0ZXN0X2lucHV0LmZpbGVu
YW1lKS5zdHJpcCgpCi0gICAgdGVzdF9vdXRwdXQgPSBkcml2ZXIucnVuX3Rlc3QodGVzdF9pbnB1
dC51cmksIHRlc3RfaW5wdXQudGltZW91dCwgaW1hZ2VfaGFzaF90b19kcml2ZXIpCisgICAgdXJp
ID0gcG9ydC5maWxlbmFtZV90b191cmkodGVzdF9pbnB1dC5maWxlbmFtZSkuc3RyaXAoKQorICAg
IHRlc3Rfb3V0cHV0ID0gZHJpdmVyLnJ1bl90ZXN0KHVyaSwgdGVzdF9pbnB1dC50aW1lb3V0LCBp
bWFnZV9oYXNoX3RvX2RyaXZlcikKICAgICByZXR1cm4gX3Byb2Nlc3Nfb3V0cHV0KHBvcnQsIG9w
dGlvbnMsIHRlc3RfaW5wdXQsIHRlc3RfdHlwZXMsIHRlc3RfYXJncywKICAgICAgICAgICAgICAg
ICAgICAgICAgICAgIHRlc3Rfb3V0cHV0KQogCkBAIC0xODYsNyArMTg2LDcgQEAgY2xhc3MgU2lu
Z2xlVGVzdFRocmVhZCh0aHJlYWRpbmcuVGhyZWFkKToKICAgICAgICAgQXJnczoKICAgICAgICAg
ICBwb3J0OiBvYmplY3QgaW1wbGVtZW50aW5nIHBvcnQtc3BlY2lmaWMgaG9va3MKICAgICAgICAg
ICBvcHRpb25zOiBjb21tYW5kIGxpbmUgYXJndW1lbnQgb2JqZWN0IGZyb20gb3B0cGFyc2UKLSAg
ICAgICAgICB0ZXN0X2lucHV0OiBPYmplY3QgY29udGFpbmluZyB0aGUgdGVzdCBmaWxlbmFtZSwg
dXJpIGFuZCB0aW1lb3V0CisgICAgICAgICAgdGVzdF9pbnB1dDogT2JqZWN0IGNvbnRhaW5pbmcg
dGhlIHRlc3QgZmlsZW5hbWUgYW5kIHRpbWVvdXQKICAgICAgICAgICB0ZXN0X3R5cGVzOiBBIGxp
c3Qgb2YgVGVzdFR5cGUgb2JqZWN0cyB0byBydW4gdGhlIHRlc3Qgb3V0cHV0CiAgICAgICAgICAg
ICAgIGFnYWluc3QuCiAgICAgICAgICAgdGVzdF9hcmdzOiBBIFRlc3RBcmd1bWVudHMgb2JqZWN0
IHRvIHBhc3MgdG8gZWFjaCBUZXN0VHlwZS4KQEAgLTQ1Nyw3ICs0NTcsNyBAQCBjbGFzcyBUZXN0
U2hlbGxUaHJlYWQoV2F0Y2hhYmxlVGhyZWFkKToKICAgICAgICAgZmlsZXMgc2luZ2x5LgogCiAg
ICAgICAgIEFyZ3M6Ci0gICAgICAgICAgdGVzdF9pbnB1dDogT2JqZWN0IGNvbnRhaW5pbmcgdGhl
IHRlc3QgZmlsZW5hbWUsIHVyaSBhbmQgdGltZW91dAorICAgICAgICAgIHRlc3RfaW5wdXQ6IE9i
amVjdCBjb250YWluaW5nIHRoZSB0ZXN0IGZpbGVuYW1lIGFuZCB0aW1lb3V0CiAKICAgICAgICAg
UmV0dXJuczoKICAgICAgICAgICBBIFRlc3RSZXN1bHQKZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xz
L1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3J1bl93ZWJraXRfdGVzdHMucHkgYi9XZWJL
aXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5fd2Via2l0X3Rlc3RzLnB5
CmluZGV4IGIzN2ZhMWViZjJiYjMyMTFkNTIyOTRiNWQ4ODE1ZWZiOTU0ZTM2ZWYuLjExOWRlOGNm
ZWZlOTk4ZTEzMDVhOGMxYzc5NjhhNzE2M2NiYjU1NDIgMTAwNzU1Ci0tLSBhL1dlYktpdFRvb2xz
L1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3J1bl93ZWJraXRfdGVzdHMucHkKKysrIGIv
V2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcnVuX3dlYmtpdF90ZXN0
cy5weQpAQCAtMTAzLDkgKzEwMyw2IEBAIGNsYXNzIFRlc3RJbnB1dDoKICAgICAgICAgc2VsZi5m
aWxlbmFtZSA9IGZpbGVuYW1lCiAgICAgICAgIHNlbGYudGltZW91dCA9IHRpbWVvdXQKIAotICAg
ICAgICAjIEZJWE1FOiBNYXliZSB0aGUgVVJJIHNob3VsZG4ndCBiZSBwYXJ0IG9mIHRoZSBUZXN0
SW5wdXQgYXQgYWxsPwotICAgICAgICBzZWxmLnVyaSA9IE5vbmUKLQogCiBjbGFzcyBSZXN1bHRT
dW1tYXJ5KG9iamVjdCk6CiAgICAgIiIiQSBjbGFzcyBmb3IgcGFydGl0aW9uaW5nIHRoZSB0ZXN0
IHJlc3VsdHMgd2UgZ2V0IGludG8gYnVja2V0cy4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>74183</attachid>
            <date>2010-11-17 17:35:27 -0800</date>
            <delta_ts>2010-11-17 18:02:49 -0800</delta_ts>
            <desc>remove-strip</desc>
            <filename>bug-49691-20101118103525.patch</filename>
            <type>text/plain</type>
            <size>3898</size>
            <attacher name="Hayato Ito">hayato</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCBjZGFjMWRhMzFiNzVmZTQyZTgzOWM3OWNiM2ZmOWE2YjAxODI3NDViLi4xYmExMTMw
YzA0MzViYWE0NWJkNTk0NGE2NWNmMGY3MjQxYzhjY2QzIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTYgQEAK
IDIwMTAtMTEtMTcgIEhheWF0byBJdG8gIDxoYXlhdG9AY2hyb21pdW0ub3JnPgogCisgICAgICAg
IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFJlbW92ZSBhIHVyaSBtZW1i
ZXIgZnJvbSBUZXN0SW5wdXQgY2xhc3MuCisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTQ5NjkxCisKKyAgICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2xh
eW91dF90ZXN0cy9sYXlvdXRfcGFja2FnZS9kdW1wX3JlbmRlcl90cmVlX3RocmVhZC5weToKKyAg
ICAgICAgKiBTY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5fd2Via2l0X3Rlc3RzLnB5
OgorCisyMDEwLTExLTE3ICBIYXlhdG8gSXRvICA8aGF5YXRvQGNocm9taXVtLm9yZz4KKwogICAg
ICAgICBSZWZhY3RvciBUZXN0VHlwZUJhc2UuY29tcGFyZV9vdXRwdXQoKS4KIAogICAgICAgICBJ
bnRyb2R1Y2UgYSBUZXN0T3V0cHV0IGNsYXNzIGFuZCB1cGRhdGUgY29tcGFyZV9vdXRwdXQoKSBv
ZiBlYWNoIHRlc3QKZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5
b3V0X3Rlc3RzL2xheW91dF9wYWNrYWdlL2R1bXBfcmVuZGVyX3RyZWVfdGhyZWFkLnB5IGIvV2Vi
S2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvbGF5b3V0X3BhY2thZ2UvZHVt
cF9yZW5kZXJfdHJlZV90aHJlYWQucHkKaW5kZXggODE0Mjg2NzJiNjA1MTg3ZTIyMTExNzQ0YzA0
NDkwZDU1OTVlYTNlZi4uODhmNDkzZDBmOThhZGIzNDExNTM5ZGY5MjA5MDQyZTQ0YjU1MzkzYyAx
MDA2NDQKLS0tIGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvbGF5
b3V0X3BhY2thZ2UvZHVtcF9yZW5kZXJfdHJlZV90aHJlYWQucHkKKysrIGIvV2ViS2l0VG9vbHMv
U2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvbGF5b3V0X3BhY2thZ2UvZHVtcF9yZW5kZXJf
dHJlZV90aHJlYWQucHkKQEAgLTkwLDcgKzkwLDcgQEAgZGVmIF9wcm9jZXNzX291dHB1dChwb3J0
LCBvcHRpb25zLCB0ZXN0X2lucHV0LCB0ZXN0X3R5cGVzLCB0ZXN0X2FyZ3MsCiAgICAgICBwb3J0
OiBwb3J0LXNwZWNpZmljIGhvb2tzCiAgICAgICBvcHRpb25zOiBjb21tYW5kIGxpbmUgb3B0aW9u
cyBhcmd1bWVudCBmcm9tIG9wdHBhcnNlCiAgICAgICBwcm9jOiBhbiBhY3RpdmUgRHVtcFJlbmRl
clRyZWUgcHJvY2VzcwotICAgICAgdGVzdF9pbnB1dDogT2JqZWN0IGNvbnRhaW5pbmcgdGhlIHRl
c3QgZmlsZW5hbWUsIHVyaSBhbmQgdGltZW91dAorICAgICAgdGVzdF9pbnB1dDogT2JqZWN0IGNv
bnRhaW5pbmcgdGhlIHRlc3QgZmlsZW5hbWUgYW5kIHRpbWVvdXQKICAgICAgIHRlc3RfdHlwZXM6
IGxpc3Qgb2YgdGVzdCB0eXBlcyB0byBzdWJqZWN0IHRoZSBvdXRwdXQgdG8KICAgICAgIHRlc3Rf
YXJnczogYXJndW1lbnRzIHRvIGJlIHBhc3NlZCB0byBlYWNoIHRlc3QKICAgICAgIHRlc3Rfb3V0
cHV0OiBhIFRlc3RPdXRwdXQgb2JqZWN0IGNvbnRhaW5pbmcgdGhlIG91dHB1dCBvZiB0aGUgdGVz
dApAQCAtMTcyLDggKzE3Miw4IEBAIGRlZiBfcnVuX3NpbmdsZV90ZXN0KHBvcnQsIG9wdGlvbnMs
IHRlc3RfaW5wdXQsIHRlc3RfdHlwZXMsIHRlc3RfYXJncywgZHJpdmVyKToKICAgICAgICAgaW1h
Z2VfaGFzaF90b19kcml2ZXIgPSBwb3J0LmV4cGVjdGVkX2NoZWNrc3VtKHRlc3RfaW5wdXQuZmls
ZW5hbWUpCiAgICAgZWxzZToKICAgICAgICAgaW1hZ2VfaGFzaF90b19kcml2ZXIgPSBOb25lCi0g
ICAgdGVzdF9pbnB1dC51cmkgPSBwb3J0LmZpbGVuYW1lX3RvX3VyaSh0ZXN0X2lucHV0LmZpbGVu
YW1lKS5zdHJpcCgpCi0gICAgdGVzdF9vdXRwdXQgPSBkcml2ZXIucnVuX3Rlc3QodGVzdF9pbnB1
dC51cmksIHRlc3RfaW5wdXQudGltZW91dCwgaW1hZ2VfaGFzaF90b19kcml2ZXIpCisgICAgdXJp
ID0gcG9ydC5maWxlbmFtZV90b191cmkodGVzdF9pbnB1dC5maWxlbmFtZSkKKyAgICB0ZXN0X291
dHB1dCA9IGRyaXZlci5ydW5fdGVzdCh1cmksIHRlc3RfaW5wdXQudGltZW91dCwgaW1hZ2VfaGFz
aF90b19kcml2ZXIpCiAgICAgcmV0dXJuIF9wcm9jZXNzX291dHB1dChwb3J0LCBvcHRpb25zLCB0
ZXN0X2lucHV0LCB0ZXN0X3R5cGVzLCB0ZXN0X2FyZ3MsCiAgICAgICAgICAgICAgICAgICAgICAg
ICAgICB0ZXN0X291dHB1dCkKIApAQCAtMTg2LDcgKzE4Niw3IEBAIGNsYXNzIFNpbmdsZVRlc3RU
aHJlYWQodGhyZWFkaW5nLlRocmVhZCk6CiAgICAgICAgIEFyZ3M6CiAgICAgICAgICAgcG9ydDog
b2JqZWN0IGltcGxlbWVudGluZyBwb3J0LXNwZWNpZmljIGhvb2tzCiAgICAgICAgICAgb3B0aW9u
czogY29tbWFuZCBsaW5lIGFyZ3VtZW50IG9iamVjdCBmcm9tIG9wdHBhcnNlCi0gICAgICAgICAg
dGVzdF9pbnB1dDogT2JqZWN0IGNvbnRhaW5pbmcgdGhlIHRlc3QgZmlsZW5hbWUsIHVyaSBhbmQg
dGltZW91dAorICAgICAgICAgIHRlc3RfaW5wdXQ6IE9iamVjdCBjb250YWluaW5nIHRoZSB0ZXN0
IGZpbGVuYW1lIGFuZCB0aW1lb3V0CiAgICAgICAgICAgdGVzdF90eXBlczogQSBsaXN0IG9mIFRl
c3RUeXBlIG9iamVjdHMgdG8gcnVuIHRoZSB0ZXN0IG91dHB1dAogICAgICAgICAgICAgICBhZ2Fp
bnN0LgogICAgICAgICAgIHRlc3RfYXJnczogQSBUZXN0QXJndW1lbnRzIG9iamVjdCB0byBwYXNz
IHRvIGVhY2ggVGVzdFR5cGUuCkBAIC00NTcsNyArNDU3LDcgQEAgY2xhc3MgVGVzdFNoZWxsVGhy
ZWFkKFdhdGNoYWJsZVRocmVhZCk6CiAgICAgICAgIGZpbGVzIHNpbmdseS4KIAogICAgICAgICBB
cmdzOgotICAgICAgICAgIHRlc3RfaW5wdXQ6IE9iamVjdCBjb250YWluaW5nIHRoZSB0ZXN0IGZp
bGVuYW1lLCB1cmkgYW5kIHRpbWVvdXQKKyAgICAgICAgICB0ZXN0X2lucHV0OiBPYmplY3QgY29u
dGFpbmluZyB0aGUgdGVzdCBmaWxlbmFtZSBhbmQgdGltZW91dAogCiAgICAgICAgIFJldHVybnM6
CiAgICAgICAgICAgQSBUZXN0UmVzdWx0CmRpZmYgLS1naXQgYS9XZWJLaXRUb29scy9TY3JpcHRz
L3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5fd2Via2l0X3Rlc3RzLnB5IGIvV2ViS2l0VG9vbHMv
U2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcnVuX3dlYmtpdF90ZXN0cy5weQppbmRleCBi
MzdmYTFlYmYyYmIzMjExZDUyMjk0YjVkODgxNWVmYjk1NGUzNmVmLi4xMTlkZThjZmVmZTk5OGUx
MzA1YThjMWM3OTY4YTcxNjNjYmI1NTQyIDEwMDc1NQotLS0gYS9XZWJLaXRUb29scy9TY3JpcHRz
L3dlYmtpdHB5L2xheW91dF90ZXN0cy9ydW5fd2Via2l0X3Rlc3RzLnB5CisrKyBiL1dlYktpdFRv
b2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3J1bl93ZWJraXRfdGVzdHMucHkKQEAg
LTEwMyw5ICsxMDMsNiBAQCBjbGFzcyBUZXN0SW5wdXQ6CiAgICAgICAgIHNlbGYuZmlsZW5hbWUg
PSBmaWxlbmFtZQogICAgICAgICBzZWxmLnRpbWVvdXQgPSB0aW1lb3V0CiAKLSAgICAgICAgIyBG
SVhNRTogTWF5YmUgdGhlIFVSSSBzaG91bGRuJ3QgYmUgcGFydCBvZiB0aGUgVGVzdElucHV0IGF0
IGFsbD8KLSAgICAgICAgc2VsZi51cmkgPSBOb25lCi0KIAogY2xhc3MgUmVzdWx0U3VtbWFyeShv
YmplY3QpOgogICAgICIiIkEgY2xhc3MgZm9yIHBhcnRpdGlvbmluZyB0aGUgdGVzdCByZXN1bHRz
IHdlIGdldCBpbnRvIGJ1Y2tldHMuCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>