Changeset 25353 in webkit


Ignore:
Timestamp:
Sep 4, 2007 7:50:03 AM (17 years ago)
Author:
antti
Message:

WebCore:

Reviewed by Maciej.


Fix <rdar://problem/5452112>
REGRESSION: Initially focused textfield on www.mac.com login page has no insertion point, and doesn't accept typed characters


Use updateLayoutIgnorePendingStylesheets() instead of updateLayout(). We need to have rendering to set input focus.

Test: fast/forms/focus-style-pending.html

  • dom/Element.cpp: (WebCore::Element::focus):

LayoutTests:

Reviewed by Maciej.


Test for <rdar://problem/5452112>
REGRESSION: Initially focused textfield on www.mac.com login page has no insertion point, and doesn't accept typed characters

  • fast/forms/focus-style-pending-expected.txt: Added.
  • fast/forms/focus-style-pending.html: Added.
Location:
trunk
Files:
2 added
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/LayoutTests/ChangeLog

    r25346 r25353  
     12007-09-04  Antti Koivisto  <antti@apple.com>
     2
     3        Reviewed by Maciej.
     4       
     5        Test for <rdar://problem/5452112>
     6        REGRESSION: Initially focused textfield on www.mac.com login page has no insertion point, and doesn't accept typed characters
     7
     8        * fast/forms/focus-style-pending-expected.txt: Added.
     9        * fast/forms/focus-style-pending.html: Added.
     10
    1112007-09-01  Darin Adler  <darin@apple.com>
    212

  • trunk/WebCore/ChangeLog

    r25352 r25353  
     12007-09-04  Antti Koivisto  <antti@apple.com>
     2
     3        Reviewed by Maciej.
     4       
     5        Fix <rdar://problem/5452112>
     6        REGRESSION: Initially focused textfield on www.mac.com login page has no insertion point, and doesn't accept typed characters
     7       
     8        Use updateLayoutIgnorePendingStylesheets() instead of updateLayout(). We need to have rendering to set input focus.
     9
     10        Test: fast/forms/focus-style-pending.html
     11
     12        * dom/Element.cpp:
     13        (WebCore::Element::focus):
     14
    1152007-09-04  Girish Ramakrishnan  <girish@trolltech.com>
    216

  • trunk/WebCore/dom/Element.cpp

    r25107 r25353  
    990990        return;
    991991
    992     doc->updateLayout();
     992    doc->updateLayoutIgnorePendingStylesheets();
    993993   
    994994    if (!supportsFocus())
Note: See TracChangeset for help on using the changeset viewer.