Change Lo-Fi bool to bitmask to support multiple Previews types

content/browser/frame_host/render_frame_host_impl.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
 #define CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_
 
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 551 matching lines @@
   void ResetLoadingState();
 
   // Tells the renderer that this RenderFrame will soon be swapped out, and thus
   // not to create any new modal dialogs until it happens.  This must be done
   // separately so that the ScopedPageLoadDeferrers of any current dialogs are
   // no longer on the stack when we attempt to swap it out.
   void SuppressFurtherDialogs();
 
   void SetHasReceivedUserGesture();
 
-  // PlzNavigate: returns the LoFi state of the last successful navigation that
-  // made a network request.
-  LoFiState last_navigation_lofi_state() const {
-    return last_navigation_lofi_state_;
+  // PlzNavigate: returns the Previews state of the last successful navigation
+  // that made a network request.
+  int last_navigation_previews_state() const {

[nasko, 2016/12/08 22:19:32]

PreviewsState

[megjablon, 2016/12/09 20:35:54]

Done.

+    return last_navigation_previews_state_;
   }
 
  protected:
   friend class RenderFrameHostFactory;
 
   // |flags| is a combination of CreateRenderFrameFlags.
   // TODO(nasko): Remove dependency on RenderViewHost here. RenderProcessHost
   // should be the abstraction needed here, but we need RenderViewHost to pass
   // into WebContentsObserver::FrameDetached for now.
   RenderFrameHostImpl(SiteInstance* site_instance,
@@ skipping 451 matching lines @@
   std::unique_ptr<WebUIImpl> pending_web_ui_;
   WebUI::TypeID pending_web_ui_type_;
 
   // If true the associated WebUI should be reused when CommitPendingWebUI is
   // called (no pending instance should be set).
   bool should_reuse_web_ui_;
 
   // If true, then the RenderFrame has selected text.
   bool has_selection_;
 
-  // PlzNavigate: The LoFi state of the last navigation. This is used during
+  // PlzNavigate: The Previews state of the last navigation. This is used during
   // history navigation of subframes to ensure that subframes navigate with the
-  // same LoFi status as the top-level frame.
-  LoFiState last_navigation_lofi_state_;
+  // same Previews status as the top-level frame.
+  int last_navigation_previews_state_;

[nasko, 2016/12/08 22:19:32]

PreviewsState

[megjablon, 2016/12/09 20:35:54]

Done.

 
   mojo::Binding<mojom::FrameHost> frame_host_binding_;
   mojom::FramePtr frame_;
 
   // If this is true then this object was created in response to a renderer
   // initiated request. Init() will be called, and until then navigation
   // requests should be queued.
   bool waiting_for_init_;
 
   typedef std::pair<CommonNavigationParams, BeginNavigationParams>
@@ skipping 11 matching lines @@
       remote_associated_interfaces_;
   // NOTE: This must be the last member.
   base::WeakPtrFactory<RenderFrameHostImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderFrameHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_RENDER_FRAME_HOST_IMPL_H_