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

Side by Side Diff: chrome/browser/shell_integration.h

Issue 1997453003: Fix "unused variable" warnings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move changes in and out to fix compile Created 4 years, 7 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 #ifndef CHROME_BROWSER_SHELL_INTEGRATION_H_ 5 #ifndef CHROME_BROWSER_SHELL_INTEGRATION_H_
6 #define CHROME_BROWSER_SHELL_INTEGRATION_H_ 6 #define CHROME_BROWSER_SHELL_INTEGRATION_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 187 matching lines...) Expand 10 before | Expand all | Expand 10 after
198 void UpdateUI(DefaultWebClientState state); 198 void UpdateUI(DefaultWebClientState state);
199 199
200 // Called with the default state after the worker is done. 200 // Called with the default state after the worker is done.
201 DefaultWebClientWorkerCallback callback_; 201 DefaultWebClientWorkerCallback callback_;
202 202
203 // Used to differentiate UMA metrics for setting the default browser and 203 // Used to differentiate UMA metrics for setting the default browser and
204 // setting the default protocol client. The pointer must be valid for the 204 // setting the default protocol client. The pointer must be valid for the
205 // lifetime of the worker. 205 // lifetime of the worker.
206 const char* worker_name_; 206 const char* worker_name_;
207 207
208 // Wait until Chrome has been confirmed as the default browser before
209 // reporting a successful attempt.
210 bool check_default_should_report_success_ = false;
211
212 DISALLOW_COPY_AND_ASSIGN(DefaultWebClientWorker); 208 DISALLOW_COPY_AND_ASSIGN(DefaultWebClientWorker);
213 }; 209 };
214 210
215 // Worker for checking and setting the default browser. 211 // Worker for checking and setting the default browser.
216 class DefaultBrowserWorker : public DefaultWebClientWorker { 212 class DefaultBrowserWorker : public DefaultWebClientWorker {
217 public: 213 public:
218 explicit DefaultBrowserWorker(const DefaultWebClientWorkerCallback& callback); 214 explicit DefaultBrowserWorker(const DefaultWebClientWorkerCallback& callback);
219 215
220 private: 216 private:
221 ~DefaultBrowserWorker() override; 217 ~DefaultBrowserWorker() override;
(...skipping 29 matching lines...) Expand all
251 void SetAsDefaultImpl(const base::Closure& on_finished_callback) override; 247 void SetAsDefaultImpl(const base::Closure& on_finished_callback) override;
252 248
253 std::string protocol_; 249 std::string protocol_;
254 250
255 DISALLOW_COPY_AND_ASSIGN(DefaultProtocolClientWorker); 251 DISALLOW_COPY_AND_ASSIGN(DefaultProtocolClientWorker);
256 }; 252 };
257 253
258 } // namespace shell_integration 254 } // namespace shell_integration
259 255
260 #endif // CHROME_BROWSER_SHELL_INTEGRATION_H_ 256 #endif // CHROME_BROWSER_SHELL_INTEGRATION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698