<?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>23789</bug_id>
          
          <creation_ts>2009-02-06 10:29:13 -0800</creation_ts>
          <short_desc>REGRESSION: Changing disabled state of a textarea does not do a repaint</short_desc>
          <delta_ts>2011-12-08 16:59:08 -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>Layout and Rendering</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc>http://dump.kkaefer.com/disabled.html</bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P1</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Konstantin Käfer">kkaefer</reporter>
          <assigned_to name="Julien Chaffraix">jchaffraix</assigned_to>
          <cc>bdakin</cc>
    
    <cc>jchaffraix</cc>
    
    <cc>ojan</cc>
    
    <cc>priyajeet.hora</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>108653</commentid>
    <comment_count>0</comment_count>
    <who name="Konstantin Käfer">kkaefer</who>
    <bug_when>2009-02-06 10:29:13 -0800</bug_when>
    <thetext>In the most recent nightly build, changing the disabled state of a textarea does not do a repaint, thus, the textarea still appears as before even though the status has changed. As soon as the user (e.g.) resizes it, it&apos;s repainted and the status is reflected.

This worked in Safari 3.1.

This behavior seems to be broken on textareas. I tested this checkboxes and textfields (input type=text) and it seemed to work correctly for those, but not for textarea.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>108655</commentid>
    <comment_count>1</comment_count>
    <who name="Konstantin Käfer">kkaefer</who>
    <bug_when>2009-02-06 10:30:06 -0800</bug_when>
    <thetext>URL: http://dump.kkaefer.com/disabled.html

(This is not necessarily restricted to Mac OS X 10.5, but I can only confirm it for 10.5)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>108656</commentid>
    <comment_count>2</comment_count>
    <who name="Konstantin Käfer">kkaefer</who>
    <bug_when>2009-02-06 10:32:01 -0800</bug_when>
    <thetext>You might not see the effect when clicking on the link for the first time. After you reload the page, the bug should appear.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>108657</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-02-06 10:33:29 -0800</bug_when>
    <thetext>This seems to be a recent regression.  ToT fails his test case, but my unreleased Safari build from several months ago does not.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>517527</commentid>
    <comment_count>4</comment_count>
    <who name="Julien Chaffraix">jchaffraix</who>
    <bug_when>2011-12-08 10:30:32 -0800</bug_when>
    <thetext>The bug does not reproduce on ToT (not sure when it was solved). I haven&apos;t found out a repaint test for this case so I will turn the original test case into one and land it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>517787</commentid>
    <comment_count>5</comment_count>
      <attachid>118469</attachid>
    <who name="Julien Chaffraix">jchaffraix</who>
    <bug_when>2011-12-08 14:21:28 -0800</bug_when>
    <thetext>Created attachment 118469
Proposed change: land a repaint test covering the existing test case.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>517793</commentid>
    <comment_count>6</comment_count>
    <who name="Konstantin Käfer">kkaefer</who>
    <bug_when>2011-12-08 14:25:34 -0800</bug_when>
    <thetext>I can confirm that the original bug I reported is fixed in current builds.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>517928</commentid>
    <comment_count>7</comment_count>
      <attachid>118469</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-12-08 16:59:04 -0800</bug_when>
    <thetext>Comment on attachment 118469
Proposed change: land a repaint test covering the existing test case.

Clearing flags on attachment: 118469

Committed r102399: &lt;http://trac.webkit.org/changeset/102399&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>517929</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2011-12-08 16:59:08 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>118469</attachid>
            <date>2011-12-08 14:21:28 -0800</date>
            <delta_ts>2011-12-08 16:59:04 -0800</delta_ts>
            <desc>Proposed change: land a repaint test covering the existing test case.</desc>
            <filename>bug-23789-20111208142127.patch</filename>
            <type>text/plain</type>
            <size>2793</size>
            <attacher name="Julien Chaffraix">jchaffraix</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTAyMjc1CmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9DaGFu
