Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(313)

Unified Diff: chrome/test/data/load_pepper_plugin.html

Issue 2356053002: [HBD] Only use Plugin Content Settings for Flash. (Closed)
Patch Set: fix Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/load_pepper_plugin.html
diff --git a/chrome/test/data/load_pepper_plugin.html b/chrome/test/data/load_pepper_plugin.html
index d1103a8e60f5016218b9e762b05446d62e7edefe..1bf74c6ae6514b95e8ca1ce1960481c64b638aa3 100644
--- a/chrome/test/data/load_pepper_plugin.html
+++ b/chrome/test/data/load_pepper_plugin.html
@@ -22,7 +22,7 @@ var mimeType = QueryString.mimetype;
function inject() {
var child = document.createElement('div');
- child.innerHTML = '<object type="' + mimeType + '" id="plugin" border=1>' +
+ child.innerHTML = '<object type="' + mimeType + '" id="plugin" data="foo">' +
raymes 2016/09/26 08:18:29 is data="foo" needed?
tommycli 2016/09/27 18:51:06 Yes, and I added a comment.
' <b>You should not see this text!</b>' +
'</object>';
document.getElementById('content').appendChild(child);
@@ -36,12 +36,12 @@ function inject() {
document.title = 'Not Loaded';
return;
}
-
+
plugin.postMessage('hello');
// If we do not get an exception, the Pepper plugin is loaded.
document.title = 'Loaded';
} catch (e) {
- var errorMessage = 'Unexpected Exception: ' + e.toString();
+ var errorMessage = 'Unexpected Exception: ' + e.toString();
document.title = errorMessage;
console.log(errorMessage);
}

Powered by Google App Engine
This is Rietveld 408576698