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

Side by Side Diff: chrome/browser/ui/views/javascript_app_modal_event_blocker_x11.h

Issue 680133002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 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_UI_VIEWS_JAVASCRIPT_APP_MODAL_EVENT_BLOCKER_X11_H_ 5 #ifndef CHROME_BROWSER_UI_VIEWS_JAVASCRIPT_APP_MODAL_EVENT_BLOCKER_X11_H_
6 #define CHROME_BROWSER_UI_VIEWS_JAVASCRIPT_APP_MODAL_EVENT_BLOCKER_X11_H_ 6 #define CHROME_BROWSER_UI_VIEWS_JAVASCRIPT_APP_MODAL_EVENT_BLOCKER_X11_H_
7 7
8 #include "ui/events/event_handler.h" 8 #include "ui/events/event_handler.h"
9 9
10 class BrowserView; 10 class BrowserView;
11 11
12 namespace aura { 12 namespace aura {
13 class Window; 13 class Window;
14 } 14 }
15 15
16 // JavascriptAppModalEventBlockerX11 blocks events to all browser windows except 16 // JavascriptAppModalEventBlockerX11 blocks events to all browser windows except
17 // the browser window which hosts |app_modal_window| for the duration of its 17 // the browser window which hosts |app_modal_window| for the duration of its
18 // lifetime. JavascriptAppModalEventBlockerX11 should not outlive 18 // lifetime. JavascriptAppModalEventBlockerX11 should not outlive
19 // |app_modal_window|. 19 // |app_modal_window|.
20 // TODO(pkotwicz): Merge this class into WindowModalityController. 20 // TODO(pkotwicz): Merge this class into WindowModalityController.
21 class JavascriptAppModalEventBlockerX11 : public ui::EventHandler { 21 class JavascriptAppModalEventBlockerX11 : public ui::EventHandler {
22 public: 22 public:
23 explicit JavascriptAppModalEventBlockerX11(aura::Window* app_modal_window); 23 explicit JavascriptAppModalEventBlockerX11(aura::Window* app_modal_window);
24 virtual ~JavascriptAppModalEventBlockerX11(); 24 ~JavascriptAppModalEventBlockerX11() override;
25 25
26 private: 26 private:
27 // Returns true if the propagation of events to |target| should be stopped. 27 // Returns true if the propagation of events to |target| should be stopped.
28 bool ShouldStopPropagationTo(ui::EventTarget* target); 28 bool ShouldStopPropagationTo(ui::EventTarget* target);
29 29
30 // ui::EventHandler: 30 // ui::EventHandler:
31 virtual void OnKeyEvent(ui::KeyEvent* event) override; 31 void OnKeyEvent(ui::KeyEvent* event) override;
32 virtual void OnMouseEvent(ui::MouseEvent* event) override; 32 void OnMouseEvent(ui::MouseEvent* event) override;
33 virtual void OnScrollEvent(ui::ScrollEvent* event) override; 33 void OnScrollEvent(ui::ScrollEvent* event) override;
34 virtual void OnTouchEvent(ui::TouchEvent* event) override; 34 void OnTouchEvent(ui::TouchEvent* event) override;
35 35
36 // The app modal dialog. 36 // The app modal dialog.
37 aura::Window* modal_window_; 37 aura::Window* modal_window_;
38 38
39 // The BrowserView which hosts the app modal dialog. 39 // The BrowserView which hosts the app modal dialog.
40 BrowserView* browser_view_with_modal_dialog_; 40 BrowserView* browser_view_with_modal_dialog_;
41 41
42 DISALLOW_COPY_AND_ASSIGN(JavascriptAppModalEventBlockerX11); 42 DISALLOW_COPY_AND_ASSIGN(JavascriptAppModalEventBlockerX11);
43 }; 43 };
44 44
45 #endif // CHROME_BROWSER_UI_VIEWS_JAVASCRIPT_APP_MODAL_EVENT_BLOCKER_X11_H_ 45 #endif // CHROME_BROWSER_UI_VIEWS_JAVASCRIPT_APP_MODAL_EVENT_BLOCKER_X11_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698