<?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>81136</bug_id>
          
          <creation_ts>2012-03-14 11:17:03 -0700</creation_ts>
          <short_desc>Need implement isWordTextBreak for QT, and WinCE for visual word movement functionality</short_desc>
          <delta_ts>2012-10-09 12:14:29 -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>HTML Editing</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>25298</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Xiaomei Ji">xji</reporter>
          <assigned_to name="Tullio Lucena">tullio.lucena</assigned_to>
          <cc>cmarcelo</cc>
    
    <cc>jdapena</cc>
    
    <cc>mrobinson</cc>
    
    <cc>ossy</cc>
    
    <cc>rafael.lobo</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>578576</commentid>
    <comment_count>0</comment_count>
    <who name="Xiaomei Ji">xji</who>
    <bug_when>2012-03-14 11:17:03 -0700</bug_when>
    <thetext>visual word movement relies isWordTextBreak.
isWordTextBreak returns true if the word break iterator moving across a word, and it returns false if the word break iterator moving across a punctuation/space.
In platforms using ICU, isWordTextBreak uses ICU break iterator functionality.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>587109</commentid>
    <comment_count>1</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2012-03-24 09:07:01 -0700</bug_when>
    <thetext>I cannot find isWordTextBreak anywhere in the WebKit source code. Is this something you&apos;re working on now? For the record GTK+ uses ICU by default, so there shouldn&apos;t be much in the way of implementation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>588024</commentid>
    <comment_count>2</comment_count>
    <who name="Xiaomei Ji">xji</who>
    <bug_when>2012-03-26 12:56:20 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; I cannot find isWordTextBreak anywhere in the WebKit source code. Is this something you&apos;re working on now? For the record GTK+ uses ICU by default, so there shouldn&apos;t be much in the way of implementation.

It is added in r110965
under Source/WebCore/platform/text/TextBreakIterator.h.

If you look at its implementation under TextBreakIteratorICU.cpp,
basically, it is checking the current word break iterator just pass over a word.
For example, &quot;abc def&quot;, both positions &quot;abc| |def&quot; are word break positions,
but isWordTextBreak returns true for &quot;abc| def&quot; since the break iterator just pass over &quot;abc&quot;. And it returns false for &quot;abc |def&quot; since the break iterator just pass over a space.

