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

Unified Diff: chrome_frame/policy_settings.h

Issue 9836037: Adding policy support to Chrome Frame's launcher so that additional parameters can be passed to Chr… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address review comments from Greg Created 8 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
« no previous file with comments | « chrome_frame/chrome_launcher_utils.cc ('k') | chrome_frame/policy_settings.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome_frame/policy_settings.h
diff --git a/chrome_frame/policy_settings.h b/chrome_frame/policy_settings.h
index 8370b47513ff6cb1d4fe51ba6c388a4a7cfdda11..aaae32c5d305968629cc2fe43b02e00fb21ab192 100644
--- a/chrome_frame/policy_settings.h
+++ b/chrome_frame/policy_settings.h
@@ -1,4 +1,4 @@
-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
@@ -7,9 +7,10 @@
#include <string>
#include <vector>
-#include "base/memory/singleton.h"
#include "base/basictypes.h"
+#include "base/command_line.h"
+#include "base/memory/singleton.h"
// A simple class that reads and caches policy settings for Chrome Frame.
// TODO(tommi): Support refreshing when new settings are pushed.
@@ -38,15 +39,22 @@ class PolicySettings {
return application_locale_;
}
+ // Contains additional parameters that can optionally be configured for the
+ // current user via the policy settings. The program part of this command
+ // line object must not be used when appending to another command line.
+ const CommandLine& AdditionalLaunchParameters() const;
+
// Helper functions for reading settings from the registry
static void ReadUrlSettings(RendererForUrl* default_renderer,
std::vector<std::wstring>* renderer_exclusion_list);
static void ReadContentTypeSetting(
std::vector<std::wstring>* content_type_list);
- static void ReadApplicationLocaleSetting(std::wstring* application_locale);
+ static void ReadStringSetting(const char* value_name, std::wstring* value);
protected:
- PolicySettings() : default_renderer_(RENDERER_NOT_SPECIFIED) {
+ PolicySettings()
+ : default_renderer_(RENDERER_NOT_SPECIFIED),
+ additional_launch_parameters_(CommandLine::NO_PROGRAM) {
RefreshFromRegistry();
}
@@ -61,6 +69,7 @@ class PolicySettings {
std::vector<std::wstring> renderer_exclusion_list_;
std::vector<std::wstring> content_type_list_;
std::wstring application_locale_;
+ CommandLine additional_launch_parameters_;
private:
// This ensures no construction is possible outside of the class itself.
« no previous file with comments | « chrome_frame/chrome_launcher_utils.cc ('k') | chrome_frame/policy_settings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698