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

Unified Diff: ios/web/web_state/ui/wk_web_view_configuration_provider.h

Issue 2741343015: Add a BrowserState* parameter to GetEarlyPageScript(). (Closed)
Patch Set: Fix errors. Created 3 years, 9 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: ios/web/web_state/ui/wk_web_view_configuration_provider.h
diff --git a/ios/web/web_state/ui/wk_web_view_configuration_provider.h b/ios/web/web_state/ui/wk_web_view_configuration_provider.h
index 84191f74ab00bb77772d945a84312f46c7b3e87b..8470895f2ca8a1d8b7a10fa8a8f00d4cd99af6c7 100644
--- a/ios/web/web_state/ui/wk_web_view_configuration_provider.h
+++ b/ios/web/web_state/ui/wk_web_view_configuration_provider.h
@@ -45,7 +45,8 @@ class WKWebViewConfigurationProvider : public base::SupportsUserData::Data {
void Purge();
private:
- explicit WKWebViewConfigurationProvider(bool is_off_the_record);
+ explicit WKWebViewConfigurationProvider(bool is_off_the_record,
Eugene But (OOO till 7-30) 2017/03/16 16:12:40 No need to pass |is_off_the_record| flag anymore a
Hiroshi Ichikawa 2017/03/17 06:09:24 Good point. Done.
+ BrowserState* browser_state);
WKWebViewConfigurationProvider() = delete;
~WKWebViewConfigurationProvider() override;
@@ -53,6 +54,7 @@ class WKWebViewConfigurationProvider : public base::SupportsUserData::Data {
base::scoped_nsobject<CRWWKScriptMessageRouter> router_;
// Result of |web::BrowserState::IsOffTheRecord| call.
bool is_off_the_record_;
+ BrowserState* browser_state_;
DISALLOW_COPY_AND_ASSIGN(WKWebViewConfigurationProvider);
};

Powered by Google App Engine
This is Rietveld 408576698