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

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

Issue 2349073002: Blimp Settings framework on the c++ side (Closed)
Patch Set: nits and sync to head 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
« no previous file with comments | « blimp/client/app/linux/blimp_main.cc ('k') | blimp/client/core/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: blimp/client/app/session/blimp_client_session.cc
diff --git a/blimp/client/app/session/blimp_client_session.cc b/blimp/client/app/session/blimp_client_session.cc
index 9af43b2ddde089dd3e658785e0de47ac2dfa359b..e28322e19aee9af4e3bfa1dd15104f4fed0f95f2 100644
--- a/blimp/client/app/session/blimp_client_session.cc
+++ b/blimp/client/app/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() {
« no previous file with comments | « blimp/client/app/linux/blimp_main.cc ('k') | blimp/client/core/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698