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

Side by Side Diff: chrome/installer/util/browser_distribution.cc

Issue 2718063002: Use install_static::GetAppGuid in place of BrowserDistribution and AppRegistrationData. (Closed)
Patch Set: huangs feedback plus Chromium test fix Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 // 4 //
5 // This file defines a class that contains various method related to branding. 5 // This file defines a class that contains various method related to branding.
6 // It provides only default implementations of these methods. Usually to add 6 // It provides only default implementations of these methods. Usually to add
7 // specific branding, we will need to extend this class with a custom 7 // specific branding, we will need to extend this class with a custom
8 // implementation. 8 // implementation.
9 9
10 #include "chrome/installer/util/browser_distribution.h" 10 #include "chrome/installer/util/browser_distribution.h"
(...skipping 74 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 &g_browser_distribution); 85 &g_browser_distribution);
86 #endif 86 #endif
87 87
88 return dist; 88 return dist;
89 } 89 }
90 90
91 const AppRegistrationData& BrowserDistribution::GetAppRegistrationData() const { 91 const AppRegistrationData& BrowserDistribution::GetAppRegistrationData() const {
92 return *app_reg_data_; 92 return *app_reg_data_;
93 } 93 }
94 94
95 base::string16 BrowserDistribution::GetAppGuid() const {
96 return app_reg_data_->GetAppGuid();
97 }
98
99 base::string16 BrowserDistribution::GetStateKey() const { 95 base::string16 BrowserDistribution::GetStateKey() const {
100 return app_reg_data_->GetStateKey(); 96 return app_reg_data_->GetStateKey();
101 } 97 }
102 98
103 base::string16 BrowserDistribution::GetStateMediumKey() const { 99 base::string16 BrowserDistribution::GetStateMediumKey() const {
104 return app_reg_data_->GetStateMediumKey(); 100 return app_reg_data_->GetStateMediumKey();
105 } 101 }
106 102
107 base::string16 BrowserDistribution::GetVersionKey() const { 103 base::string16 BrowserDistribution::GetVersionKey() const {
108 return app_reg_data_->GetVersionKey(); 104 return app_reg_data_->GetVersionKey();
(...skipping 107 matching lines...) Expand 10 before | Expand all | Expand 10 after
216 installer::InstallStatus install_status) { 212 installer::InstallStatus install_status) {
217 } 213 }
218 214
219 bool BrowserDistribution::ShouldSetExperimentLabels() { 215 bool BrowserDistribution::ShouldSetExperimentLabels() {
220 return false; 216 return false;
221 } 217 }
222 218
223 bool BrowserDistribution::HasUserExperiments() { 219 bool BrowserDistribution::HasUserExperiments() {
224 return false; 220 return false;
225 } 221 }
OLDNEW
« no previous file with comments | « chrome/installer/util/browser_distribution.h ('k') | chrome/installer/util/google_chrome_distribution.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698