<?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>45791</bug_id>
          
          <creation_ts>2010-09-14 16:23:44 -0700</creation_ts>
          <short_desc>NRWT fails with UnicodeDecodeError on editing/selection/mixed-editability-10.html</short_desc>
          <delta_ts>2010-09-15 10:47:48 -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>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="Mihai Parparita">mihaip</reporter>
          <assigned_to name="Dirk Pranke">dpranke</assigned_to>
          <cc>abarth</cc>
    
    <cc>commit-queue</cc>
    
    <cc>dpranke</cc>
    
    <cc>eric</cc>
    
    <cc>ojan</cc>
    
    <cc>tony</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>279246</commentid>
    <comment_count>0</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 16:23:44 -0700</bug_when>
    <thetext>Stack trace:
  File &quot;./WebKitTools/Scripts/new-run-webkit-tests&quot;, line 38, in &lt;module&gt;
    sys.exit(run_webkit_tests.main())
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/run_webkit_tests.py&quot;, line 1722, in main
    return run(port_obj, options, args)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/run_webkit_tests.py&quot;, line 1425, in run
    num_unexpected_results = test_runner.run(result_summary)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/run_webkit_tests.py&quot;, line 780, in run
    self._run_tests(self._test_files_list, result_summary))
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/run_webkit_tests.py&quot;, line 651, in _run_tests
    result_summary)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/run_webkit_tests.py&quot;, line 619, in _instantiate_dump_render_tree_threads
    thread.run_in_main_thread(self, result_summary)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py&quot;, line 344, in run_in_main_thread
    self._run(test_runner, result_summary)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py&quot;, line 392, in _run
    result = self._run_test(test_info)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py&quot;, line 510, in _run_test
    output, error)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/layout_package/dump_render_tree_thread.py&quot;, line 125, in _process_output
    configuration)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/test_types/text_diff.py&quot;, line 104, in compare_output
    print_text_diffs=True)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/test_types/test_type_base.py&quot;, line 204, in write_output_files
    diff = port.diff_text(expected, output, expected_filename, actual_filename)
  File &quot;/Users/mihaip/Developer/source/chromium1/src/third_party/WebKit/WebKitTools/Scripts/webkitpy/layout_tests/port/base.py&quot;, line 159, in diff_text
    return &apos;&apos;.join(diff)
UnicodeDecodeError: &apos;ascii&apos; codec can&apos;t decode byte 0xc2 in position 7: ordinal not in range(128)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279276</commentid>
    <comment_count>1</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-09-14 17:41:22 -0700</bug_when>
    <thetext>The problem seems to be that we&apos;re passing in unicode strings for the filenames and raw strings for the content to diff. That results in the unified output being a mixture of the two datatypes, which causes &apos;&apos;.join() to do encoding conversions using the default &apos;ascii&apos; converter, which keels over.

The easiest fix is probably to convert the filenames to raw strings.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279282</commentid>
    <comment_count>2</comment_count>
      <attachid>67626</attachid>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 17:47:48 -0700</bug_when>
    <thetext>Created attachment 67626
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279284</commentid>
    <comment_count>3</comment_count>
      <attachid>67626</attachid>
    <who name="Tony Chang">tony</who>
    <bug_when>2010-09-14 17:52:13 -0700</bug_when>
    <thetext>Comment on attachment 67626
Patch

Should test_expectations.txt be updated as well or does the test still fail?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279287</commentid>
    <comment_count>4</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 17:55:29 -0700</bug_when>
    <thetext>Test still fails, NRWT was just barfing when outputting the diff after the test had run.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279290</commentid>
    <comment_count>5</comment_count>
    <who name="Ojan Vafai">ojan</who>
    <bug_when>2010-09-14 17:58:37 -0700</bug_when>
    <thetext>I would slightly prefer something like the following. I feel like it&apos;s more explicit. But this is fine too.

