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

Unified Diff: content/renderer/render_frame_impl.h

Issue 2484633004: Change Lo-Fi bool to bitmask to support multiple Previews types (Closed)
Patch Set: fix ContentResourceProviderTest 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
« no previous file with comments | « content/public/test/render_view_test.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/render_frame_impl.h
diff --git a/content/renderer/render_frame_impl.h b/content/renderer/render_frame_impl.h
index 2132cb23060fef2c33c36566932275c1b7763222..e8f1dcc7c87663210d7f4ee186aa890392f167b6 100644
--- a/content/renderer/render_frame_impl.h
+++ b/content/renderer/render_frame_impl.h
@@ -34,6 +34,7 @@
#include "content/common/renderer.mojom.h"
#include "content/public/common/console_message_level.h"
#include "content/public/common/javascript_message_type.h"
+#include "content/public/common/previews_state.h"
#include "content/public/common/referrer.h"
#include "content/public/common/stop_find_action.h"
#include "content/public/renderer/render_frame.h"
@@ -447,7 +448,7 @@ class CONTENT_EXPORT RenderFrameImpl
v8::Local<v8::Context> context) override;
void AddMessageToConsole(ConsoleMessageLevel level,
const std::string& message) override;
- bool IsUsingLoFi() const override;
+ PreviewsState GetPreviewsState() const override;
bool IsPasting() const override;
blink::WebPageVisibilityState GetVisibilityState() const override;
bool IsBrowserSideNavigationPending() override;
@@ -1298,8 +1299,10 @@ class CONTENT_EXPORT RenderFrameImpl
// lifetime is tied to the RenderFrame via the RenderFrameObserver interface.
media::RendererWebMediaPlayerDelegate* media_player_delegate_;
- // Whether or not this RenderFrame is using Lo-Fi mode.
- bool is_using_lofi_;
+ // The PreviewsState of this RenderFrame that indicates which Previews can
+ // be used. The PreviewsState is a bitmask of potentially several Previews
+ // optimizations.
+ PreviewsState previews_state_;
// Effective connection type when the document of this frame was fetched.
blink::WebEffectiveConnectionType effective_connection_type_;
« no previous file with comments | « content/public/test/render_view_test.cc ('k') | content/renderer/render_frame_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698