RESOLVED FIXED 117969
Don't snapshot plug-ins that come from the same host as the page
https://bugs.webkit.org/show_bug.cgi?id=117969
Summary Don't snapshot plug-ins that come from the same host as the page
Dean Jackson
Reported 2013-06-24 20:38:20 PDT
Typically if a page has a plug-in that is served from the same host, it's content that is important to the site. Make sure we don't snapshot that content. <rdar://problem/12885400>
Attachments
Patch (1.99 KB, patch)
2013-06-24 20:41 PDT, Dean Jackson
thorton: review+
buildbot: commit-queue-
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion (632.08 KB, application/zip)
2013-06-25 00:44 PDT, Build Bot
no flags
Dean Jackson
Comment 1 2013-06-24 20:41:09 PDT
Jon Lee
Comment 2 2013-06-24 21:10:23 PDT
Comment on attachment 205359 [details] Patch LGTM. r=me
Build Bot
Comment 3 2013-06-25 00:43:59 PDT
Comment on attachment 205359 [details] Patch Attachment 205359 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/960850 New failing tests: svg/batik/filters/feTile.svg
Build Bot
Comment 4 2013-06-25 00:44:00 PDT
Created attachment 205370 [details] Archive of layout-test-results from webkit-ews-06 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-06 Port: mac-mountainlion Platform: Mac OS X 10.8.3
Tim Horton
Comment 5 2013-06-25 12:52:07 PDT
Comment on attachment 205359 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=205359&action=review r+ but talk to Darin about impact, as discussed. > Source/WebCore/ChangeLog:14 > + Add a test that compares the plug-in's URL against the main page's This looks indented strangely.
Dean Jackson
Comment 6 2013-06-25 14:49:04 PDT
Note You need to log in before you can comment on or make changes to this bug.