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

Unified Diff: content/public/renderer/render_frame.h

Issue 1497623002: Plugin Power Saver: Improve Poster behavior for essential plugins. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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: content/public/renderer/render_frame.h
diff --git a/content/public/renderer/render_frame.h b/content/public/renderer/render_frame.h
index 82caa92217e6e281ab5d77b9bed2b0ab8c53105a..6b3681a3f1c9554d8bd54a2a818a7170289589a5 100644
--- a/content/public/renderer/render_frame.h
+++ b/content/public/renderer/render_frame.h
@@ -147,6 +147,8 @@ class CONTENT_EXPORT RenderFrame : public IPC::Listener,
// |content_origin| is the origin of the plugin content.
//
// |width| and |height| are zoom and device scale independent logical pixels.
groby-ooo-7-16 2015/12/04 22:06:56 Can we make this gfx::Size? And we probably want
tommycli 2015/12/07 22:06:18 Done. Yeah that's a good idea. That way I can use
groby-ooo-7-16 2015/12/07 22:16:04 Careful - it _might_ be we need to treat 0x0 plugi
tommycli 2015/12/07 22:22:13 Yeah this comment has gone away. Currently, 0x0 pl
+ // If they are negative, the sizing is ignored, and the method returns false
+ // if and only if the origin marks the content as essential.
//
// |cross_origin_main_content| may be NULL. It is set to true if the
// plugin content is cross-origin but still the "main attraction" of the page.

Powered by Google App Engine
This is Rietveld 408576698