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

Unified Diff: blimp/client/session/blimp_client_session.cc

Issue 2349073002: Blimp Settings framework on the c++ side (Closed)
Patch Set: Remove OnRecordWholeDocumentChanged from SettingsObserver and SettingsFeature Created 4 years, 2 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: blimp/client/session/blimp_client_session.cc
diff --git a/blimp/client/session/blimp_client_session.cc b/blimp/client/session/blimp_client_session.cc
index 26460d74b58a6a3271bf89df54f6438ce46332cd..2a54fcf3db389841e3ac61528547fd30eb2c2cd9 100644
--- a/blimp/client/session/blimp_client_session.cc
+++ b/blimp/client/session/blimp_client_session.cc
@@ -55,7 +55,7 @@ BlimpClientSession::BlimpClientSession(const GURL& assigner_endpoint)
navigation_feature_(new NavigationFeature),
ime_feature_(new ImeFeature),
render_widget_feature_(new RenderWidgetFeature),
- settings_feature_(new SettingsFeature),
+ settings_feature_(new SettingsFeature(nullptr)),
weak_factory_(this) {
base::Thread::Options options;
options.message_loop_type = base::MessageLoop::TYPE_IO;
@@ -151,10 +151,6 @@ void BlimpClientSession::RegisterFeatures() {
ime_feature_->set_outgoing_message_processor(
thread_pipe_manager_->RegisterFeature(BlimpMessage::kIme,
ime_feature_.get()));
-
- if (base::CommandLine::ForCurrentProcess()->HasSwitch(
- switches::kDownloadWholeDocument))
- settings_feature_->SetRecordWholeDocument(true);
}
void BlimpClientSession::DropConnection() {

Powered by Google App Engine
This is Rietveld 408576698