WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
99778
Automatically start plugins created within a user gesture, skipping snapshotting
https://bugs.webkit.org/show_bug.cgi?id=99778
Summary
Automatically start plugins created within a user gesture, skipping snapshotting
Kiran Muppala
Reported
2012-10-18 17:13:54 PDT
When plugin snapshotting is enabled, a snapshot of the plugin is displayed until the user clicks on the plugin. However, on some sites such as cnn.com, use an image element as a placeholder for a video and replace the image with the video when the user clicks on it. In such instances, plugin snapshotting results in the user having to click twice to watch the video. This is undesirable. Hence, plugins should start automatically when created within a user gesture, even if snapshotting is enabled.
Attachments
Patch
(1.72 KB, patch)
2012-10-18 17:25 PDT
,
Kiran Muppala
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Kiran Muppala
Comment 1
2012-10-18 17:25:49 PDT
Created
attachment 169512
[details]
Patch
Kiran Muppala
Comment 2
2012-10-18 17:26:19 PDT
<
rdar://problem/12482210
>
Jon Lee
Comment 3
2012-10-18 17:44:28 PDT
Have you been able to test this? I am not a reviewer, but I give it an unofficial r+.
Kiran Muppala
Comment 4
2012-10-18 17:47:39 PDT
(In reply to
comment #3
)
> Have you been able to test this? > > I am not a reviewer, but I give it an unofficial r+.
Yes, I tested using cnn.com, which tends to have a "Click to Play" image element in the center, like you showed before. The video started playing right after I clicked the image, without the interim plugin snapshot.
WebKit Review Bot
Comment 5
2012-10-18 22:36:10 PDT
Comment on
attachment 169512
[details]
Patch Clearing flags on attachment: 169512 Committed
r131857
: <
http://trac.webkit.org/changeset/131857
>
WebKit Review Bot
Comment 6
2012-10-18 22:36:14 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