<?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>59032</bug_id>
          
          <creation_ts>2011-04-20 16:18:15 -0700</creation_ts>
          <short_desc>[Mac]Regression: extending word that is already marked misspelled doesn&apos;t remove misspelling underline.</short_desc>
          <delta_ts>2011-04-20 18:28:51 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.6</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jia Pu">jiapu.mail</reporter>
          <assigned_to name="Jia Pu">jiapu.mail</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>ddavidso</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>389604</commentid>
    <comment_count>0</comment_count>
    <who name="Jia Pu">jiapu.mail</who>
    <bug_when>2011-04-20 16:18:15 -0700</bug_when>
    <thetext>If the word &quot;brough&quot; is already marked as misspelled, appending &quot;t&quot; at the end doesn&apos;t remove the misspelling marker. 

This is a regression introduced in changeset 83344. In Editor.cpp, &quot;removeSpellAndCorrectionMarkersFromWordsToBeEdited(isSpaceOrNewline(text[0])); &quot; is replaced &quot;updateMarkersForWordsAffectedByEditing(text[0]);&quot;, and the the call to isSpaceOrNewline() is missing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>389652</commentid>
    <comment_count>1</comment_count>
    <who name="Jia Pu">jiapu.mail</who>
    <bug_when>2011-04-20 17:03:16 -0700</bug_when>
    <thetext>&lt;rdar://problem/9312444&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>389662</commentid>
    <comment_count>2</comment_count>
      <attachid>90452</attachid>
    <who name="Jia Pu">jiapu.mail</who>
    <bug_when>2011-04-20 17:16:48 -0700</bug_when>
    <thetext>Created attachment 90452
Patch (v1)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>389684</commentid>
    <comment_count>3</comment_count>
      <attachid>90452</attachid>
    <who name="Maciej Stachowiak">mjs</who>
    <bug_when>2011-04-20 17:32:59 -0700</bug_when>
    <thetext>Comment on attachment 90452
Patch (v1)

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>389722</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-04-20 18:26:29 -0700</bug_when>
    <thetext>The commit-queue encountered the following flaky tests while processing attachment 90452:

animations/suspend-resume-animation.html bug 48161 (author: cmarrin@apple.com)
The commit-queue is continuing to process your patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>389725</commentid>
    <comment_count>5</comment_count>
      <attachid>90452</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-04-20 18:28:46 -0700</bug_when>
    <thetext>Comment on attachment 90452
Patch (v1)

Clearing flags on attachment: 90452

Committed r84448: &lt;http://trac.webkit.org/changeset/84448&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>389726</commentid>
    <comment_count>6</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2011-04-20 18:28:51 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>90452</attachid>
            <date>2011-04-20 17:16:48 -0700</date>
            <delta_ts>2011-04-20 18:28:46 -0700</delta_ts>
            <desc>Patch (v1)</desc>
            <filename>bug-59032-20110420171647.patch</filename>
            <type>text/plain</type>
            <size>3556</size>
            <attacher name="Jia Pu">jiapu.mail</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogODQyOTcKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCAyNTQxZDAwZTlhYjBiNGNl