If GTK+ uses ICU, that should be easy to implemented.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>588044</commentid>
    <comment_count>3</comment_count>
    <who name="Martin Robinson">mrobinson</who>
    <bug_when>2012-03-26 13:13:20 -0700</bug_when>
    <thetext>(In reply to comment #2)

&gt; If GTK+ uses ICU, that should be easy to implemented.

Look like this just needs to be implemented for the GLib unicode backend, which we don&apos;t have complete support for anyway. By default GTK+ uses Source/WebCore/platform/text/TextBreakIteratorICU.cpp.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>588082</commentid>
    <comment_count>4</comment_count>
    <who name="Xiaomei Ji">xji</who>
    <bug_when>2012-03-26 13:40:48 -0700</bug_when>
    <thetext>(In reply to comment #3)
&gt; (In reply to comment #2)
&gt; 
&gt; &gt; If GTK+ uses ICU, that should be easy to implemented.
&gt; 
&gt; Look like this just needs to be implemented for the GLib unicode backend, which we don&apos;t have complete support for anyway. By default GTK+ uses Source/WebCore/platform/text/TextBreakIteratorICU.cpp.

Ah, then we can turn it on for GTK.
I thought GTK uses TextBreakIteratorGtk.cpp.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>588862</commentid>
    <comment_count>5</comment_count>
    <who name="José Dapena Paz">jdapena</who>
    <bug_when>2012-03-27 07:25:27 -0700</bug_when>
    <thetext>(In reply to comment #2)
&gt; It is added in r110965
&gt; under Source/WebCore/platform/text/TextBreakIterator.h.
&gt; 
&gt; If you look at its implementation under TextBreakIteratorICU.cpp,
&gt; basically, it is checking the current word break iterator just pass over a word.
&gt; For example, &quot;abc def&quot;, both positions &quot;abc| |def&quot; are word break positions,
&gt; but isWordTextBreak returns true for &quot;abc| def&quot; since the break iterator just pass over &quot;abc&quot;. And it returns false for &quot;abc |def&quot; since the break iterator just pass over a space.
&gt; 
&gt; If GTK+ uses ICU, that should be easy to implemented.

I&apos;ve tried running the Skipped tests, and all but one run without problems in webkitgtk, as it uses ICU backend.

The only test failing is editing/selection/move-by-word-visually-multi-line, with this kind of failure:

 Test 1, LTR:
 Move right by one word
-&quot;abc def ghi jkl mn &quot;[0, 4, 8, 12, 16, 19], &quot;opq rst uvw xyz&quot;[0, 4, 8, 12, 15]
+&quot;abc def ghi jkl mn &quot;[0, 4, 8, 12, 16], &quot;opq rst uvw xyz&quot;[0, 4, 8, 12, 15]
 Move left by one word
 &quot;opq rst uvw xyz&quot;[15, 12, 8, 4, 0], &quot;abc def ghi jkl mn &quot;[16, 12, 8, 4, 0]


 Test 7, RTL:
 Move left by one word
-&quot;abc def ghi jkl mn &quot;[0, 3, 8, 11, 16, 19], &quot;opq rst uvw xyz&quot;[0, 3, 8, 11, 15]
+&quot;abc def ghi jkl mn &quot;[0, 3, 8, 11, 16], &quot;opq rst uvw xyz&quot;[0, 3, 8, 11, 15]
 Move right by one word
 &quot;opq rst uvw xyz&quot;[15, 11, 8, 3, 0], &quot;abc def ghi jkl mn &quot;[16, 11, 8, 3, 0]

Seems that the ICU implementation is not considering the blank space at the end of lines as a word break. Is it expected?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>600553</commentid>
    <comment_count>6</comment_count>
    <who name="Xiaomei Ji">xji</who>
    <bug_when>2012-04-11 13:38:10 -0700</bug_when>
    <thetext>Sorry for replying late.
I think gtk&apos;s result is more reasonable (in terms of consistent with other output in other tests).

Since both gtk and chromium uses ICU, I will need to check why chromium&apos;s result is different.

(In reply to comment #5)
&gt; (In reply to comment #2)
&gt; &gt; It is added in r110965
&gt; &gt; under Source/WebCore/platform/text/TextBreakIterator.h.
&gt; &gt; 
&gt; &gt; If you look at its implementation under TextBreakIteratorICU.cpp,
&gt; &gt; basically, it is checking the current word break iterator just pass over a word.
&gt; &gt; For example, &quot;abc def&quot;, both positions &quot;abc| |def&quot; are word break positions,
&gt; &gt; but isWordTextBreak returns true for &quot;abc| def&quot; since the break iterator just pass over &quot;abc&quot;. And it returns false for &quot;abc |def&quot; since the break iterator just pass over a space.
&gt; &gt; 
&gt; &gt; If GTK+ uses ICU, that should be easy to implemented.
&gt; 
&gt; I&apos;ve tried running the Skipped tests, and all but one run without problems in webkitgtk, as it uses ICU backend.
&gt; 
&gt; The only test failing is editing/selection/move-by-word-visually-multi-line, with this kind of failure:
&gt; 
&gt;  Test 1, LTR:
&gt;  Move right by one word
&gt; -&quot;abc def ghi jkl mn &quot;[0, 4, 8, 12, 16, 19], &quot;opq rst uvw xyz&quot;[0, 4, 8, 12, 15]
&gt; +&quot;abc def ghi jkl mn &quot;[0, 4, 8, 12, 16], &quot;opq rst uvw xyz&quot;[0, 4, 8, 12, 15]
&gt;  Move left by one word
&gt;  &quot;opq rst uvw xyz&quot;[15, 12, 8, 4, 0], &quot;abc def ghi jkl mn &quot;[16, 12, 8, 4, 0]
&gt; 
&gt; 
&gt;  Test 7, RTL:
&gt;  Move left by one word
&gt; -&quot;abc def ghi jkl mn &quot;[0, 3, 8, 11, 16, 19], &quot;opq rst uvw xyz&quot;[0, 3, 8, 11, 15]
&gt; +&quot;abc def ghi jkl mn &quot;[0, 3, 8, 11, 16], &quot;opq rst uvw xyz&quot;[0, 3, 8, 11, 15]
&gt;  Move right by one word
&gt;  &quot;opq rst uvw xyz&quot;[15, 11, 8, 3, 0], &quot;abc def ghi jkl mn &quot;[16, 11, 8, 3, 0]
&gt; 
&gt; Seems that the ICU implementation is not considering the blank space at the end of lines as a word break. Is it expected?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>726958</commentid>
    <comment_count>7</comment_count>
      <attachid>165437</attachid>
    <who name="Tullio Lucena">tullio.lucena</who>
    <bug_when>2012-09-24 13:27:07 -0700</bug_when>
    <thetext>Created attachment 165437
Unskipping passing tests</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>728670</commentid>
    <comment_count>8</comment_count>
      <attachid>165437</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-09-26 09:36:17 -0700</bug_when>
    <thetext>Comment on attachment 165437
Unskipping passing tests

I tested it and works, so r=me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>728671</commentid>
    <comment_count>9</comment_count>
      <attachid>165437</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-26 09:38:29 -0700</bug_when>
    <thetext>Comment on attachment 165437
Unskipping passing tests

Rejecting attachment 165437 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: 1

ERROR: /mnt/git/webkit-commit-queue/LayoutTests/ChangeLog neither lists a valid reviewer nor contains the string &quot;Unreviewed&quot; or &quot;Rubber stamp&quot; (case insensitive).

Full output: http://queues.webkit.org/results/14038354</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>728703</commentid>
    <comment_count>10</comment_count>
      <attachid>165827</attachid>
    <who name="Tullio Lucena">tullio.lucena</who>
    <bug_when>2012-09-26 10:19:40 -0700</bug_when>
    <thetext>Created attachment 165827
Fi</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>728704</commentid>
    <comment_count>11</comment_count>
      <attachid>165827</attachid>
    <who name="Tullio Lucena">tullio.lucena</who>
    <bug_when>2012-09-26 10:20:12 -0700</bug_when>
    <thetext>Comment on attachment 165827
Fi

fixed changelog</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>728749</commentid>
    <comment_count>12</comment_count>
      <attachid>165827</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-26 11:00:59 -0700</bug_when>
    <thetext>Comment on attachment 165827
Fi

Clearing flags on attachment: 165827

Committed r129671: &lt;http://trac.webkit.org/changeset/129671&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>729577</commentid>
    <comment_count>13</comment_count>
      <attachid>165827</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-09-27 09:05:39 -0700</bug_when>
    <thetext>Comment on attachment 165827
Fi

Ooops, it was already landed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>729581</commentid>
    <comment_count>14</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-09-27 09:06:35 -0700</bug_when>
    <thetext>Yay. Is the bug fixed?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>729672</commentid>
    <comment_count>15</comment_count>
    <who name="Xiaomei Ji">xji</who>
    <bug_when>2012-09-27 10:24:29 -0700</bug_when>
    <thetext>If it is fixed for QT, how about change the bug to WinCE only and keep it open?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>730644</commentid>
    <comment_count>16</comment_count>
    <who name="Rafael Brandao">rafael.lobo</who>
    <bug_when>2012-09-28 07:38:02 -0700</bug_when>
    <thetext>Ossy, Tullio is investigating one of the two remaining tests:
editing/selection/move-by-word-visually-multi-line.html

I think we can keep this bug open for instance. I believe this came after Qt started to use ICU. :-)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>733918</commentid>
    <comment_count>17</comment_count>
      <attachid>166937</attachid>
    <who name="Tullio Lucena">tullio.lucena</who>
    <bug_when>2012-10-03 12:20:25 -0700</bug_when>
    <thetext>Created attachment 166937
Unskipping last tests

The problem in this last tests was at the Hebrew fonts.
The changes at testfonts [1] made this tests pass now.
If this patch pass at bots, i think that this bug is fixed.

[1] https://gitorious.org/qtwebkit/testfonts/merge_requests/1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738044</commentid>
    <comment_count>18</comment_count>
      <attachid>166937</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-09 12:14:25 -0700</bug_when>
    <thetext>Comment on attachment 166937
Unskipping last tests

Clearing flags on attachment: 166937

Committed r130788: &lt;http://trac.webkit.org/changeset/130788&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>738045</commentid>
    <comment_count>19</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-09 12:14:29 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>165437</attachid>
            <date>2012-09-24 13:27:07 -0700</date>
            <delta_ts>2012-09-27 09:05:08 -0700</delta_ts>
            <desc>Unskipping passing tests</desc>
            <filename>patch.diff</filename>
            <type>text/plain</type>
            <size>2032</size>
            <attacher name="Tullio Lucena">tullio.lucena</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCBkMzAyOGI2Li5jM2E5NWVjIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTIgQEAKKzIwMTItMDkt
MjQgIFR1bGxpbyBMdWNlbmEgIDx0dWxsaW8ubHVjZW5hQG9wZW5ib3NzYS5vcmc+CisKKyAgICAg
ICAgTmVlZCBpbXBsZW1lbnQgaXNXb3JkVGV4dEJyZWFrIGZvciBRVCwgYW5kIFdpbkNFIGZvciB2
aXN1YWwgd29yZCBtb3ZlbWVudCBmdW5jdGlvbmFsaXR5CisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTEzNgorCisgICAgICAgIFVuc2tpcCBzb21lIHRl
c3RzIHRoYXQgYXJlIHBhc3NpbmcuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9xdC9Ta2lwcGVkOgor
CiAyMDEyLTA5LTI0ICBSYXBoYWVsIEt1Ym8gZGEgQ29zdGEgIDxyYXBoYWVsLmt1Ym8uZGEuY29z
dGFAaW50ZWwuY29tPgogCiAgICAgICAgIFtFRkxdIFVucmV2aWV3ZWQgZ2FyZGVuaW5nLgpkaWZm
IC0tZ2l0IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQvU2tpcHBlZCBiL0xheW91dFRlc3RzL3Bs
YXRmb3JtL3F0L1NraXBwZWQKaW5kZXggMDViNzQwNS4uY2Y3Njg5YyAxMDA2NDQKLS0tIGEvTGF5
b3V0VGVzdHMvcGxhdGZvcm0vcXQvU2tpcHBlZAorKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9x
dC9Ta2lwcGVkCkBAIC0yNTI1LDIxICsyNTI1LDcgQEAgZmFzdC93b3JrZXJzL3dvcmtlci1jcmFz
aC13aXRoLWludmFsaWQtbG9jYXRpb24uaHRtbAogZmFzdC9ydWJ5L3RleHQtZGVjb3JhdGlvbi1p
bi1kZXNjZW5kYW50cy1ydWJ5Lmh0bWwKIAogIyBGYWlsaW5nIHNpbmNlIGlzV29yZFRleHRCcmVh
ayBpcyBub3QgaW1wbGVtZW50ZWQuIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD04MTEzNgotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5LWNyYXNo
LXRlc3QtMS5odG1sCi1lZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHktY3Jh
c2gtdGVzdC0yLmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnktd29yZC12aXN1YWxseS1j
cmFzaC10ZXN0LTMuaHRtbAotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5
LWNyYXNoLXRlc3QtNC5odG1sCi1lZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFs
bHktY3Jhc2gtdGVzdC01Lmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnktd29yZC12aXN1
YWxseS1jcmFzaC10ZXN0LXRleHRhcmVhLmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnkt
d29yZC12aXN1YWxseS1jcmFzaC10ZXN0LWNzcy1nZW5lcmF0ZWQtY29udGVudC5odG1sCi1lZGl0
aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHktaW5saW5lLWJsb2NrLXBvc2l0aW9u
ZWQtZWxlbWVudC5odG1sCiBlZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHkt
bXVsdGktbGluZS5odG1sCi1lZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHkt
bXVsdGktc3BhY2UuaHRtbAotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5
LW51bGwtYm94Lmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnktd29yZC12aXN1YWxseS1z
aW5nbGUtc3BhY2UtaW5saW5lLWVsZW1lbnQuaHRtbAotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1i
eS13b3JkLXZpc3VhbGx5LXNpbmdsZS1zcGFjZS1vbmUtZWxlbWVudC5odG1sCi1lZGl0aW5nL3Nl
bGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHktdGV4dGFyZWEuaHRtbAotZWRpdGluZy9zZWxl
Y3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5LXdyb25nLWxlZnQtcmlnaHQuaHRtbAogZWRpdGlu
Zy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5LW1hYy5odG1sCiAKICMgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTgyNjI4Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>165827</attachid>
            <date>2012-09-26 10:19:40 -0700</date>
            <delta_ts>2012-09-27 09:05:39 -0700</delta_ts>
            <desc>Fi</desc>
            <filename>patch.diff</filename>
            <type>text/plain</type>
            <size>2079</size>
            <attacher name="Tullio Lucena">tullio.lucena</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCAxNDUyN2I0Li4xMGNhYzE3IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTItMDkt
MjYgIFR1bGxpbyBMdWNlbmEgIDx0dWxsaW8ubHVjZW5hQG9wZW5ib3NzYS5vcmc+CisKKyAgICAg
ICAgTmVlZCBpbXBsZW1lbnQgaXNXb3JkVGV4dEJyZWFrIGZvciBRVCwgYW5kIFdpbkNFIGZvciB2
aXN1YWwgd29yZCBtb3ZlbWVudCBmdW5jdGlvbmFsaXR5CisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTEzNgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFVuc2tpcCBzb21lIHRlc3RzIHRoYXQgYXJlIHBhc3Np
bmcuCisKKyAgICAgICAgKiBwbGF0Zm9ybS9xdC9Ta2lwcGVkOgorCiAyMDEyLTA5LTI2ICBQZXRl
ciBCZXZlcmxvbyAgPHBldGVyQGNocm9taXVtLm9yZz4KIAogICAgICAgICBbQ2hyb21pdW1dIFNl
Y29uZCBwYXJ0IG9mIEFuZHJvaWQncyBmYXN0LyByZWJhc2VsaW5pbmcuCmRpZmYgLS1naXQgYS9M
YXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9Ta2lwcGVkIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQv
U2tpcHBlZAppbmRleCAyM2ViNWQyLi5jY2M4MWNkIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9w
bGF0Zm9ybS9xdC9Ta2lwcGVkCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0L1NraXBwZWQK
QEAgLTI1MzIsMjEgKzI1MzIsNyBAQCBmYXN0L3dvcmtlcnMvd29ya2VyLWNyYXNoLXdpdGgtaW52
YWxpZC1sb2NhdGlvbi5odG1sCiBmYXN0L3J1YnkvdGV4dC1kZWNvcmF0aW9uLWluLWRlc2NlbmRh
bnRzLXJ1YnkuaHRtbAogCiAjIEZhaWxpbmcgc2luY2UgaXNXb3JkVGV4dEJyZWFrIGlzIG5vdCBp
bXBsZW1lbnRlZC4gaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTgxMTM2
Ci1lZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHktY3Jhc2gtdGVzdC0xLmh0
bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnktd29yZC12aXN1YWxseS1jcmFzaC10ZXN0LTIu
aHRtbAotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5LWNyYXNoLXRlc3Qt
My5odG1sCi1lZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHktY3Jhc2gtdGVz
dC00Lmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnktd29yZC12aXN1YWxseS1jcmFzaC10
ZXN0LTUuaHRtbAotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5LWNyYXNo
LXRlc3QtdGV4dGFyZWEuaHRtbAotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3Vh
bGx5LWNyYXNoLXRlc3QtY3NzLWdlbmVyYXRlZC1jb250ZW50Lmh0bWwKLWVkaXRpbmcvc2VsZWN0
aW9uL21vdmUtYnktd29yZC12aXN1YWxseS1pbmxpbmUtYmxvY2stcG9zaXRpb25lZC1lbGVtZW50
Lmh0bWwKIGVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnktd29yZC12aXN1YWxseS1tdWx0aS1saW5l
Lmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUtYnktd29yZC12aXN1YWxseS1tdWx0aS1zcGFj
ZS5odG1sCi1lZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHktbnVsbC1ib3gu
aHRtbAotZWRpdGluZy9zZWxlY3Rpb24vbW92ZS1ieS13b3JkLXZpc3VhbGx5LXNpbmdsZS1zcGFj
ZS1pbmxpbmUtZWxlbWVudC5odG1sCi1lZGl0aW5nL3NlbGVjdGlvbi9tb3ZlLWJ5LXdvcmQtdmlz
dWFsbHktc2luZ2xlLXNwYWNlLW9uZS1lbGVtZW50Lmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21v
dmUtYnktd29yZC12aXN1YWxseS10ZXh0YXJlYS5odG1sCi1lZGl0aW5nL3NlbGVjdGlvbi9tb3Zl
LWJ5LXdvcmQtdmlzdWFsbHktd3JvbmctbGVmdC1yaWdodC5odG1sCiBlZGl0aW5nL3NlbGVjdGlv
bi9tb3ZlLWJ5LXdvcmQtdmlzdWFsbHktbWFjLmh0bWwKIAogIyBodHRwczovL2J1Z3Mud2Via2l0
Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODI2MjgK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>166937</attachid>
            <date>2012-10-03 12:20:25 -0700</date>
            <delta_ts>2012-10-09 12:14:25 -0700</delta_ts>
            <desc>Unskipping last tests</desc>
            <filename>patch.diff</filename>
            <type>text/plain</type>
            <size>1438</size>
            <attacher name="Tullio Lucena">tullio.lucena</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCA3NWFmZDUxLi5lMzcyOWY2IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0cy9DaGFuZ2VM
b2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTItMTAt
MDMgIFR1bGxpbyBMdWNlbmEgIDx0dWxsaW8ubHVjZW5hQG9wZW5ib3NzYS5vcmc+CisKKyAgICAg
ICAgTmVlZCBpbXBsZW1lbnQgaXNXb3JkVGV4dEJyZWFrIGZvciBRVCwgYW5kIFdpbkNFIGZvciB2
aXN1YWwgd29yZCBtb3ZlbWVudCBmdW5jdGlvbmFsaXR5CisgICAgICAgIGh0dHBzOi8vYnVncy53
ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTEzNgorCisgICAgICAgIFJldmlld2VkIGJ5IE5P
Qk9EWSAoT09QUyEpLgorCisgICAgICAgIFVza2lwcGluZyB0ZXN0cyB0aGF0IGFyZSBwYXNzaW5n
IGFmdGVyIHRoZSBtZXJnZSBodHRwczovL2dpdG9yaW91cy5vcmcvcXR3ZWJraXQvdGVzdGZvbnRz
L21lcmdlX3JlcXVlc3RzLzEKKworICAgICAgICAqIHBsYXRmb3JtL3F0L1Rlc3RFeHBlY3RhdGlv
bnM6CisKIDIwMTItMTAtMDMgIFJhcGhhZWwgS3VibyBkYSBDb3N0YSAgPHJhcGhhZWwua3Viby5k
YS5jb3N0YUBpbnRlbC5jb20+CiAKICAgICAgICAgW0VGTF0gUGl4ZWwgdGVzdCByZWJhc2VsaW5l
cyB0byBlbmFibGUgcGl4ZWwgdGVzdGluZyBvbiB0aGUgYm90cywgcGFydCAzLgpkaWZmIC0tZ2l0
IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQvVGVzdEV4cGVjdGF0aW9ucyBiL0xheW91dFRlc3Rz
L3BsYXRmb3JtL3F0L1Rlc3RFeHBlY3RhdGlvbnMKaW5kZXggZTI4MDYyOC4uZDA2ZDc0MCAxMDA2
NDQKLS0tIGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQvVGVzdEV4cGVjdGF0aW9ucworKysgYi9M
YXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9UZXN0RXhwZWN0YXRpb25zCkBAIC0yNTE3LDEwICsyNTE3
LDYgQEAgZmFzdC93b3JrZXJzL3dvcmtlci1jcmFzaC13aXRoLWludmFsaWQtbG9jYXRpb24uaHRt
bAogIyBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzEyNjYKIGZhc3Qv
cnVieS90ZXh0LWRlY29yYXRpb24taW4tZGVzY2VuZGFudHMtcnVieS5odG1sCiAKLSMgRmFpbGlu
ZyBzaW5jZSBpc1dvcmRUZXh0QnJlYWsgaXMgbm90IGltcGxlbWVudGVkLiBodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9ODExMzYKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUt
Ynktd29yZC12aXN1YWxseS1tdWx0aS1saW5lLmh0bWwKLWVkaXRpbmcvc2VsZWN0aW9uL21vdmUt
Ynktd29yZC12aXN1YWxseS1tYWMuaHRtbAotCiAjIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3No
b3dfYnVnLmNnaT9pZD04MjYyOAogc3ZnL2hpdHRlc3Qvc3ZnLWVsbGlwc2Utbm9uLXNjYWxlLXN0
cm9rZS54aHRtbAogCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>