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

Unified Diff: android_webview/browser/aw_web_preferences_populater.cc

Issue 2884353003: [WebView] Remove more redundant classes (Closed)
Patch Set: Created 3 years, 7 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: android_webview/browser/aw_web_preferences_populater.cc
diff --git a/android_webview/browser/aw_web_preferences_populater.cc b/android_webview/browser/aw_web_preferences_populater.cc
index 008cf5cb761f2e6b7e5e1cb0be6359e24546fe7c..3d4fb147c39b4a676ee17603f37c80d46166b315 100644
--- a/android_webview/browser/aw_web_preferences_populater.cc
+++ b/android_webview/browser/aw_web_preferences_populater.cc
@@ -3,9 +3,17 @@
// found in the LICENSE file.
#include "android_webview/browser/aw_web_preferences_populater.h"
+#include "android_webview/browser/aw_settings.h"
namespace android_webview {
-AwWebPreferencesPopulater::~AwWebPreferencesPopulater() {}
+void AwWebPreferencesPopulater::PopulateFor(
boliu 2017/05/17 16:18:11 you can remove this class entirely and just inline
+ content::WebContents* web_contents,
+ content::WebPreferences* web_prefs) {
+ AwSettings* aw_settings = AwSettings::FromWebContents(web_contents);
+ if (aw_settings) {
+ aw_settings->PopulateWebPreferences(web_prefs);
+ }
+}
} // namespace android_webview

Powered by Google App Engine
This is Rietveld 408576698