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

Unified Diff: chrome/browser/ui/views/aura/screenshot_taker.h

Issue 9545014: Make the visual feedback for taking screenshot, with fix. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/aura/screenshot_taker.h
diff --git a/chrome/browser/ui/views/aura/screenshot_taker.h b/chrome/browser/ui/views/aura/screenshot_taker.h
index c6707e24a798f17f8bacd5fb313f016da54684b4..f4b255cd8f4283f64bd7577a818b03084e450d02 100644
--- a/chrome/browser/ui/views/aura/screenshot_taker.h
+++ b/chrome/browser/ui/views/aura/screenshot_taker.h
@@ -9,6 +9,7 @@
#include "ash/screenshot_delegate.h"
#include "base/basictypes.h"
#include "base/compiler_specific.h"
+#include "ui/gfx/compositor/layer.h"
namespace aura {
class Window;
@@ -17,6 +18,7 @@ class Window;
class ScreenshotTaker : public ash::ScreenshotDelegate {
public:
ScreenshotTaker();
+ virtual ~ScreenshotTaker();
// Overridden from ash::ScreenshotDelegate:
virtual void HandleTakeScreenshot(aura::Window* window) OVERRIDE;
@@ -24,6 +26,15 @@ class ScreenshotTaker : public ash::ScreenshotDelegate {
aura::Window* window, const gfx::Rect& rect) OVERRIDE;
private:
+ // Flashes the screen to provide visual feedback that a screenshot has
+ // been taken.
+ void DisplayVisualFeedback(const gfx::Rect& rect);
+
+ // Closes the visual feedback layer.
+ void CloseVisualFeedbackLayer();
+
+ scoped_ptr<ui::Layer> visual_feedback_layer_;
+
DISALLOW_COPY_AND_ASSIGN(ScreenshotTaker);
};
« no previous file with comments | « no previous file | chrome/browser/ui/views/aura/screenshot_taker.cc » ('j') | chrome/browser/ui/views/aura/screenshot_taker.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698