WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
199490
Touching media controls sometimes shows software keyboard
https://bugs.webkit.org/show_bug.cgi?id=199490
Summary
Touching media controls sometimes shows software keyboard
Eric Carlson
Reported
2019-07-03 19:55:40 PDT
Touching media controls immediately after page load sometimes shows software keyboard.
Attachments
Patch
(1.63 KB, patch)
2019-07-03 19:59 PDT
,
Eric Carlson
no flags
Details
Formatted Diff
Diff
Patch
(19.19 KB, patch)
2019-07-04 15:41 PDT
,
Wenson Hsieh
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Eric Carlson
Comment 1
2019-07-03 19:55:52 PDT
<
rdar://problem/52076270
>
Eric Carlson
Comment 2
2019-07-03 19:59:49 PDT
Created
attachment 373450
[details]
Patch
Wenson Hsieh
Comment 3
2019-07-04 14:12:00 PDT
Comment on
attachment 373450
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=373450&action=review
> Source/WebKit/WebProcess/WebPage/WebPage.cpp:2899 > + if (oldFocusedElement == newFocusedElement && newFocusedElement->isContentEditable())
I got a chance to debug this a bit — a few observations… 1. newFocusedElement in this case is <INPUT id='' type='text' class='active’> when tapping on the media controls. 2. It looks like this change would make the bug go away by preventing the <INPUT id='' type='text' class='active’> from being refocused. This is because the input element itself is not content editable, but rather, is a text form control (which contains an editable area).
Wenson Hsieh
Comment 4
2019-07-04 15:41:55 PDT
Created
attachment 373473
[details]
Patch
Eric Carlson
Comment 5
2019-07-05 08:48:13 PDT
Thanks Wenson!
WebKit Commit Bot
Comment 6
2019-07-05 09:22:49 PDT
Comment on
attachment 373473
[details]
Patch Clearing flags on attachment: 373473 Committed
r247158
: <
https://trac.webkit.org/changeset/247158
>
WebKit Commit Bot
Comment 7
2019-07-05 09:22:50 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug