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

Side by Side Diff: chrome/browser/apps/drive/drive_app_converter.h

Issue 668903002: Standardize usage of virtual/override/final in chrome/browser/apps/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_APPS_DRIVE_DRIVE_APP_CONVERTER_H_ 5 #ifndef CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_CONVERTER_H_
6 #define CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_CONVERTER_H_ 6 #define CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_CONVERTER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 14 matching lines...) Expand all
25 // DriveAppConverter creates and installs a local URL app for the given 25 // DriveAppConverter creates and installs a local URL app for the given
26 // DriveAppInfo into the given profile. 26 // DriveAppInfo into the given profile.
27 class DriveAppConverter : public extensions::InstallObserver { 27 class DriveAppConverter : public extensions::InstallObserver {
28 public: 28 public:
29 typedef base::Callback<void(const DriveAppConverter*, bool success)> 29 typedef base::Callback<void(const DriveAppConverter*, bool success)>
30 FinishedCallback; 30 FinishedCallback;
31 31
32 DriveAppConverter(Profile* profile, 32 DriveAppConverter(Profile* profile,
33 const drive::DriveAppInfo& drive_app_info, 33 const drive::DriveAppInfo& drive_app_info,
34 const FinishedCallback& finished_callback); 34 const FinishedCallback& finished_callback);
35 virtual ~DriveAppConverter(); 35 ~DriveAppConverter() override;
36 36
37 void Start(); 37 void Start();
38 bool IsStarted() const; 38 bool IsStarted() const;
39 39
40 bool IsInstalling(const std::string& app_id) const; 40 bool IsInstalling(const std::string& app_id) const;
41 41
42 const drive::DriveAppInfo& drive_app_info() const { return drive_app_info_; } 42 const drive::DriveAppInfo& drive_app_info() const { return drive_app_info_; }
43 const extensions::Extension* extension() const { return extension_; } 43 const extensions::Extension* extension() const { return extension_; }
44 bool is_new_install() const { return is_new_install_; } 44 bool is_new_install() const { return is_new_install_; }
45 45
46 private: 46 private:
47 class IconFetcher; 47 class IconFetcher;
48 48
49 // Callbacks from IconFetcher. 49 // Callbacks from IconFetcher.
50 void OnIconFetchComplete(const IconFetcher* fetcher); 50 void OnIconFetchComplete(const IconFetcher* fetcher);
51 51
52 void StartInstall(); 52 void StartInstall();
53 void PostInstallCleanUp(); 53 void PostInstallCleanUp();
54 54
55 // extensions::InstallObserver: 55 // extensions::InstallObserver:
56 virtual void OnFinishCrxInstall(const std::string& extension_id, 56 void OnFinishCrxInstall(const std::string& extension_id,
57 bool success) override; 57 bool success) override;
58 58
59 Profile* profile_; 59 Profile* profile_;
60 const drive::DriveAppInfo drive_app_info_; 60 const drive::DriveAppInfo drive_app_info_;
61 61
62 WebApplicationInfo web_app_; 62 WebApplicationInfo web_app_;
63 const extensions::Extension* extension_; 63 const extensions::Extension* extension_;
64 bool is_new_install_; 64 bool is_new_install_;
65 65
66 ScopedVector<IconFetcher> fetchers_; 66 ScopedVector<IconFetcher> fetchers_;
67 scoped_refptr<extensions::CrxInstaller> crx_installer_; 67 scoped_refptr<extensions::CrxInstaller> crx_installer_;
68 68
69 FinishedCallback finished_callback_; 69 FinishedCallback finished_callback_;
70 70
71 DISALLOW_COPY_AND_ASSIGN(DriveAppConverter); 71 DISALLOW_COPY_AND_ASSIGN(DriveAppConverter);
72 }; 72 };
73 73
74 #endif // CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_CONVERTER_H_ 74 #endif // CHROME_BROWSER_APPS_DRIVE_DRIVE_APP_CONVERTER_H_
OLDNEW
« no previous file with comments | « chrome/browser/apps/app_window_browsertest.cc ('k') | chrome/browser/apps/drive/drive_app_converter.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698