if isinstance(value, unicode):
    value = value.encode(&apos;utf-8&apos;)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279292</commentid>
    <comment_count>6</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-09-14 18:05:59 -0700</bug_when>
    <thetext>I&apos;m fine w/ the patch as is. I&apos;m not sure that Ojan&apos;s mod is worth the extra verbosity.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279296</commentid>
    <comment_count>7</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2010-09-14 18:12:31 -0700</bug_when>
    <thetext>(In reply to comment #5)
&gt; I would slightly prefer something like the following. I feel like it&apos;s more explicit. But this is fine too.
&gt; 
&gt; if isinstance(value, unicode):
&gt;     value = value.encode(&apos;utf-8&apos;)

|value| will always be a unicode string, won&apos;t it?  Explicitly calling encode seems a bit better.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279297</commentid>
    <comment_count>8</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-09-14 18:16:46 -0700</bug_when>
    <thetext>(In reply to comment #7)
&gt; (In reply to comment #5)
&gt; &gt; I would slightly prefer something like the following. I feel like it&apos;s more explicit. But this is fine too.
&gt; &gt; 
&gt; &gt; if isinstance(value, unicode):
&gt; &gt;     value = value.encode(&apos;utf-8&apos;)
&gt; 
&gt; |value| will always be a unicode string, won&apos;t it?  Explicitly calling encode seems a bit better.

I assumed Ojan&apos;s comment was pseudocode out of context. There isn&apos;t actually a &apos;value&apos; variable anywhere in the code AFAIK.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279326</commentid>
    <comment_count>9</comment_count>
      <attachid>67626</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-14 19:05:46 -0700</bug_when>
    <thetext>Comment on attachment 67626
Patch

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

&gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base.py:160
&gt; +        expected_filename = str(expected_filename)
&gt; +        actual_filename = str(actual_filename)
This will take expected_filename and try to decode it as ascii.  Is that what you intended?  If expected_filename is already a str() object it has no effect, but if it&apos;s unicode() that&apos;s effectively calling filename.encode(&apos;ascii&apos;) which may not be what you want?  (At least that&apos;s my understanding.)

&gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base_unittest.py:175
&gt; +        port.diff_text(&apos;exp&apos;, &apos;act&apos;, &apos;exp.txt&apos;, &apos;act.txt&apos;)
&gt; +        port.diff_text(&apos;exp&apos;, &apos;act&apos;, u&apos;exp.txt&apos;, &apos;act.txt&apos;)
&gt; +
&gt; +        port.diff_text(&apos;exp&apos; + chr(255), &apos;act&apos;, &apos;exp.txt&apos;, &apos;act.txt&apos;)
&gt; +        port.diff_text(&apos;exp&apos; + chr(255), &apos;act&apos;, u&apos;exp.txt&apos;, &apos;act.txt&apos;)
Seems we need a test with unicode characters here.

I don&apos;t think this is 100% right, but please correct me if I&apos;m wrong. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279330</commentid>
    <comment_count>10</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-09-14 19:24:20 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; (From update of attachment 67626 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=67626&amp;action=prettypatch
&gt; 
&gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base.py:160
&gt; &gt; +        expected_filename = str(expected_filename)
&gt; &gt; +        actual_filename = str(actual_filename)
&gt; This will take expected_filename and try to decode it as ascii.  Is that what you intended?  If expected_filename is already a str() object it has no effect, but if it&apos;s unicode() that&apos;s effectively calling filename.encode(&apos;ascii&apos;) which may not be what you want?  (At least that&apos;s my understanding.)
&gt; 
&gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base_unittest.py:175
&gt; &gt; +        port.diff_text(&apos;exp&apos;, &apos;act&apos;, &apos;exp.txt&apos;, &apos;act.txt&apos;)
&gt; &gt; +        port.diff_text(&apos;exp&apos;, &apos;act&apos;, u&apos;exp.txt&apos;, &apos;act.txt&apos;)
&gt; &gt; +
&gt; &gt; +        port.diff_text(&apos;exp&apos; + chr(255), &apos;act&apos;, &apos;exp.txt&apos;, &apos;act.txt&apos;)
&gt; &gt; +        port.diff_text(&apos;exp&apos; + chr(255), &apos;act&apos;, u&apos;exp.txt&apos;, &apos;act.txt&apos;)
&gt; Seems we need a test with unicode characters here.
&gt; 
&gt; I don&apos;t think this is 100% right, but please correct me if I&apos;m wrong. :)

The second line gives us the test with unicode in it. Really, we only needed those two. The third and fourth test to make sure that we&apos;re *not* doing an ascii conversion, because those would choke on a non-ascii character (the original test contained non-ascii characters in the expected and actual strings).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279331</commentid>
    <comment_count>11</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-09-14 19:25:40 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; (From update of attachment 67626 [details])
&gt; View in context: https://bugs.webkit.org/attachment.cgi?id=67626&amp;action=prettypatch
&gt; 
&gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base.py:160
&gt; &gt; +        expected_filename = str(expected_filename)
&gt; &gt; +        actual_filename = str(actual_filename)
&gt; This will take expected_filename and try to decode it as ascii.  Is that what you intended?  If expected_filename is already a str() object it has no effect, but if it&apos;s unicode() that&apos;s effectively calling filename.encode(&apos;ascii&apos;) which may not be what you want?  (At least that&apos;s my understanding.)
&gt; 

Yup, that&apos;s what we wanted, except that we actually want &quot;turn this into a raw string so that I can safely join it to other raw strings&quot;. We don&apos;t want to force every line to ascii, or we&apos;d still run into the problem we have now (expected strings containing non-ascii characters).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279333</commentid>
    <comment_count>12</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 19:27:32 -0700</bug_when>
    <thetext>(In reply to comment #11)
&gt; (In reply to comment #9)
&gt; &gt; (From update of attachment 67626 [details] [details])
&gt; &gt; View in context: https://bugs.webkit.org/attachment.cgi?id=67626&amp;action=prettypatch
&gt; &gt; 
&gt; &gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base.py:160
&gt; &gt; &gt; +        expected_filename = str(expected_filename)
&gt; &gt; &gt; +        actual_filename = str(actual_filename)
&gt; &gt; This will take expected_filename and try to decode it as ascii.  Is that what you intended?  If expected_filename is already a str() object it has no effect, but if it&apos;s unicode() that&apos;s effectively calling filename.encode(&apos;ascii&apos;) which may not be what you want?  (At least that&apos;s my understanding.)
&gt; &gt; 
&gt; 
&gt; Yup, that&apos;s what we wanted, except that we actually want &quot;turn this into a raw string so that I can safely join it to other raw strings&quot;. We don&apos;t want to force every line to ascii, or we&apos;d still run into the problem we have now (expected strings containing non-ascii characters).

However, Eric is right that str() on a unicode string with non-ASCII characters will thrown an exception. Using filename.encode(&apos;utf-8&apos;) (as Ojan suggested) is safer. Will upload a new patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279338</commentid>
    <comment_count>13</comment_count>
    <who name="Dirk Pranke">dpranke</who>
    <bug_when>2010-09-14 19:35:24 -0700</bug_when>
    <thetext>(In reply to comment #12)
&gt; (In reply to comment #11)
&gt; &gt; (In reply to comment #9)
&gt; &gt; &gt; (From update of attachment 67626 [details] [details] [details])
&gt; &gt; &gt; View in context: https://bugs.webkit.org/attachment.cgi?id=67626&amp;action=prettypatch
&gt; &gt; &gt; 
&gt; &gt; &gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base.py:160
&gt; &gt; &gt; &gt; +        expected_filename = str(expected_filename)
&gt; &gt; &gt; &gt; +        actual_filename = str(actual_filename)
&gt; &gt; &gt; This will take expected_filename and try to decode it as ascii.  Is that what you intended?  If expected_filename is already a str() object it has no effect, but if it&apos;s unicode() that&apos;s effectively calling filename.encode(&apos;ascii&apos;) which may not be what you want?  (At least that&apos;s my understanding.)
&gt; &gt; &gt; 
&gt; &gt; 
&gt; &gt; Yup, that&apos;s what we wanted, except that we actually want &quot;turn this into a raw string so that I can safely join it to other raw strings&quot;. We don&apos;t want to force every line to ascii, or we&apos;d still run into the problem we have now (expected strings containing non-ascii characters).
&gt; 
&gt; However, Eric is right that str() on a unicode string with non-ASCII characters will thrown an exception. Using filename.encode(&apos;utf-8&apos;) (as Ojan suggested) is safer. Will upload a new patch.

I&apos;m not sure that we actually want to allow non-ASCII filenames, so I&apos;m not sure that this is a bad thing. Lord knows what else would break if we did allow them and one gets checked in.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279341</commentid>
    <comment_count>14</comment_count>
      <attachid>67635</attachid>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 19:43:51 -0700</bug_when>
    <thetext>Created attachment 67635
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279348</commentid>
    <comment_count>15</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-14 20:16:40 -0700</bug_when>
    <thetext>(In reply to comment #13)
&gt; I&apos;m not sure that we actually want to allow non-ASCII filenames, so I&apos;m not sure that this is a bad thing. Lord knows what else would break if we did allow them and one gets checked in.

We have in the past had non-ascii file names.  I believe they had to be removed when the windows port came online.  I remember adding some since, but then again having to remove them since they broke on windows.  But the general goal of supporting unicode urls and filenames is a good one.

We should assume unicode whenever we&apos;re dealing with &quot;strings&quot;.  The str() type in Python was renamed ByteArray() in Python 3. and unicode() became the default for &quot;foo&quot;.  The u&quot;foo&quot; syntax no longer exists in Python 3.

This is part of how we always want to use codecs.open for dealing with files and always pass an encoding.  codecs.open became open() in Python 3, again as part of moving to real unicode support throughout the interpreter.

(I&apos;ve not used Python 3, I&apos;ve just read the unicode powerpoints/docs online as part of the grand &quot;make webkitpy understand unicode, dammit!&quot; project I undertook 9 months ago to make Tor Arne Vestbø not break the commit-queue every time he was involved with a patch. :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279351</commentid>
    <comment_count>16</comment_count>
      <attachid>67635</attachid>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 20:23:35 -0700</bug_when>
    <thetext>Comment on attachment 67635
Patch

Missed a return statement in to_raw_bytes</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279355</commentid>
    <comment_count>17</comment_count>
      <attachid>67639</attachid>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 20:27:29 -0700</bug_when>
    <thetext>Created attachment 67639
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279364</commentid>
    <comment_count>18</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2010-09-14 20:58:56 -0700</bug_when>
    <thetext>(In reply to comment #15)
&gt; (In reply to comment #13)
&gt; &gt; I&apos;m not sure that we actually want to allow non-ASCII filenames, so I&apos;m not sure that this is a bad thing. Lord knows what else would break if we did allow them and one gets checked in.
&gt; 
&gt; We have in the past had non-ascii file names.  I believe they had to be removed when the windows port came online.  I remember adding some since, but then again having to remove them since they broke on windows.  But the general goal of supporting unicode urls and filenames is a good one.

Where do these filenames come from?  On Linux, filenames are just a stream of bytes so you can&apos;t always assume unicode and calling encode(&apos;utf8&apos;) is going to produce &quot;\xef\xbf\xbd&quot;s.

Ultimately, I don&apos;t think it matters what we do here because for lots of reasons, we&apos;re only going to have ascii filenames.  If you want to try to future proof the code, I&apos;m not opposed, but I don&apos;t think it actually matters.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279375</commentid>
    <comment_count>19</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-14 21:44:44 -0700</bug_when>
    <thetext>(In reply to comment #18)
&gt; Where do these filenames come from?  On Linux, filenames are just a stream of bytes so you can&apos;t always assume unicode and calling encode(&apos;utf8&apos;) is going to produce &quot;\xef\xbf\xbd&quot;s.
&gt; 
&gt; Ultimately, I don&apos;t think it matters what we do here because for lots of reasons, we&apos;re only going to have ascii filenames.  If you want to try to future proof the code, I&apos;m not opposed, but I don&apos;t think it actually matters.

I just wanted a string with not-valid-in-ASCII characters in them, I got it from http://docs.python.org/howto/unicode.html#unicode-literals-in-python-source-code. If the filenames are not unicode (and just a stream of bytes, i.e. str), then this will degrade fine.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279390</commentid>
    <comment_count>20</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-14 22:36:33 -0700</bug_when>
    <thetext>(In reply to comment #18)
&gt; Where do these filenames come from?  On Linux, filenames are just a stream of bytes so you can&apos;t always assume unicode and calling encode(&apos;utf8&apos;) is going to produce &quot;\xef\xbf\xbd&quot;s.

Mac HFS+ is always utf8 for all paths.  But I realize that is not true for all platforms.

We&apos;ve had the non-ascii file names at various times because we need to be able to test handling of non-ascii URLs both locally and remotely for WebKit.  There are also handling unicode-aware parsing algorithms for element attributes, etc.  In general it&apos;s not a huge issue, but we ended up removing these tests due to Windows troubles iirc.

&gt; Ultimately, I don&apos;t think it matters what we do here because for lots of reasons, we&apos;re only going to have ascii filenames.  If you want to try to future proof the code, I&apos;m not opposed, but I don&apos;t think it actually matters.

Agreed.  Still is nice to attempt to be sane about unicode.

I think the confusion comes here because I read the patch too quickly and didn&apos;t realize Mihai&apos;s char(255) were how he was getting unicode code points.  I&apos;m used to using u&quot;foo\u1235bar&quot; to get unicode codepoints in python.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279391</commentid>
    <comment_count>21</comment_count>
      <attachid>67639</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2010-09-14 22:39:29 -0700</bug_when>
    <thetext>Comment on attachment 67639
Patch

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

I really need to write some documentation for dealing with unicode vs. byte arrays in webkitpy.

&gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base.py:162
&gt; +        def to_raw_bytes(str):
&gt; +            if isinstance(str, unicode):
&gt; +                return str.encode(&apos;utf-8&apos;)
&gt; +            return str
I suspect other areas of the code I&apos;ve assumed that file names were unicode strings.  If that&apos;s not a valid assumption across platforms we&apos;re going to have more of these bugs.  We may need to make a module-wide fix to this.

&gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base_unittest.py:170
&gt; +        # actual input is always a raw string, not unicode).
I would have said &quot;raw bytes&quot; instead of &quot;raw string&quot;, but that&apos;s a total nit.

&gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base_unittest.py:180
&gt; +        # be safe
safe.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279566</commentid>
    <comment_count>22</comment_count>
      <attachid>67681</attachid>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-15 08:58:00 -0700</bug_when>
    <thetext>Created attachment 67681
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279567</commentid>
    <comment_count>23</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-15 08:59:08 -0700</bug_when>
    <thetext>(In reply to comment #21)
&gt; I suspect other areas of the code I&apos;ve assumed that file names were unicode strings.  If that&apos;s not a valid assumption across platforms we&apos;re going to have more of these bugs.  We may need to make a module-wide fix to this.

It&apos;s a valid assumption, until we try to combine filenames with their content. Diff output is most likely the only place where we try to do that.
 
&gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base_unittest.py:170
&gt; &gt; +        # actual input is always a raw string, not unicode).
&gt; I would have said &quot;raw bytes&quot; instead of &quot;raw string&quot;, but that&apos;s a total nit.

Fixed (in the ChangeLog too)
 
&gt; &gt; WebKitTools/Scripts/webkitpy/layout_tests/port/base_unittest.py:180
&gt; &gt; +        # be safe
&gt; safe.

Fixed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279589</commentid>
    <comment_count>24</comment_count>
      <attachid>67681</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-09-15 10:06:22 -0700</bug_when>
    <thetext>Comment on attachment 67681
Patch

Clearing flags on attachment: 67681

Committed r67560: &lt;http://trac.webkit.org/changeset/67560&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279590</commentid>
    <comment_count>25</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-09-15 10:06:28 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279614</commentid>
    <comment_count>26</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-09-15 10:41:34 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/67560 might have broken GTK Linux 32-bit Release
The following changes are on the blame list:
http://trac.webkit.org/changeset/67560
http://trac.webkit.org/changeset/67561</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>279618</commentid>
    <comment_count>27</comment_count>
    <who name="Mihai Parparita">mihaip</who>
    <bug_when>2010-09-15 10:47:48 -0700</bug_when>
    <thetext>(In reply to comment #26)
&gt; http://trac.webkit.org/changeset/67560 might have broken GTK Linux 32-bit Release
&gt; The following changes are on the blame list:
&gt; http://trac.webkit.org/changeset/67560
&gt; http://trac.webkit.org/changeset/67561

Since accessibility/ellipsis-text.html failed, I assume it&apos;s r67561&apos;s fault.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>67626</attachid>
            <date>2010-09-14 17:47:48 -0700</date>
            <delta_ts>2010-09-14 19:43:47 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-45791-20100914174747.patch</filename>
            <type>text/plain</type>
            <size>2970</size>
            <attacher name="Mihai Parparita">mihaip</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCA1YWFkZWM5MGE4NDk5NWYwYWQ4ODFlODRjZmIxNGM5ODEwMTMxZGE3Li4yMDg4NjU1
MDA5OTQyZWRjNzQwZWU1MzRmY2EzNjEwNjVhMGM5N2EzIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAK
KzIwMTAtMDktMTQgIE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTlJXVCBmYWlscyB3
aXRoIFVuaWNvZGVEZWNvZGVFcnJvciBvbiBlZGl0aW5nL3NlbGVjdGlvbi9taXhlZC1lZGl0YWJp
bGl0eS0xMC5odG1sCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD00NTc5MQorCisgICAgICAgIEZvcmNlIGZpbGVuYW1lcyB0byBiZSByYXcgc3RyaW5ncyBi
ZWZvcmUgcnVubmluZyBkaWZmbGliLnVuaWZpZWRfZGlmZi4KKworICAgICAgICAqIFNjcmlwdHMv
d2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZS5weToKKyAgICAgICAgKiBTY3JpcHRzL3dl
YmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jhc2VfdW5pdHRlc3QucHk6CisKIDIwMTAtMDktMTQg
IFNhbSBXZWluaWcgIDxzYW1Ad2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBBbmRl
cnMgQ2FybHNzb24uCmRpZmYgLS1naXQgYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xh
eW91dF90ZXN0cy9wb3J0L2Jhc2UucHkgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xh
eW91dF90ZXN0cy9wb3J0L2Jhc2UucHkKaW5kZXggOTEyNWY5ZWQyM2Y5MDUyOWYyNWI4NjIzMDYx
OGRlYWZjZWQ3Mjk2NS4uYzY3MWZkMjFiNTU5M2ViOTI5MzNjODlhMjlkNjg1ODI3YjMxYzU2MSAx
MDA2NDQKLS0tIGEvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcG9y
dC9iYXNlLnB5CisrKyBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3Rz
L3BvcnQvYmFzZS5weQpAQCAtMTUyLDYgKzE1MiwxMiBAQCBjbGFzcyBQb3J0KG9iamVjdCk6CiAK
ICAgICAgICAgV2hpbGUgdGhpcyBpcyBhIGdlbmVyaWMgcm91dGluZSwgd2UgaW5jbHVkZSBpdCBp
biB0aGUgUG9ydAogICAgICAgICBpbnRlcmZhY2Ugc28gdGhhdCBpdCBjYW4gYmUgb3ZlcnJpZGVu
IGZvciB0ZXN0aW5nIHB1cnBvc2VzLiIiIgorCisgICAgICAgICMgVGhlIGZpbGVuYW1lcyBzaG93
IHVwIGluIHRoZSBkaWZmIG91dHB1dCwgbWFrZSBzdXJlIHRoZXkncmUKKyAgICAgICAgIyByYXcg
c3RyaW5ncyBhbmQgbm90IHVuaWNvZGUsIHNvIHRoYXQgdGhleSBkb24ndCB0cmlnZ2VyIGpvaW4K
KyAgICAgICAgIyB0cnlpbmcgdG8gZGVjb2RlIHRoZSBpbnB1dAorICAgICAgICBleHBlY3RlZF9m
aWxlbmFtZSA9IHN0cihleHBlY3RlZF9maWxlbmFtZSkKKyAgICAgICAgYWN0dWFsX2ZpbGVuYW1l
ID0gc3RyKGFjdHVhbF9maWxlbmFtZSkKICAgICAgICAgZGlmZiA9IGRpZmZsaWIudW5pZmllZF9k
aWZmKGV4cGVjdGVkX3RleHQuc3BsaXRsaW5lcyhUcnVlKSwKICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgIGFjdHVhbF90ZXh0LnNwbGl0bGluZXMoVHJ1ZSksCiAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICBleHBlY3RlZF9maWxlbmFtZSwKZGlmZiAtLWdpdCBh
L1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZV91bml0
dGVzdC5weSBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQv
YmFzZV91bml0dGVzdC5weQppbmRleCAzOTI2ODM4OGVmOTc0MDc0ZDQyNzEwN2ZhN2M4YWUyMDI0
MDZmN2YxLi42YjRhYWViMjdiMzg0NzdiZmY1OWI4ZGNlZGRlYWNlM2ZhNzM1OGVjIDEwMDY0NAot
LS0gYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jhc2Vf
dW5pdHRlc3QucHkKKysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVz
dHMvcG9ydC9iYXNlX3VuaXR0ZXN0LnB5CkBAIC0xNjMsNiArMTYzLDE3IEBAIGNsYXNzIFBvcnRU
ZXN0KHVuaXR0ZXN0LlRlc3RDYXNlKToKICAgICAgICAgc2VsZi5hc3NlcnRGYWxzZShiYXNlLl93
ZGlmZl9hdmFpbGFibGUpCiAgICAgICAgIGJhc2UuX3dkaWZmX2F2YWlsYWJsZSA9IFRydWUKIAor
ICAgIGRlZiB0ZXN0X2RpZmZfdGV4dChzZWxmKToKKyAgICAgICAgcG9ydCA9IGJhc2UuUG9ydCgp
CisgICAgICAgICMgTWFrZSBzdXJlIHRoYXQgd2UgZG9uJ3QgcnVuIGludG8gZGVjb2RpbmcgZXhj
ZXB0aW9ucyB3aGVuIHRoZQorICAgICAgICAjIGZpbGVuYW1lcyBhcmUgdW5pY29kZSwgd2l0aCBy
ZWd1bGFyIG9yIG1hbGZvcm1lZCBpbnB1dCAoZXhwZWN0ZWQgb3IKKyAgICAgICAgIyBhY3R1YWwg
aW5wdXQgaXMgYWx3YXlzIGEgcmF3IHN0cmluZywgbm90IHVuaWNvZGUpLgorICAgICAgICBwb3J0
LmRpZmZfdGV4dCgnZXhwJywgJ2FjdCcsICdleHAudHh0JywgJ2FjdC50eHQnKQorICAgICAgICBw
b3J0LmRpZmZfdGV4dCgnZXhwJywgJ2FjdCcsIHUnZXhwLnR4dCcsICdhY3QudHh0JykKKworICAg
ICAgICBwb3J0LmRpZmZfdGV4dCgnZXhwJyArIGNocigyNTUpLCAnYWN0JywgJ2V4cC50eHQnLCAn
YWN0LnR4dCcpCisgICAgICAgIHBvcnQuZGlmZl90ZXh0KCdleHAnICsgY2hyKDI1NSksICdhY3Qn
LCB1J2V4cC50eHQnLCAnYWN0LnR4dCcpCisKICAgICBkZWYgdGVzdF9kZWZhdWx0X2NvbmZpZ3Vy
YXRpb25fbm90Zm91bmQoc2VsZik6CiAgICAgICAgIHBvcnQgPSBVbml0VGVzdFBvcnQoKQogICAg
ICAgICBzZWxmLmFzc2VydEVxdWFsKHBvcnQuZGVmYXVsdF9jb25maWd1cmF0aW9uKCksICJSZWxl
YXNlIikK
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>67635</attachid>
            <date>2010-09-14 19:43:51 -0700</date>
            <delta_ts>2010-09-14 20:23:35 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-45791-20100914194350.patch</filename>
            <type>text/plain</type>
            <size>3593</size>
            <attacher name="Mihai Parparita">mihaip</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCAzNDk3ZDMzMjBiNTRkZGQ3YWI4YTM1NTIxMmU5Y2Y0NjY1NDUzMDQ3Li5mNWM1ZmU5
Mjc5Njg1M2JjNGE4N2Q4MzBhOTNjN2IwNzQ2MTYzMTRhIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTcgQEAK
IDIwMTAtMDktMTQgIE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CiAKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTlJXVCBmYWlscyB3
aXRoIFVuaWNvZGVEZWNvZGVFcnJvciBvbiBlZGl0aW5nL3NlbGVjdGlvbi9taXhlZC1lZGl0YWJp
bGl0eS0xMC5odG1sCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD00NTc5MQorCisgICAgICAgIEZvcmNlIGZpbGVuYW1lcyB0byBiZSByYXcgc3RyaW5ncyBi
ZWZvcmUgcnVubmluZyBkaWZmbGliLnVuaWZpZWRfZGlmZi4KKworICAgICAgICAqIFNjcmlwdHMv
d2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZS5weToKKyAgICAgICAgKiBTY3JpcHRzL3dl
YmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jhc2VfdW5pdHRlc3QucHk6CisKKzIwMTAtMDktMTQg
IE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CisKICAgICAgICAgUmV2aWV3
ZWQgYnkgQWRhbSBCYXJ0aC4KIAogICAgICAgICBTZW5kIHdlYmtpdCBhY2Nlc3NpYmlsaXR5IG5v
dGlmaWNhdGlvbnMgdG8gQ2hyb21pdW0KZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1NjcmlwdHMv
d2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZS5weSBiL1dlYktpdFRvb2xzL1NjcmlwdHMv
d2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZS5weQppbmRleCA5MTI1ZjllZDIzZjkwNTI5
ZjI1Yjg2MjMwNjE4ZGVhZmNlZDcyOTY1Li5jMWI1YTgxN2MzYjY4OTQ0Y2UzNzUyOTU2N2ZhZDUz
NDk2ZmM4ZWRlIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91
dF90ZXN0cy9wb3J0L2Jhc2UucHkKKysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9s
YXlvdXRfdGVzdHMvcG9ydC9iYXNlLnB5CkBAIC0xNTIsNiArMTUyLDE1IEBAIGNsYXNzIFBvcnQo
b2JqZWN0KToKIAogICAgICAgICBXaGlsZSB0aGlzIGlzIGEgZ2VuZXJpYyByb3V0aW5lLCB3ZSBp
bmNsdWRlIGl0IGluIHRoZSBQb3J0CiAgICAgICAgIGludGVyZmFjZSBzbyB0aGF0IGl0IGNhbiBi
ZSBvdmVycmlkZW4gZm9yIHRlc3RpbmcgcHVycG9zZXMuIiIiCisKKyAgICAgICAgIyBUaGUgZmls
ZW5hbWVzIHNob3cgdXAgaW4gdGhlIGRpZmYgb3V0cHV0LCBtYWtlIHN1cmUgdGhleSdyZQorICAg
ICAgICAjIHJhdyBzdHJpbmdzIGFuZCBub3QgdW5pY29kZSwgc28gdGhhdCB0aGV5IGRvbid0IHRy
aWdnZXIgam9pbgorICAgICAgICAjIHRyeWluZyB0byBkZWNvZGUgdGhlIGlucHV0CisgICAgICAg
IGRlZiB0b19yYXdfYnl0ZXMoc3RyKToKKyAgICAgICAgICAgIGlmIGlzaW5zdGFuY2Uoc3RyLCB1
bmljb2RlKToKKyAgICAgICAgICAgICAgICByZXR1cm4gc3RyLmVuY29kZSgndXRmLTgnKQorICAg
ICAgICBleHBlY3RlZF9maWxlbmFtZSA9IHRvX3Jhd19ieXRlcyhleHBlY3RlZF9maWxlbmFtZSkK
KyAgICAgICAgYWN0dWFsX2ZpbGVuYW1lID0gdG9fcmF3X2J5dGVzKGFjdHVhbF9maWxlbmFtZSkK
ICAgICAgICAgZGlmZiA9IGRpZmZsaWIudW5pZmllZF9kaWZmKGV4cGVjdGVkX3RleHQuc3BsaXRs
aW5lcyhUcnVlKSwKICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIGFjdHVhbF90
ZXh0LnNwbGl0bGluZXMoVHJ1ZSksCiAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg
ICBleHBlY3RlZF9maWxlbmFtZSwKZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Vi
a2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZV91bml0dGVzdC5weSBiL1dlYktpdFRvb2xzL1Nj
cmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZV91bml0dGVzdC5weQppbmRleCAz
OTI2ODM4OGVmOTc0MDc0ZDQyNzEwN2ZhN2M4YWUyMDI0MDZmN2YxLi5iMWU2OWMxMDZjNDQ1ODI5
ZWQxOTE5NjU0OGNmODM1MjUwYmM2Mjg2IDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9TY3JpcHRz
L3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jhc2VfdW5pdHRlc3QucHkKKysrIGIvV2ViS2l0
VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcG9ydC9iYXNlX3VuaXR0ZXN0LnB5
CkBAIC0xNjMsNiArMTYzLDI1IEBAIGNsYXNzIFBvcnRUZXN0KHVuaXR0ZXN0LlRlc3RDYXNlKToK
ICAgICAgICAgc2VsZi5hc3NlcnRGYWxzZShiYXNlLl93ZGlmZl9hdmFpbGFibGUpCiAgICAgICAg
IGJhc2UuX3dkaWZmX2F2YWlsYWJsZSA9IFRydWUKIAorICAgIGRlZiB0ZXN0X2RpZmZfdGV4dChz
ZWxmKToKKyAgICAgICAgcG9ydCA9IGJhc2UuUG9ydCgpCisgICAgICAgICMgTWFrZSBzdXJlIHRo
YXQgd2UgZG9uJ3QgcnVuIGludG8gZGVjb2RpbmcgZXhjZXB0aW9ucyB3aGVuIHRoZQorICAgICAg
ICAjIGZpbGVuYW1lcyBhcmUgdW5pY29kZSwgd2l0aCByZWd1bGFyIG9yIG1hbGZvcm1lZCBpbnB1
dCAoZXhwZWN0ZWQgb3IKKyAgICAgICAgIyBhY3R1YWwgaW5wdXQgaXMgYWx3YXlzIGEgcmF3IHN0
cmluZywgbm90IHVuaWNvZGUpLgorICAgICAgICBwb3J0LmRpZmZfdGV4dCgnZXhwJywgJ2FjdCcs
ICdleHAudHh0JywgJ2FjdC50eHQnKQorICAgICAgICBwb3J0LmRpZmZfdGV4dCgnZXhwJywgJ2Fj
dCcsIHUnZXhwLnR4dCcsICdhY3QudHh0JykKKyAgICAgICAgcG9ydC5kaWZmX3RleHQoJ2V4cCcs
ICdhY3QnLCB1J2FceGFjXHUxMjM0XHUyMGFjXFUwMDAwODAwMCcsICdhY3QudHh0JykKKworICAg
ICAgICBwb3J0LmRpZmZfdGV4dCgnZXhwJyArIGNocigyNTUpLCAnYWN0JywgJ2V4cC50eHQnLCAn
YWN0LnR4dCcpCisgICAgICAgIHBvcnQuZGlmZl90ZXh0KCdleHAnICsgY2hyKDI1NSksICdhY3Qn
LCB1J2V4cC50eHQnLCAnYWN0LnR4dCcpCisKKyAgICAgICAgIyBUaG91Z2ggZXhwZWN0ZWQgYW5k
IGFjdHVhbCBmaWxlcyBzaG91bGQgYWx3YXlzIGJlIHJlYWQgaW4gd2l0aCBubworICAgICAgICAj
IGVuY29kaW5nIChhbmQgYmUgc3RvcmVkIGFzIHN0ciBvYmplY3RzKSwgdGVzdCB1bmljb2RlIGlu
cHV0cyBqdXN0IHRvCisgICAgICAgICMgYmUgc2FmZQorICAgICAgICBwb3J0LmRpZmZfdGV4dCh1
J2V4cCcsICdhY3QnLCAnZXhwLnR4dCcsICdhY3QudHh0JykKKyAgICAgICAgcG9ydC5kaWZmX3Rl
eHQoCisgICAgICAgICAgICB1J2FceGFjXHUxMjM0XHUyMGFjXFUwMDAwODAwMCcsICdhY3QnLCAn
ZXhwLnR4dCcsICdhY3QudHh0JykKKwogICAgIGRlZiB0ZXN0X2RlZmF1bHRfY29uZmlndXJhdGlv
bl9ub3Rmb3VuZChzZWxmKToKICAgICAgICAgcG9ydCA9IFVuaXRUZXN0UG9ydCgpCiAgICAgICAg
IHNlbGYuYXNzZXJ0RXF1YWwocG9ydC5kZWZhdWx0X2NvbmZpZ3VyYXRpb24oKSwgIlJlbGVhc2Ui
KQo=
</data>
<flag name="review"
          id="56991"
          type_id="1"
          status="-"
          setter="mihaip"
    />
    <flag name="commit-queue"
          id="56992"
          type_id="3"
          status="-"
          setter="mihaip"
    />
          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>67639</attachid>
            <date>2010-09-14 20:27:29 -0700</date>
            <delta_ts>2010-09-15 08:57:56 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-45791-20100914202727.patch</filename>
            <type>text/plain</type>
            <size>3956</size>
            <attacher name="Mihai Parparita">mihaip</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCAzNDk3ZDMzMjBiNTRkZGQ3YWI4YTM1NTIxMmU5Y2Y0NjY1NDUzMDQ3Li5mNWM1ZmU5
Mjc5Njg1M2JjNGE4N2Q4MzBhOTNjN2IwNzQ2MTYzMTRhIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTcgQEAK
IDIwMTAtMDktMTQgIE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CiAKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTlJXVCBmYWlscyB3
aXRoIFVuaWNvZGVEZWNvZGVFcnJvciBvbiBlZGl0aW5nL3NlbGVjdGlvbi9taXhlZC1lZGl0YWJp
bGl0eS0xMC5odG1sCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD00NTc5MQorCisgICAgICAgIEZvcmNlIGZpbGVuYW1lcyB0byBiZSByYXcgc3RyaW5ncyBi
ZWZvcmUgcnVubmluZyBkaWZmbGliLnVuaWZpZWRfZGlmZi4KKworICAgICAgICAqIFNjcmlwdHMv
d2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZS5weToKKyAgICAgICAgKiBTY3JpcHRzL3dl
YmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jhc2VfdW5pdHRlc3QucHk6CisKKzIwMTAtMDktMTQg
IE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CisKICAgICAgICAgUmV2aWV3
ZWQgYnkgQWRhbSBCYXJ0aC4KIAogICAgICAgICBTZW5kIHdlYmtpdCBhY2Nlc3NpYmlsaXR5IG5v
dGlmaWNhdGlvbnMgdG8gQ2hyb21pdW0KZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL1NjcmlwdHMv
d2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZS5weSBiL1dlYktpdFRvb2xzL1NjcmlwdHMv
d2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZS5weQppbmRleCA5MTI1ZjllZDIzZjkwNTI5
ZjI1Yjg2MjMwNjE4ZGVhZmNlZDcyOTY1Li45YjY4OTI5ZWJiYjk1NzVjYjZhNjZkNmNmODVlYTMz
ZjhiN2NhZDQxIDEwMDY0NAotLS0gYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91
dF90ZXN0cy9wb3J0L2Jhc2UucHkKKysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9s
YXlvdXRfdGVzdHMvcG9ydC9iYXNlLnB5CkBAIC0xNTIsNiArMTUyLDE2IEBAIGNsYXNzIFBvcnQo
b2JqZWN0KToKIAogICAgICAgICBXaGlsZSB0aGlzIGlzIGEgZ2VuZXJpYyByb3V0aW5lLCB3ZSBp
bmNsdWRlIGl0IGluIHRoZSBQb3J0CiAgICAgICAgIGludGVyZmFjZSBzbyB0aGF0IGl0IGNhbiBi
ZSBvdmVycmlkZW4gZm9yIHRlc3RpbmcgcHVycG9zZXMuIiIiCisKKyAgICAgICAgIyBUaGUgZmls
ZW5hbWVzIHNob3cgdXAgaW4gdGhlIGRpZmYgb3V0cHV0LCBtYWtlIHN1cmUgdGhleSdyZQorICAg
ICAgICAjIHJhdyBzdHJpbmdzIGFuZCBub3QgdW5pY29kZSwgc28gdGhhdCB0aGV5IGRvbid0IHRy
aWdnZXIgam9pbgorICAgICAgICAjIHRyeWluZyB0byBkZWNvZGUgdGhlIGlucHV0CisgICAgICAg
IGRlZiB0b19yYXdfYnl0ZXMoc3RyKToKKyAgICAgICAgICAgIGlmIGlzaW5zdGFuY2Uoc3RyLCB1
bmljb2RlKToKKyAgICAgICAgICAgICAgICByZXR1cm4gc3RyLmVuY29kZSgndXRmLTgnKQorICAg
ICAgICAgICAgcmV0dXJuIHN0cgorICAgICAgICBleHBlY3RlZF9maWxlbmFtZSA9IHRvX3Jhd19i
eXRlcyhleHBlY3RlZF9maWxlbmFtZSkKKyAgICAgICAgYWN0dWFsX2ZpbGVuYW1lID0gdG9fcmF3
X2J5dGVzKGFjdHVhbF9maWxlbmFtZSkKICAgICAgICAgZGlmZiA9IGRpZmZsaWIudW5pZmllZF9k
aWZmKGV4cGVjdGVkX3RleHQuc3BsaXRsaW5lcyhUcnVlKSwKICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgIGFjdHVhbF90ZXh0LnNwbGl0bGluZXMoVHJ1ZSksCiAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICBleHBlY3RlZF9maWxlbmFtZSwKZGlmZiAtLWdpdCBh
L1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZV91bml0
dGVzdC5weSBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQv
YmFzZV91bml0dGVzdC5weQppbmRleCAzOTI2ODM4OGVmOTc0MDc0ZDQyNzEwN2ZhN2M4YWUyMDI0
MDZmN2YxLi41NTYyMzM4NjlhMzJjMTZjZjRkOGE4N2M3YWE2MTRmYTJhNGNkYjc1IDEwMDY0NAot
LS0gYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jhc2Vf
dW5pdHRlc3QucHkKKysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRfdGVz
dHMvcG9ydC9iYXNlX3VuaXR0ZXN0LnB5CkBAIC0xNjMsNiArMTYzLDMzIEBAIGNsYXNzIFBvcnRU
ZXN0KHVuaXR0ZXN0LlRlc3RDYXNlKToKICAgICAgICAgc2VsZi5hc3NlcnRGYWxzZShiYXNlLl93
ZGlmZl9hdmFpbGFibGUpCiAgICAgICAgIGJhc2UuX3dkaWZmX2F2YWlsYWJsZSA9IFRydWUKIAor
ICAgIGRlZiB0ZXN0X2RpZmZfdGV4dChzZWxmKToKKyAgICAgICAgcG9ydCA9IGJhc2UuUG9ydCgp
CisgICAgICAgICMgTWFrZSBzdXJlIHRoYXQgd2UgZG9uJ3QgcnVuIGludG8gZGVjb2RpbmcgZXhj
ZXB0aW9ucyB3aGVuIHRoZQorICAgICAgICAjIGZpbGVuYW1lcyBhcmUgdW5pY29kZSwgd2l0aCBy
ZWd1bGFyIG9yIG1hbGZvcm1lZCBpbnB1dCAoZXhwZWN0ZWQgb3IKKyAgICAgICAgIyBhY3R1YWwg
aW5wdXQgaXMgYWx3YXlzIGEgcmF3IHN0cmluZywgbm90IHVuaWNvZGUpLgorICAgICAgICBwb3J0
LmRpZmZfdGV4dCgnZXhwJywgJ2FjdCcsICdleHAudHh0JywgJ2FjdC50eHQnKQorICAgICAgICBw
b3J0LmRpZmZfdGV4dCgnZXhwJywgJ2FjdCcsIHUnZXhwLnR4dCcsICdhY3QudHh0JykKKyAgICAg
ICAgcG9ydC5kaWZmX3RleHQoJ2V4cCcsICdhY3QnLCB1J2FceGFjXHUxMjM0XHUyMGFjXFUwMDAw
ODAwMCcsICdhY3QudHh0JykKKworICAgICAgICBwb3J0LmRpZmZfdGV4dCgnZXhwJyArIGNocigy
NTUpLCAnYWN0JywgJ2V4cC50eHQnLCAnYWN0LnR4dCcpCisgICAgICAgIHBvcnQuZGlmZl90ZXh0
KCdleHAnICsgY2hyKDI1NSksICdhY3QnLCB1J2V4cC50eHQnLCAnYWN0LnR4dCcpCisKKyAgICAg
ICAgIyBUaG91Z2ggZXhwZWN0ZWQgYW5kIGFjdHVhbCBmaWxlcyBzaG91bGQgYWx3YXlzIGJlIHJl
YWQgaW4gd2l0aCBubworICAgICAgICAjIGVuY29kaW5nIChhbmQgYmUgc3RvcmVkIGFzIHN0ciBv
YmplY3RzKSwgdGVzdCB1bmljb2RlIGlucHV0cyBqdXN0IHRvCisgICAgICAgICMgYmUgc2FmZQor
ICAgICAgICBwb3J0LmRpZmZfdGV4dCh1J2V4cCcsICdhY3QnLCAnZXhwLnR4dCcsICdhY3QudHh0
JykKKyAgICAgICAgcG9ydC5kaWZmX3RleHQoCisgICAgICAgICAgICB1J2FceGFjXHUxMjM0XHUy
MGFjXFUwMDAwODAwMCcsICdhY3QnLCAnZXhwLnR4dCcsICdhY3QudHh0JykKKworICAgICAgICAj
IEFuZCBtYWtlIHN1cmUgd2UgYWN0dWFsbHkgZ2V0IGRpZmYgb3V0cHV0CisgICAgICAgIGRpZmYg
PSBwb3J0LmRpZmZfdGV4dCgnZm9vJywgJ2JhcicsICdleHAudHh0JywgJ2FjdC50eHQnKQorICAg
ICAgICBzZWxmLmFzc2VydFRydWUoJ2ZvbycgaW4gZGlmZikKKyAgICAgICAgc2VsZi5hc3NlcnRU
cnVlKCdiYXInIGluIGRpZmYpCisgICAgICAgIHNlbGYuYXNzZXJ0VHJ1ZSgnZXhwLnR4dCcgaW4g
ZGlmZikKKyAgICAgICAgc2VsZi5hc3NlcnRUcnVlKCdhY3QudHh0JyBpbiBkaWZmKQorICAgICAg
ICBzZWxmLmFzc2VydEZhbHNlKCdub3N1Y2h0aGluZycgaW4gZGlmZikKKwogICAgIGRlZiB0ZXN0
X2RlZmF1bHRfY29uZmlndXJhdGlvbl9ub3Rmb3VuZChzZWxmKToKICAgICAgICAgcG9ydCA9IFVu
aXRUZXN0UG9ydCgpCiAgICAgICAgIHNlbGYuYXNzZXJ0RXF1YWwocG9ydC5kZWZhdWx0X2NvbmZp
Z3VyYXRpb24oKSwgIlJlbGVhc2UiKQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>67681</attachid>
            <date>2010-09-15 08:58:00 -0700</date>
            <delta_ts>2010-09-15 10:06:22 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-45791-20100915085758.patch</filename>
            <type>text/plain</type>
            <size>3901</size>
            <attacher name="Mihai Parparita">mihaip</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFRvb2xzL0NoYW5nZUxvZyBiL1dlYktpdFRvb2xzL0NoYW5nZUxv
ZwppbmRleCAzMWQzNzIyMGQ0YTgxNGUzMDJhMDBiMjE2YWEzMDBmODQ4YjM1MDUyLi41MmUyMzE1
YTkzZGQ5YTMzODZiODBlYTI2ODkxZjc5ZGU4ODViNDdmIDEwMDY0NAotLS0gYS9XZWJLaXRUb29s
cy9DaGFuZ2VMb2cKKysrIGIvV2ViS2l0VG9vbHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAK
KzIwMTAtMDktMTQgIE1paGFpIFBhcnBhcml0YSAgPG1paGFpcEBjaHJvbWl1bS5vcmc+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgTlJXVCBmYWlscyB3
aXRoIFVuaWNvZGVEZWNvZGVFcnJvciBvbiBlZGl0aW5nL3NlbGVjdGlvbi9taXhlZC1lZGl0YWJp
bGl0eS0xMC5odG1sCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD00NTc5MQorCisgICAgICAgIEZvcmNlIGZpbGVuYW1lcyB0byBiZSByYXcgYnl0ZXMgYmVm
b3JlIHJ1bm5pbmcgZGlmZmxpYi51bmlmaWVkX2RpZmYuCisKKyAgICAgICAgKiBTY3JpcHRzL3dl
YmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jhc2UucHk6CisgICAgICAgICogU2NyaXB0cy93ZWJr
aXRweS9sYXlvdXRfdGVzdHMvcG9ydC9iYXNlX3VuaXR0ZXN0LnB5OgorCiAyMDEwLTA5LTE1ICBT
aW1vbiBIYXVzbWFubiAgPHNpbW9uLmhhdXNtYW5uQG5va2lhLmNvbT4KIAogICAgICAgICBSZXZp
ZXdlZCBieSBUb3IgQXJuZSBWZXN0YsO4LgpkaWZmIC0tZ2l0IGEvV2ViS2l0VG9vbHMvU2NyaXB0
cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcG9ydC9iYXNlLnB5IGIvV2ViS2l0VG9vbHMvU2NyaXB0
cy93ZWJraXRweS9sYXlvdXRfdGVzdHMvcG9ydC9iYXNlLnB5CmluZGV4IDkxMjVmOWVkMjNmOTA1
MjlmMjViODYyMzA2MThkZWFmY2VkNzI5NjUuLmMyMWM4ODk0N2JjOTdjMDZjYTcwZDk0NjYxOGM1
MGYyZDU5N2Y4NzUgMTAwNjQ0Ci0tLSBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5
b3V0X3Rlc3RzL3BvcnQvYmFzZS5weQorKysgYi9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5
L2xheW91dF90ZXN0cy9wb3J0L2Jhc2UucHkKQEAgLTE1Miw2ICsxNTIsMTYgQEAgY2xhc3MgUG9y
dChvYmplY3QpOgogCiAgICAgICAgIFdoaWxlIHRoaXMgaXMgYSBnZW5lcmljIHJvdXRpbmUsIHdl
IGluY2x1ZGUgaXQgaW4gdGhlIFBvcnQKICAgICAgICAgaW50ZXJmYWNlIHNvIHRoYXQgaXQgY2Fu
IGJlIG92ZXJyaWRlbiBmb3IgdGVzdGluZyBwdXJwb3Nlcy4iIiIKKworICAgICAgICAjIFRoZSBm
aWxlbmFtZXMgc2hvdyB1cCBpbiB0aGUgZGlmZiBvdXRwdXQsIG1ha2Ugc3VyZSB0aGV5J3JlCisg
ICAgICAgICMgcmF3IGJ5dGVzIGFuZCBub3QgdW5pY29kZSwgc28gdGhhdCB0aGV5IGRvbid0IHRy
aWdnZXIgam9pbigpCisgICAgICAgICMgdHJ5aW5nIHRvIGRlY29kZSB0aGUgaW5wdXQuCisgICAg
ICAgIGRlZiB0b19yYXdfYnl0ZXMoc3RyKToKKyAgICAgICAgICAgIGlmIGlzaW5zdGFuY2Uoc3Ry
LCB1bmljb2RlKToKKyAgICAgICAgICAgICAgICByZXR1cm4gc3RyLmVuY29kZSgndXRmLTgnKQor
ICAgICAgICAgICAgcmV0dXJuIHN0cgorICAgICAgICBleHBlY3RlZF9maWxlbmFtZSA9IHRvX3Jh
d19ieXRlcyhleHBlY3RlZF9maWxlbmFtZSkKKyAgICAgICAgYWN0dWFsX2ZpbGVuYW1lID0gdG9f
cmF3X2J5dGVzKGFjdHVhbF9maWxlbmFtZSkKICAgICAgICAgZGlmZiA9IGRpZmZsaWIudW5pZmll
ZF9kaWZmKGV4cGVjdGVkX3RleHQuc3BsaXRsaW5lcyhUcnVlKSwKICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgIGFjdHVhbF90ZXh0LnNwbGl0bGluZXMoVHJ1ZSksCiAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICBleHBlY3RlZF9maWxlbmFtZSwKZGlmZiAtLWdp
dCBhL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3BvcnQvYmFzZV91
bml0dGVzdC5weSBiL1dlYktpdFRvb2xzL1NjcmlwdHMvd2Via2l0cHkvbGF5b3V0X3Rlc3RzL3Bv
cnQvYmFzZV91bml0dGVzdC5weQppbmRleCAzOTI2ODM4OGVmOTc0MDc0ZDQyNzEwN2ZhN2M4YWUy
MDI0MDZmN2YxLi40MDdiOTA2MWMxZjQ3Y2ExZTc5YjdkYTlkNDMwNWUyYThhMTJjOWQ2IDEwMDY0
NAotLS0gYS9XZWJLaXRUb29scy9TY3JpcHRzL3dlYmtpdHB5L2xheW91dF90ZXN0cy9wb3J0L2Jh
c2VfdW5pdHRlc3QucHkKKysrIGIvV2ViS2l0VG9vbHMvU2NyaXB0cy93ZWJraXRweS9sYXlvdXRf
dGVzdHMvcG9ydC9iYXNlX3VuaXR0ZXN0LnB5CkBAIC0xNjMsNiArMTYzLDMzIEBAIGNsYXNzIFBv
cnRUZXN0KHVuaXR0ZXN0LlRlc3RDYXNlKToKICAgICAgICAgc2VsZi5hc3NlcnRGYWxzZShiYXNl
Ll93ZGlmZl9hdmFpbGFibGUpCiAgICAgICAgIGJhc2UuX3dkaWZmX2F2YWlsYWJsZSA9IFRydWUK
IAorICAgIGRlZiB0ZXN0X2RpZmZfdGV4dChzZWxmKToKKyAgICAgICAgcG9ydCA9IGJhc2UuUG9y
dCgpCisgICAgICAgICMgTWFrZSBzdXJlIHRoYXQgd2UgZG9uJ3QgcnVuIGludG8gZGVjb2Rpbmcg
ZXhjZXB0aW9ucyB3aGVuIHRoZQorICAgICAgICAjIGZpbGVuYW1lcyBhcmUgdW5pY29kZSwgd2l0
aCByZWd1bGFyIG9yIG1hbGZvcm1lZCBpbnB1dCAoZXhwZWN0ZWQgb3IKKyAgICAgICAgIyBhY3R1
YWwgaW5wdXQgaXMgYWx3YXlzIHJhdyBieXRlcywgbm90IHVuaWNvZGUpLgorICAgICAgICBwb3J0
LmRpZmZfdGV4dCgnZXhwJywgJ2FjdCcsICdleHAudHh0JywgJ2FjdC50eHQnKQorICAgICAgICBw
b3J0LmRpZmZfdGV4dCgnZXhwJywgJ2FjdCcsIHUnZXhwLnR4dCcsICdhY3QudHh0JykKKyAgICAg
ICAgcG9ydC5kaWZmX3RleHQoJ2V4cCcsICdhY3QnLCB1J2FceGFjXHUxMjM0XHUyMGFjXFUwMDAw
ODAwMCcsICdhY3QudHh0JykKKworICAgICAgICBwb3J0LmRpZmZfdGV4dCgnZXhwJyArIGNocigy
NTUpLCAnYWN0JywgJ2V4cC50eHQnLCAnYWN0LnR4dCcpCisgICAgICAgIHBvcnQuZGlmZl90ZXh0
KCdleHAnICsgY2hyKDI1NSksICdhY3QnLCB1J2V4cC50eHQnLCAnYWN0LnR4dCcpCisKKyAgICAg
ICAgIyBUaG91Z2ggZXhwZWN0ZWQgYW5kIGFjdHVhbCBmaWxlcyBzaG91bGQgYWx3YXlzIGJlIHJl
YWQgaW4gd2l0aCBubworICAgICAgICAjIGVuY29kaW5nIChhbmQgYmUgc3RvcmVkIGFzIHN0ciBv
YmplY3RzKSwgdGVzdCB1bmljb2RlIGlucHV0cyBqdXN0IHRvCisgICAgICAgICMgYmUgc2FmZS4K
KyAgICAgICAgcG9ydC5kaWZmX3RleHQodSdleHAnLCAnYWN0JywgJ2V4cC50eHQnLCAnYWN0LnR4
dCcpCisgICAgICAgIHBvcnQuZGlmZl90ZXh0KAorICAgICAgICAgICAgdSdhXHhhY1x1MTIzNFx1
MjBhY1xVMDAwMDgwMDAnLCAnYWN0JywgJ2V4cC50eHQnLCAnYWN0LnR4dCcpCisKKyAgICAgICAg
IyBBbmQgbWFrZSBzdXJlIHdlIGFjdHVhbGx5IGdldCBkaWZmIG91dHB1dC4KKyAgICAgICAgZGlm
ZiA9IHBvcnQuZGlmZl90ZXh0KCdmb28nLCAnYmFyJywgJ2V4cC50eHQnLCAnYWN0LnR4dCcpCisg
ICAgICAgIHNlbGYuYXNzZXJ0VHJ1ZSgnZm9vJyBpbiBkaWZmKQorICAgICAgICBzZWxmLmFzc2Vy
dFRydWUoJ2JhcicgaW4gZGlmZikKKyAgICAgICAgc2VsZi5hc3NlcnRUcnVlKCdleHAudHh0JyBp
biBkaWZmKQorICAgICAgICBzZWxmLmFzc2VydFRydWUoJ2FjdC50eHQnIGluIGRpZmYpCisgICAg
ICAgIHNlbGYuYXNzZXJ0RmFsc2UoJ25vc3VjaHRoaW5nJyBpbiBkaWZmKQorCiAgICAgZGVmIHRl
c3RfZGVmYXVsdF9jb25maWd1cmF0aW9uX25vdGZvdW5kKHNlbGYpOgogICAgICAgICBwb3J0ID0g
VW5pdFRlc3RQb3J0KCkKICAgICAgICAgc2VsZi5hc3NlcnRFcXVhbChwb3J0LmRlZmF1bHRfY29u
ZmlndXJhdGlvbigpLCAiUmVsZWFzZSIpCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>