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

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

Issue 2627513002: Plugin Power Saver: Refactor some Tiny plugin code and tests. (Closed)
Patch Set: Also update histograms.xml Created 3 years, 11 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: content/public/renderer/render_frame.h
diff --git a/content/public/renderer/render_frame.h b/content/public/renderer/render_frame.h
index 81512cda9c6523dd62f5d934a6dc311df28eabf9..09544aeea03bb466d9e2c1c0ecabc3dc7f2c9c14 100644
--- a/content/public/renderer/render_frame.h
+++ b/content/public/renderer/render_frame.h
@@ -66,7 +66,7 @@ class CONTENT_EXPORT RenderFrame : public IPC::Listener,
public:
// These numeric values are used in UMA logs; do not change them.
enum PeripheralContentStatus {
- // Content is peripheral because it doesn't meet any of the below criteria.
+ // Content is peripheral, and should be throttled, but is not tiny.
CONTENT_STATUS_PERIPHERAL = 0,
// Content is essential because it's same-origin with the top-level frame.
CONTENT_STATUS_ESSENTIAL_SAME_ORIGIN = 1,
@@ -74,10 +74,10 @@ class CONTENT_EXPORT RenderFrame : public IPC::Listener,
CONTENT_STATUS_ESSENTIAL_CROSS_ORIGIN_BIG = 2,
// Content is essential because there's large content from the same origin.
CONTENT_STATUS_ESSENTIAL_CROSS_ORIGIN_WHITELISTED = 3,
- // Content is essential because it's tiny in size.
- CONTENT_STATUS_ESSENTIAL_CROSS_ORIGIN_TINY = 4,
- // Content is essential because it has an unknown size.
- CONTENT_STATUS_ESSENTIAL_UNKNOWN_SIZE = 5,
+ // Content is tiny in size. These are usually blocked.
+ CONTENT_STATUS_TINY = 4,
+ // Content has an unknown size.
+ CONTENT_STATUS_UNKNOWN_SIZE = 5,
// Must be last.
CONTENT_STATUS_NUM_ITEMS
};
« no previous file with comments | « components/plugins/renderer/loadable_plugin_placeholder.cc ('k') | content/renderer/pepper/plugin_power_saver_helper.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698