Changeset 96134 in webkit


Ignore:
Timestamp:
Sep 27, 2011 11:17:32 AM (13 years ago)
Author:
commit-queue@webkit.org
Message:

Autofocus on readonly inputs does not focus the element.
​https://bugs.webkit.org/show_bug.cgi?id=24092

Patch by Kaustubh Atrawalkar <Kaustubh Atrawalkar> on 2011-09-27
Reviewed by Ryosuke Niwa.

Source/WebCore:

Readonly input elements should be autofocusable. Removed the check.

Tests: fast/forms/autofocus-readonly-attribute.html

  • html/HTMLFormControlElement.cpp:

(WebCore::shouldAutofocus):

LayoutTests:

  • fast/forms/autofocus-readonly-attribute-expected.txt: Added.
  • fast/forms/autofocus-readonly-attribute.html: Added.
Location:
trunk
Files:
2 added
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r96132 r96134  
     12011-09-27  Kaustubh Atrawalkar  <kaustubh@motorola.com>
     2
     3        Autofocus on readonly inputs does not focus the element.
     4        https://bugs.webkit.org/show_bug.cgi?id=24092
     5
     6        Reviewed by Ryosuke Niwa.
     7
     8        * fast/forms/autofocus-readonly-attribute-expected.txt: Added.
     9        * fast/forms/autofocus-readonly-attribute.html: Added.
     10
    1112011-09-27  Mihai Parparita  <mihaip@chromium.org>
    212

  • trunk/Source/WebCore/ChangeLog

    r96130 r96134  
     12011-09-27  Kaustubh Atrawalkar  <kaustubh@motorola.com>
     2
     3        Autofocus on readonly inputs does not focus the element.
     4        https://bugs.webkit.org/show_bug.cgi?id=24092
     5
     6        Reviewed by Ryosuke Niwa.
     7
     8        Readonly input elements should be autofocusable. Removed the check.
     9
     10        Tests: fast/forms/autofocus-readonly-attribute.html
     11
     12        * html/HTMLFormControlElement.cpp:
     13        (WebCore::shouldAutofocus):
     14
    1152011-09-27  Simon Fraser  <simon.fraser@apple.com>
    216

  • trunk/Source/WebCore/html/HTMLFormControlElement.cpp

    r96078 r96134  
    129129    if (element->document()->ignoreAutofocus())
    130130        return false;
    131     if (element->isReadOnlyFormControl())
    132         return false;
    133131    if (element->hasAutofocused())
    134132        return false;
Note: See TracChangeset for help on using the changeset viewer.