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

Side by Side Diff: chrome/browser/chromeos/ui/focus_ring_layer.h

Issue 602813003: Animate the accessibility focus ring and fix some minor visual issues. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@focus_ring_in_chromevox
Patch Set: Fix unit tests 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
« no previous file with comments | « chrome/browser/chromeos/ui/accessibility_focus_ring_layer.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_CHROMEOS_UI_FOCUS_RING_LAYER_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_UI_FOCUS_RING_LAYER_H_
6 #define CHROME_BROWSER_CHROMEOS_UI_FOCUS_RING_LAYER_H_ 6 #define CHROME_BROWSER_CHROMEOS_UI_FOCUS_RING_LAYER_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "ui/compositor/layer_delegate.h" 10 #include "ui/compositor/layer_delegate.h"
(...skipping 22 matching lines...) Expand all
33 class FocusRingLayer : public ui::LayerDelegate { 33 class FocusRingLayer : public ui::LayerDelegate {
34 public: 34 public:
35 explicit FocusRingLayer(FocusRingLayerDelegate* delegate); 35 explicit FocusRingLayer(FocusRingLayerDelegate* delegate);
36 virtual ~FocusRingLayer(); 36 virtual ~FocusRingLayer();
37 37
38 // Move the focus ring layer to the given bounds in the coordinates of 38 // Move the focus ring layer to the given bounds in the coordinates of
39 // the given root window. 39 // the given root window.
40 void Set(aura::Window* root_window, const gfx::Rect& bounds); 40 void Set(aura::Window* root_window, const gfx::Rect& bounds);
41 41
42 ui::Layer* layer() { return layer_.get(); } 42 ui::Layer* layer() { return layer_.get(); }
43 aura::Window* root_window() { return root_window_; }
43 44
44 protected: 45 protected:
45 // Updates |root_window_| and creates |layer_| if it doesn't exist, 46 // Updates |root_window_| and creates |layer_| if it doesn't exist,
46 // or if the root window has changed. Moves the layer to the top if 47 // or if the root window has changed. Moves the layer to the top if
47 // it wasn't there already. 48 // it wasn't there already.
48 void CreateOrUpdateLayer(aura::Window* root_window, const char* layer_name); 49 void CreateOrUpdateLayer(aura::Window* root_window, const char* layer_name);
49 50
50 private: 51 private:
51 // ui::LayerDelegate overrides: 52 // ui::LayerDelegate overrides:
52 virtual void OnPaintLayer(gfx::Canvas* canvas) OVERRIDE; 53 virtual void OnPaintLayer(gfx::Canvas* canvas) OVERRIDE;
(...skipping 14 matching lines...) Expand all
67 // The bounding rectangle of the focused object, in |root_window_| 68 // The bounding rectangle of the focused object, in |root_window_|
68 // coordinates. 69 // coordinates.
69 gfx::Rect focus_ring_; 70 gfx::Rect focus_ring_;
70 71
71 DISALLOW_COPY_AND_ASSIGN(FocusRingLayer); 72 DISALLOW_COPY_AND_ASSIGN(FocusRingLayer);
72 }; 73 };
73 74
74 } // namespace chromeos 75 } // namespace chromeos
75 76
76 #endif // CHROME_BROWSER_CHROMEOS_UI_FOCUS_RING_LAYER_H_ 77 #endif // CHROME_BROWSER_CHROMEOS_UI_FOCUS_RING_LAYER_H_
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/ui/accessibility_focus_ring_layer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698