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

Unified Diff: chrome/browser/previews/previews_infobar_tab_helper.h

Issue 2833523002: Adding opt out and previews type information to DRP pingback (Closed)
Patch Set: moved definition up Created 3 years, 8 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/browser/previews/previews_infobar_tab_helper.h
diff --git a/chrome/browser/previews/previews_infobar_tab_helper.h b/chrome/browser/previews/previews_infobar_tab_helper.h
index 512726f127512966e6d584d7c32d679b052af916..e938da7f66bbd5f5c80b05f4a8797b4e1bb5921d 100644
--- a/chrome/browser/previews/previews_infobar_tab_helper.h
+++ b/chrome/browser/previews/previews_infobar_tab_helper.h
@@ -6,6 +6,7 @@
#define CHROME_BROWSER_PREVIEWS_PREVIEWS_INFOBAR_TAB_HELPER_H_
#include "base/macros.h"
+#include "base/optional.h"
#include "content/public/browser/web_contents_observer.h"
#include "content/public/browser/web_contents_user_data.h"
@@ -29,6 +30,11 @@ class PreviewsInfoBarTabHelper
displayed_preview_infobar_ = displayed;
}
+ // The data saver page identifier of the current page load.
+ const base::Optional<uint64_t>& committed_data_saver_page_id() {
bengr 2017/04/20 17:36:18 #include <stdint.h>
RyanSturm 2017/04/20 20:25:44 Acknowledged.
+ return committed_data_saver_page_id_;
+ }
+
private:
friend class content::WebContentsUserData<PreviewsInfoBarTabHelper>;
friend class PreviewsInfoBarTabHelperUnitTest;
@@ -39,9 +45,16 @@ class PreviewsInfoBarTabHelper
void DidFinishNavigation(
content::NavigationHandle* navigation_handle) override;
+ // Stored for use in the destructor.
+ content::BrowserContext* browser_context_;
+ content::WebContents* web_contents_;
+
// True if the InfoBar for a preview has been shown for the page.
bool displayed_preview_infobar_;
+ // The data saver page identifier of the current page load.
+ base::Optional<uint64_t> committed_data_saver_page_id_;
+
DISALLOW_COPY_AND_ASSIGN(PreviewsInfoBarTabHelper);
};

Powered by Google App Engine
This is Rietveld 408576698