YzRlZDMzZGU1MjM1OTYyZDUzZTc1ODMyLi5iMjY5ZjhjMzhlNGRiNzVkNTJkNmIyYjEwNGQxMjNi
MGQ0MWY2NjgwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291
cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTkgQEAKKzIwMTEtMDQtMjAgIEppYSBQ
dSAgPGpwdUBhcHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgW01hY11SZWdyZXNzaW9uOiBleHRlbmRpbmcgd29yZCB0aGF0IGlzIGFscmVh
ZHkgbWFya2VkIG1pc3NwZWxsZWQgZG9lc24ndCByZW1vdmUgbWlzc3BlbGxpbmcgdW5kZXJsaW5l
LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTkwMzIK
KyAgICAgICAgPHJkYXI6Ly9wcm9ibGVtLzkzMTI0NDQ+CisKKyAgICAgICAgVGhpcyBmaXhlcyBh
IHJlZ3Jlc3Npb24gaW50cm9kdWNlZCBpbiByZWNlbnQgcmVmYWN0b3Jpbmcgd29yayBpbiBFZGl0
b3IuIFNlZSBidWcgd2VicGFnZSBmb3IgZGV0YWlsLgorCisgICAgICAgIG1hbnVhbCB0ZXN0OiBT
b3VyY2UvV2ViQ29yZS9tYW51YWwtdGVzdHMvYXV0b2NvcnJlY3Rpb24vcmVtb3ZlLW1pc3NwZWxs
aW5nLW1hcmtlci1hZnRlci1hcHBlbmRpbmctbGV0dGVyLmh0bWwKKworICAgICAgICAqIGVkaXRp
bmcvRWRpdG9yLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkVkaXRvcjo6aW5zZXJ0VGV4dFdpdGhv
dXRTZW5kaW5nVGV4dEV2ZW50KToKKyAgICAgICAgKiBtYW51YWwtdGVzdHMvYXV0b2NvcnJlY3Rp
b24vcmVtb3ZlLW1pc3NwZWxsaW5nLW1hcmtlci1hZnRlci1hcHBlbmRpbmctbGV0dGVyLmh0bWw6
IEFkZGVkLgorCiAyMDExLTA0LTE5ICBCZXRoIERha2luICA8YmRha2luQGFwcGxlLmNvbT4KIAog
ICAgICAgICBSZXZpZXdlZCBieSBNYWNpZWogU3RhY2hvd2lhay4KZGlmZiAtLWdpdCBhL1NvdXJj
ZS9XZWJDb3JlL2VkaXRpbmcvRWRpdG9yLmNwcCBiL1NvdXJjZS9XZWJDb3JlL2VkaXRpbmcvRWRp
dG9yLmNwcAppbmRleCBhMWRmNzRlY2E4MGJkYTYxYmY0NmIzN2M5OGNmMjU1OGFiNWJjOTc4Li5m
ZDVhZjJlMGI5MzI4NDdjY2UzMzRlMWE4NGFiYTI5OTY3OGI3Njc2IDEwMDY0NAotLS0gYS9Tb3Vy
Y2UvV2ViQ29yZS9lZGl0aW5nL0VkaXRvci5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvZWRpdGlu
Zy9FZGl0b3IuY3BwCkBAIC0xMDkyLDcgKzEwOTIsNyBAQCBib29sIEVkaXRvcjo6aW5zZXJ0VGV4
dFdpdGhvdXRTZW5kaW5nVGV4dEV2ZW50KGNvbnN0IFN0cmluZyYgdGV4dCwgYm9vbCBzZWxlY3RJ
bgogICAgICAgICByZXR1cm4gdHJ1ZTsKIAogICAgIGlmICghdGV4dC5pc0VtcHR5KCkpCi0gICAg
ICAgIHVwZGF0ZU1hcmtlcnNGb3JXb3Jkc0FmZmVjdGVkQnlFZGl0aW5nKHRleHRbMF0pOworICAg
ICAgICB1cGRhdGVNYXJrZXJzRm9yV29yZHNBZmZlY3RlZEJ5RWRpdGluZyhpc1NwYWNlT3JOZXds
aW5lKHRleHRbMF0pKTsKIAogICAgIGJvb2wgc2hvdWxkQ29uc2lkZXJBcHBseWluZ0F1dG9jb3Jy
ZWN0aW9uID0gZmFsc2U7CiAgICAgaWYgKHRleHQgPT0gIiAiIHx8IHRleHQgPT0gIlx0IikKZGlm
ZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL21hbnVhbC10ZXN0cy9hdXRvY29ycmVjdGlvbi9yZW1v
dmUtbWlzc3BlbGxpbmctbWFya2VyLWFmdGVyLWFwcGVuZGluZy1sZXR0ZXIuaHRtbCBiL1NvdXJj
ZS9XZWJDb3JlL21hbnVhbC10ZXN0cy9hdXRvY29ycmVjdGlvbi9yZW1vdmUtbWlzc3BlbGxpbmct
bWFya2VyLWFmdGVyLWFwcGVuZGluZy1sZXR0ZXIuaHRtbApuZXcgZmlsZSBtb2RlIDEwMDY0NApp
bmRleCAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwLi5mMWM1MzUwNGMx
NDkxMmNkMjNmMTZhZjZmMDdhY2VkYThkZTQ5OTIxCi0tLSAvZGV2L251bGwKKysrIGIvU291cmNl
L1dlYkNvcmUvbWFudWFsLXRlc3RzL2F1dG9jb3JyZWN0aW9uL3JlbW92ZS1taXNzcGVsbGluZy1t
YXJrZXItYWZ0ZXItYXBwZW5kaW5nLWxldHRlci5odG1sCkBAIC0wLDAgKzEsNDUgQEAKKzxodG1s
PiAKKzxoZWFkPgorCis8c3R5bGU+CisuZWRpdGluZyB7IAorICAgIGJvcmRlcjogMnB4IHNvbGlk
IHJlZDsgCisgICAgcGFkZGluZzogMTJweDsgCisgICAgZm9udC1zaXplOiAyNHB4OyAKK30KKzwv
c3R5bGU+Cis8c2NyaXB0IHNyYz0uLi8uLi8uLi8uLi9MYXlvdXRUZXN0cy9lZGl0aW5nL2VkaXRp
bmcuanMgbGFuZ3VhZ2U9IkphdmFTY3JpcHQiIHR5cGU9InRleHQvSmF2YVNjcmlwdCIgPjwvc2Ny
aXB0PgorCis8c2NyaXB0PgorCitmdW5jdGlvbiBlZGl0aW5nVGVzdCgpIHsKKyAgICB0ZXh0YXJl
YSA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCd0ZXN0Jyk7CisgICAgdGV4dGFyZWEuZm9jdXMo
KTsKKyAgICBkb2N1bWVudC5leGVjQ29tbWFuZCgiSW5zZXJ0VGV4dCIsIGZhbHNlLCAidGhpcyBi
cm91Z3QiKTsKKyAgICB0eXBlQ2hhcmFjdGVyQ29tbWFuZCgnICcpOworICAgIGRvY3VtZW50LmV4
ZWNDb21tYW5kKCJEZWxldGUiKTsKKyAgICBkb2N1bWVudC5leGVjQ29tbWFuZCgiRGVsZXRlIik7
CisgICAgdHlwZUNoYXJhY3RlckNvbW1hbmQoJyAnKTsKKyAgICAKKyAgICAvLyBOb3cgdGhlIHdv
cmQgImJyb3VnaCIgc2hvdWxkIGNhcnJ5IG1pc3NwZWxsaW5nIG1hcmtlci4KKyAgICAKKyAgICBk
b2N1bWVudC5leGVjQ29tbWFuZCgiRGVsZXRlIik7Cit9CisKKzwvc2NyaXB0PgorCis8dGl0bGU+
UmVtb3ZpbmcgbWlzc3BlbGxpbmcgbWFya2VyIGFmdGVyIGFwcGVuZGluZyBsZXR0ZXIgdGVzdC48
L3RpdGxlPiAKKzwvaGVhZD4gCis8Ym9keT4KKzxkaXY+VGhlIHRlc3QgdmVyaWZpZWQgdGhhdCBt
aXNzcGVsbGluZyBtYXJrZXIgaXMgcmVtb3ZlZCBhZnRlciBhcHBlbmRpbmcgbmV3IGxldHRlciB0
byBtaXNzcGVsbGVkIHdvcmQuIFlvdSBzaG91bGQgc2VlIHRoZSBwaHJhc2UgInRoaXMgYnJvdWdo
dCIgd2l0aG91dCByZWQgdW5kZXJsaW5lLjwvZGl2PgorPHAgIHN0eWxlPSJjb2xvcjpncmVlbiI+
Tm90ZSwgdGhpcyB0ZXN0IGNhbiBmYWlsIGR1ZSB0byB1c2VyIHNwZWNpZmljIHNwZWxsIGNoZWNr
aW5nIGRhdGEuIElmIHRoZSB1c2VyIGhhcyBwcmV2aW91c2x5IGRpc21pc3NlZCAnYnJvdWdodCcg
YXMgdGhlIGNvcnJlY3Qgc3BlbGxpbmcgb2YgJ2Jyb3VndGgnIHNldmVyYWwgdGltZXMsIHRoZSBz
cGVsbCBjaGVja2VyIHdpbGwgbm90IHByb3ZpZGUgJ25vdGF0aW9uYWwnIGFzIGEgc3VnZ2VzdGlv
biBhbnltb3JlLiBUbyBmaXggdGhpcywgcmVtb3ZlIGFsbCBmaWxlcyBpbiB+L0xpYnJhcnkvU3Bl
bGxpbmcuPC9wPgorPHRleHRhcmVhIGlkPSJ0ZXN0IiBjb2xzPSI4MCIgcm93cz0iMTAiPjwvdGV4
dGFyZWE+Cis8L2Rpdj4KKworCis8c2NyaXB0PgorcnVuRWRpdGluZ1Rlc3QoKTsKKzwvc2NyaXB0
PgorCis8L2JvZHk+Cis8L2h0bWw+Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>