Bug 131949

Summary: REGRESSION (r157328): popover to check into flight ba.com dismisses instantly when focusing form
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: UI EventsAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, benjamin, buildbot, commit-queue, darin, esprehn+autocc, kangil.han, kling, koivisto, rniwa, sam
Priority: P2 Keywords: Regression
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 122680    
Bug Blocks:    
Attachments:
Description Flags
Fixes the bug
none
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2
none
Patch for landing
none
Patch for landing none

Description Ryosuke Niwa 2014-04-21 13:43:38 PDT
Reproduction steps:
1. Go to: <http://www.britishairways.com/travel/home/public/en_us>
2. Mouse over "Manage My Bookin".
3. Try to enter in the fields for "Online check-in" or "Find my booking". 

Results:
It is impossible enter in those fields, the pop-up UI disappears before you start typing. Please see the attached screen capture video.
Comment 1 Ryosuke Niwa 2014-04-21 22:20:50 PDT
Created attachment 229863 [details]
Fixes the bug
Comment 2 Build Bot 2014-04-21 23:12:35 PDT
Comment on attachment 229863 [details]
Fixes the bug

Attachment 229863 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6027582352916480

New failing tests:
fast/events/shadow-event-path.html
Comment 3 Build Bot 2014-04-21 23:12:39 PDT
Created attachment 229864 [details]
Archive of layout-test-results from webkit-ews-14 for mac-mountainlion-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-14  Port: mac-mountainlion-wk2  Platform: Mac OS X 10.8.5
Comment 4 Darin Adler 2014-04-22 07:38:18 PDT
Comment on attachment 229863 [details]
Fixes the bug

View in context: https://bugs.webkit.org/attachment.cgi?id=229863&action=review

Patch seems fine, but I find the indirect way all this code is written really confusing.

The new test seems to be failing on EWS, so please check to be sure that’s resolved before landing.

> Source/WebCore/dom/EventDispatcher.cpp:180
> +        Node* previousHost = 0;

nullptr

> Source/WebCore/dom/EventDispatcher.cpp:192
> +        return 0;

nullptr
Comment 5 Ryosuke Niwa 2014-04-22 15:45:51 PDT
Created attachment 229916 [details]
Patch for landing
Comment 6 Ryosuke Niwa 2014-04-22 15:50:27 PDT
Created attachment 229917 [details]
Patch for landing
Comment 7 WebKit Commit Bot 2014-04-22 16:26:26 PDT
Comment on attachment 229917 [details]
Patch for landing

Clearing flags on attachment: 229917

Committed r167689: <http://trac.webkit.org/changeset/167689>
Comment 8 WebKit Commit Bot 2014-04-22 16:26:31 PDT
All reviewed patches have been landed.  Closing bug.
Comment 9 Ryosuke Niwa 2014-04-23 21:25:05 PDT
Committed r167744: <http://trac.webkit.org/changeset/167744>
Comment 10 Alexey Proskuryakov 2014-04-25 16:21:41 PDT
This caused bug 132214.
Comment 11 Alexey Proskuryakov 2014-04-25 16:29:50 PDT
(and apparently many more bugs)
Comment 12 Darin Adler 2014-04-27 10:22:10 PDT
(In reply to comment #11)
> (and apparently many more bugs)

Where can I find out more about the “many more”?