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

Side by Side Diff: chrome/browser/ui/ash/screenshot_taker.h

Issue 686563002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 6 years, 1 month 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_UI_ASH_SCREENSHOT_TAKER_H_ 5 #ifndef CHROME_BROWSER_UI_ASH_SCREENSHOT_TAKER_H_
6 #define CHROME_BROWSER_UI_ASH_SCREENSHOT_TAKER_H_ 6 #define CHROME_BROWSER_UI_ASH_SCREENSHOT_TAKER_H_
7 7
8 #include "ash/screenshot_delegate.h" 8 #include "ash/screenshot_delegate.h"
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "base/compiler_specific.h" 10 #include "base/compiler_specific.h"
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 const base::FilePath& screenshot_path) = 0; 46 const base::FilePath& screenshot_path) = 0;
47 47
48 protected: 48 protected:
49 virtual ~ScreenshotTakerObserver() {} 49 virtual ~ScreenshotTakerObserver() {}
50 }; 50 };
51 51
52 class ScreenshotTaker : public ash::ScreenshotDelegate { 52 class ScreenshotTaker : public ash::ScreenshotDelegate {
53 public: 53 public:
54 ScreenshotTaker(); 54 ScreenshotTaker();
55 55
56 virtual ~ScreenshotTaker(); 56 ~ScreenshotTaker() override;
57 57
58 // Overridden from ash::ScreenshotDelegate: 58 // Overridden from ash::ScreenshotDelegate:
59 virtual void HandleTakeScreenshotForAllRootWindows() override; 59 void HandleTakeScreenshotForAllRootWindows() override;
60 virtual void HandleTakePartialScreenshot(aura::Window* window, 60 void HandleTakePartialScreenshot(aura::Window* window,
61 const gfx::Rect& rect) override; 61 const gfx::Rect& rect) override;
62 virtual bool CanTakeScreenshot() override; 62 bool CanTakeScreenshot() override;
63 63
64 void ShowNotification( 64 void ShowNotification(
65 ScreenshotTakerObserver::Result screenshot_result, 65 ScreenshotTakerObserver::Result screenshot_result,
66 const base::FilePath& screenshot_path); 66 const base::FilePath& screenshot_path);
67 67
68 void AddObserver(ScreenshotTakerObserver* observer); 68 void AddObserver(ScreenshotTakerObserver* observer);
69 void RemoveObserver(ScreenshotTakerObserver* observer); 69 void RemoveObserver(ScreenshotTakerObserver* observer);
70 bool HasObserver(ScreenshotTakerObserver* observer) const; 70 bool HasObserver(ScreenshotTakerObserver* observer) const;
71 71
72 private: 72 private:
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
106 base::FilePath screenshot_directory_for_test_; 106 base::FilePath screenshot_directory_for_test_;
107 std::string screenshot_basename_for_test_; 107 std::string screenshot_basename_for_test_;
108 Profile* profile_for_test_; 108 Profile* profile_for_test_;
109 109
110 base::WeakPtrFactory<ScreenshotTaker> factory_; 110 base::WeakPtrFactory<ScreenshotTaker> factory_;
111 111
112 DISALLOW_COPY_AND_ASSIGN(ScreenshotTaker); 112 DISALLOW_COPY_AND_ASSIGN(ScreenshotTaker);
113 }; 113 };
114 114
115 #endif // CHROME_BROWSER_UI_ASH_SCREENSHOT_TAKER_H_ 115 #endif // CHROME_BROWSER_UI_ASH_SCREENSHOT_TAKER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698