MacViews: Handle three-finger swipe gestures for navigation.

chrome/browser/ui/views/frame/browser_view.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 #define CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_
 
 #include <map>
 #include <memory>
 #include <string>
@@ skipping 423 matching lines @@
 
   // Overridden from views::View:
   const char* GetClassName() const override;
   void Layout() override;
   void ViewHierarchyChanged(
       const ViewHierarchyChangedDetails& details) override;
   void ChildPreferredSizeChanged(View* child) override;
   void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
   void OnThemeChanged() override;
   void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
+  void OnGestureEvent(ui::GestureEvent* event) override;

[tapted, 2017/02/13 11:26:24]

nit: sort according to view.h order (double-check, but I think this should move
up a line)

[themblsha, 2017/02/15 17:35:34]

Sorting them all correctly will introduce a major diff, as I would also need to
re-shuffle the .cc methods. This is probably an overkill for this CL.

--- a/chrome/browser/ui/views/frame/browser_view.h
+++ b/chrome/browser/ui/views/frame/browser_view.h
@@ -433,15 +433,15 @@ class BrowserView : public BrowserWindow,
   bool DrawInfoBarArrows(int* x) const override;
 
   // Overridden from views::View:
-  const char* GetClassName() const override;
   void Layout() override;
+  const char* GetClassName() const override;
+  void OnGestureEvent(ui::GestureEvent* event) override;
+  void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
+  void ChildPreferredSizeChanged(View* child) override;
   void ViewHierarchyChanged(
       const ViewHierarchyChangedDetails& details) override;
-  void ChildPreferredSizeChanged(View* child) override;
-  void GetAccessibleNodeData(ui::AXNodeData* node_data) override;
   void OnThemeChanged() override;
   void OnNativeThemeChanged(const ui::NativeTheme* theme) override;
-  void OnGestureEvent(ui::GestureEvent* event) override;
 
   // Overridden from ui::AcceleratorTarget:
   bool AcceleratorPressed(const ui::Accelerator& accelerator) override;

[tapted, 2017/02/15 22:25:52]

Acknowledged.

 
   // Overridden from ui::AcceleratorTarget:
   bool AcceleratorPressed(const ui::Accelerator& accelerator) override;
 
   // OmniboxPopupModelObserver overrides
   void OnOmniboxPopupShownOrHidden() override;
 
   // ExclusiveAccessContext overrides
   Profile* GetProfile() override;
   content::WebContents* GetActiveWebContents() override;
@@ skipping 256 matching lines @@
   // The class that registers for keyboard shortcuts for extension commands.
   std::unique_ptr<ExtensionKeybindingRegistryViews>
       extension_keybinding_registry_;
 
   mutable base::WeakPtrFactory<BrowserView> activate_modal_dialog_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserView);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_FRAME_BROWSER_VIEW_H_