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

Unified Diff: chrome_frame/utils.cc

Issue 5581008: Add a new GetInstance() method for singleton classes, take 2. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 10 years 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: chrome_frame/utils.cc
diff --git a/chrome_frame/utils.cc b/chrome_frame/utils.cc
index e4279a707105f5abddfc03f9a69128452deabf2f..781d6d4371544a326b01e8e7f3007cb5566d5443 100644
--- a/chrome_frame/utils.cc
+++ b/chrome_frame/utils.cc
@@ -723,8 +723,8 @@ bool IsGcfDefaultRenderer() {
DWORD is_default = 0; // NOLINT
// First check policy settings
- Singleton<PolicySettings> policy;
- PolicySettings::RendererForUrl renderer = policy->default_renderer();
+ PolicySettings::RendererForUrl renderer =
+ PolicySettings::GetInstance()->default_renderer();
if (renderer != PolicySettings::RENDERER_NOT_SPECIFIED) {
is_default = (renderer == PolicySettings::RENDER_IN_CHROME_FRAME);
} else {
@@ -742,9 +742,8 @@ bool IsGcfDefaultRenderer() {
RendererType RendererTypeForUrl(const std::wstring& url) {
// First check if the default renderer settings are specified by policy.
// If so, then that overrides the user settings.
- Singleton<PolicySettings> policy;
- PolicySettings::RendererForUrl renderer = policy->GetRendererForUrl(
- url.c_str());
+ PolicySettings::RendererForUrl renderer =
+ PolicySettings::GetInstance()->GetRendererForUrl(url.c_str());
if (renderer != PolicySettings::RENDERER_NOT_SPECIFIED) {
// We may know at this point that policy says do NOT render in Chrome Frame.
// To maintain consistency, we return RENDERER_TYPE_UNDETERMINED so that

Powered by Google App Engine
This is Rietveld 408576698