WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
161833
Apple.com keynote does not display media controls
https://bugs.webkit.org/show_bug.cgi?id=161833
Summary
Apple.com keynote does not display media controls
Wenson Hsieh
Reported
2016-09-09 19:33:36 PDT
Apple.com keynote does not display media controls
Attachments
Patch
(14.10 KB, patch)
2016-09-09 19:47 PDT
,
Wenson Hsieh
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Wenson Hsieh
Comment 1
2016-09-09 19:34:00 PDT
<
rdar://problem/28230123
>
Wenson Hsieh
Comment 2
2016-09-09 19:47:48 PDT
Created
attachment 288478
[details]
Patch
Tim Horton
Comment 3
2016-09-09 23:44:00 PDT
Comment on
attachment 288478
[details]
Patch It's not totally clear to me why we'd want the two heuristics to be different (why we wouldn't loosen both), but this seems like a safer fix for now (guessing it is about risk), I guess. Can you file a follow up bug about maybe reunifying in the future, or is there a good reason we wouldn't want to do that?
Wenson Hsieh
Comment 4
2016-09-10 00:58:48 PDT
(In reply to
comment #3
)
> Comment on
attachment 288478
[details]
> Patch > > It's not totally clear to me why we'd want the two heuristics to be > different (why we wouldn't loosen both), but this seems like a safer fix for > now (guessing it is about risk), I guess. Can you file a follow up bug about > maybe reunifying in the future, or is there a good reason we wouldn't want > to do that?
It's primarily about mitigating risk. For other reasons included in the radar though, I think there's some argument to be made for keeping the heuristic slightly different in the two cases.
WebKit Commit Bot
Comment 5
2016-09-10 01:21:13 PDT
Comment on
attachment 288478
[details]
Patch Clearing flags on attachment: 288478 Committed
r205784
: <
http://trac.webkit.org/changeset/205784
>
WebKit Commit Bot
Comment 6
2016-09-10 01:21:17 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