Bug 197632

Summary: [iOS] Unable to commit search on MSN.com, qq.com, or sina.com.cn using enter key (hardware or software keyboard)
Product: WebKit Reporter: Daniel Bates <dbates>
Component: WebKit Misc.Assignee: Daniel Bates <dbates>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, ews-watchlist, webkit-bug-importer, wenson_hsieh
Priority: P2 Keywords: InRadar, PlatformOnly
Version: WebKit Local Build   
Hardware: iPhone / iPad   
OS: iOS 12   
URL: http://www.msn.com
See Also: https://bugs.webkit.org/show_bug.cgi?id=197721
Bug Depends on: 197710    
Bug Blocks: 190571    
Attachments:
Description Flags
Just to see...
none
Archive of layout-test-results from ews214 for win-future
none
Patch
none
Patch
none
Patch
none
Archive of layout-test-results from ews215 for win-future
none
Patch none

Description Daniel Bates 2019-05-06 15:53:06 PDT
Seen on iPad using hardware keyboard.

Steps to reproduce:

Ensure iPad is in landscape orientation. (That's right, only happens in landscape)

1. Visit <http://www.msn.com> in landscape orientation.
2. Type something in the search field.
3. Press the Return key on the keyboard.

Nothing happens, but it should open a new window with search results.
Comment 1 Daniel Bates 2019-05-06 15:53:18 PDT
<rdar://problem/47902054>
Comment 2 Daniel Bates 2019-05-06 15:55:59 PDT
Created attachment 369189 [details]
Just to see...
Comment 3 EWS Watchlist 2019-05-07 05:41:18 PDT
Comment on attachment 369189 [details]
Just to see...

Attachment 369189 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12122697

New failing tests:
js/dom/custom-constructors.html
Comment 4 EWS Watchlist 2019-05-07 05:41:20 PDT
Created attachment 369277 [details]
Archive of layout-test-results from ews214 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews214  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 5 Daniel Bates 2019-05-08 14:46:58 PDT
Hack works well, but leaves a yucky feeling. So, going to try to fix this another way.
Comment 6 Daniel Bates 2019-05-08 15:10:18 PDT
Created attachment 369428 [details]
Patch
Comment 7 Brent Fulgham 2019-05-08 15:21:31 PDT
Comment on attachment 369428 [details]
Patch

I think this looks good, but the patch doesn't apply to trunk. Could you please rebase it and upload again?
Comment 8 Daniel Bates 2019-05-08 16:14:09 PDT
Created attachment 369441 [details]
Patch
Comment 9 Daniel Bates 2019-05-08 16:33:29 PDT
Created attachment 369446 [details]
Patch
Comment 10 EWS Watchlist 2019-05-08 22:34:54 PDT
Comment on attachment 369446 [details]
Patch

Attachment 369446 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12140332

New failing tests:
http/tests/css/filters-on-iframes.html
Comment 11 EWS Watchlist 2019-05-08 22:34:56 PDT
Created attachment 369464 [details]
Archive of layout-test-results from ews215 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews215  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit
Comment 12 Daniel Bates 2019-05-09 10:36:04 PDT
Created attachment 369497 [details]
Patch
Comment 13 Daniel Bates 2019-05-09 10:41:26 PDT
Committed r245144: <https://trac.webkit.org/changeset/245144>
Comment 14 Daniel Bates 2019-05-16 10:31:57 PDT
Added missing file to make the test work in <https://trac.webkit.org/changeset/245399> and fixed up the ChangeLog entry in <https://trac.webkit.org/changeset/245400>.