Z2VMb2cgYi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKaW5kZXggNDRiYzQwYmYyMzA0NTBmN2E4ODg2
OWRjZmZhZWVhM2Y2NDY3N2E3YS4uZDVlZGI3ZDhhOGJhNWZlYTE3MmM4YTA2MmMzNzFiYmNkNWYz
OGQ2NyAxMDA3NTUKLS0tIGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCisrKyBiL0xheW91dFRlc3Rz
L0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDExLTEyLTA3ICBKdWxpZW4gQ2hhZmZyYWl4
ICA8amNoYWZmcmFpeEB3ZWJraXQub3JnPgorCisgICAgICAgIFJFR1JFU1NJT046IENoYW5naW5n
IGRpc2FibGVkIHN0YXRlIG9mIGEgdGV4dGFyZWEgZG9lcyBub3QgZG8gYSByZXBhaW50CisgICAg
ICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yMzc4OQorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFRoZSBidWcgZ290IGZp
eGVkIChJIGRpZCBub3QgYmlzZWN0IHRvIGZpbmQgb3V0IHdoZW4pIHRodXMgbGFuZGluZworICAg
ICAgICB0aGUgdGVzdCBjYXNlIGNvbnZlcnRlZCB0byBhIHJlcGFpbnQgdGVzdCBhbmQgbWFkZSBj
cm9zcy1wbGF0Zm9ybS4KKworICAgICAgICAqIGZhc3QvcmVwYWludC90ZXh0YXJlYS1zZXQtZGlz
YWJsZWQtZXhwZWN0ZWQucG5nOiBBZGRlZC4KKyAgICAgICAgKiBmYXN0L3JlcGFpbnQvdGV4dGFy
ZWEtc2V0LWRpc2FibGVkLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9yZXBh
aW50L3RleHRhcmVhLXNldC1kaXNhYmxlZC5odG1sOiBBZGRlZC4KKwogMjAxMS0xMi0wNyAgQWRh
bSBLbGVpbiAgPGFkYW1rQGNocm9taXVtLm9yZz4KIAogICAgICAgICBbTXV0YXRpb25PYnNlcnZl
cnNdIEVuYWJsZSBpbiBDaHJvbWl1bSB0cnVuawpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFz
dC9yZXBhaW50L3RleHRhcmVhLXNldC1kaXNhYmxlZC1leHBlY3RlZC5wbmcgYi9MYXlvdXRUZXN0
cy9mYXN0L3JlcGFpbnQvdGV4dGFyZWEtc2V0LWRpc2FibGVkLWV4cGVjdGVkLnBuZwpuZXcgZmls
ZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAw
MDAwLi4yNzE0MTEwOTViOTU3OWNmMjVkYTU1YzM2MDI1ZDI2ZjFmZDFiOWFiCkdJVCBiaW5hcnkg
cGF0Y2gKbGl0ZXJhbCAyODkwCnpjbWVBU0BOPyhvbEh5YHVWQnEhaWEweX55VXsrdllWMmE+aTAq
Wik9aF5obEElQFdzJGxIYG9rPG19P29UbjV3Ugp6UkM1Q31PR0MzbmdIJWhpPGRqcjFiNHpwTSMx
ejljcWcyQ1dPWEhtZSlfV0tqeGJBcW1JRUdacmQzJEZnWj9sNjQKenRLKWBAI3h1M0N5PGVYOHNL
cWUkZyg9S1V6dkxhYTxJP216Kjw+OUNoTFNqY2VnPWpheSQ2XldJeWlNMDhmSVdICnpaal5tM2tV
MmlyVEozU1VBO1hYSz41V0A7UEZMM0QtOFAoRFRXQGQ2JFdTcCFTZHFjT0N4TWFTMiNlUyNXOVNE
JAp6JmhMLWctb0wrSCM9eml7RVNxZXpteXNicyVkZHJqIUxYNSlLfG5aU2xyYjZ6cWJYckQ3bVZo
RChPZldeM3IyR1IKZDlkcDQoe1VmdEBTKzYhYm1qV0JkNDQkcmpGNioyVW5nQiNDJnE0cUkKCmxp
dGVyYWwgMApIY21WP2QwMDAwMQoKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zhc3QvcmVwYWlu
dC90ZXh0YXJlYS1zZXQtZGlzYWJsZWQtZXhwZWN0ZWQudHh0IGIvTGF5b3V0VGVzdHMvZmFzdC9y
ZXBhaW50L3RleHRhcmVhLXNldC1kaXNhYmxlZC1leHBlY3RlZC50eHQKbmV3IGZpbGUgbW9kZSAx
MDA2NDQKaW5kZXggMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMC4uOGIx
Mzc4OTE3OTFmZTk2OTI3YWQ3OGU2NGIwYWFkN2JkZWQwOGJkYwotLS0gL2Rldi9udWxsCisrKyBi
L0xheW91dFRlc3RzL2Zhc3QvcmVwYWludC90ZXh0YXJlYS1zZXQtZGlzYWJsZWQtZXhwZWN0ZWQu
dHh0CkBAIC0wLDAgKzEgQEAKKwpkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvZmFzdC9yZXBhaW50
L3RleHRhcmVhLXNldC1kaXNhYmxlZC5odG1sIGIvTGF5b3V0VGVzdHMvZmFzdC9yZXBhaW50L3Rl
eHRhcmVhLXNldC1kaXNhYmxlZC5odG1sCm5ldyBmaWxlIG1vZGUgMTAwNzU1CmluZGV4IDAwMDAw
MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAuLjAwNWI1NDM5OTY5NDRiOGEyNmEy
MTM4MTAxODE0ZDk3ZmY1NTU0N2IKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRUZXN0cy9mYXN0
L3JlcGFpbnQvdGV4dGFyZWEtc2V0LWRpc2FibGVkLmh0bWwKQEAgLTAsMCArMSwyMCBAQAorPCFE
T0NUWVBFIGh0bWw+Cis8aHRtbD4KKzxoZWFkPgorICAgIDxzY3JpcHQgc3JjPSJyZXNvdXJjZXMv
cmVwYWludC5qcyIgdHlwZT0idGV4dC9qYXZhc2NyaXB0Ij48L3NjcmlwdD4KKyAgICA8c2NyaXB0
PgorICAgICAgICBpZiAod2luZG93LmxheW91dFRlc3RDb250cm9sbGVyKQorICAgICAgICAgICAg
bGF5b3V0VGVzdENvbnRyb2xsZXIuZHVtcEFzVGV4dCh0cnVlKTsKKworICAgICAgICBmdW5jdGlv
biByZXBhaW50VGVzdCgpCisgICAgICAgIHsKKyAgICAgICAgICAgIGRvY3VtZW50LmdldEVsZW1l
bnRzQnlUYWdOYW1lKCd0ZXh0YXJlYScpWzBdLmRpc2FibGVkID0gdHJ1ZTsKKyAgICAgICAgfQor
ICAgIDwvc2NyaXB0PgorPC9oZWFkPgorPGJvZHkgb25sb2FkPSJydW5SZXBhaW50VGVzdCgpIj4K
Kzxmb3JtPgorICA8dGV4dGFyZWEgc3R5bGU9ImZvbnQ6IDEwcHggQWhlbTsgd2lkdGg6IDEwMHB4
OyBoZWlnaHQ6IDEwMHB4OyI+PC90ZXh0YXJlYT4KKzwvZm9ybT4KKzwvYm9keT4KKzwvaHRtbD4K
</data>

          </attachment>
      

    </bug>

</bugzilla>