Consolidate Widget Event code, other cleanup.

views/widget/widget.h

 // Copyright (c) 2011 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 VIEWS_WIDGET_WIDGET_H_
 #define VIEWS_WIDGET_WIDGET_H_
 #pragma once
 
 #include <vector>
 
@@ skipping 279 matching lines @@
   // _before_ the focus manager/tooltip manager.
   // TODO(beng): remove once we fold those objects onto this one.
   void DestroyRootView();
 
   // TODO(beng): Temporarily provided as a way to associate the subclass'
   //             implementation of NativeWidget with this.
   void set_native_widget(NativeWidget* native_widget) {
     native_widget_ = native_widget;
   }
 
+  // Actually handle mouse events. These functions are called by subclasses who
+  // override the message handlers above to do the actual real work of handling
+  // the event in the View system.
+  bool ProcessMousePressed(const MouseEvent& event);

[Ben Goodger (Google), 2011/03/30 23:22:55]

I want you to rename these methods:

OnMousePressed, OnMouseReleased, OnMouseMoved, OnMouseCaptureLost, and move them
to NativeWidgetDelegate. Then Widget can implement them via that interface.

[msw, 2011/03/31 14:01:18]

Done.

+  bool ProcessMouseReleased(const MouseEvent& event);
+  void ProcessMouseMoved(const MouseEvent& event);
+  void ProcessMouseCaptureLost();
+
   // Used for testing.
   void ReplaceFocusManager(FocusManager* focus_manager);
 
+  // If true, the mouse is currently down.

[Ben Goodger (Google), 2011/03/30 23:22:55]

Can you add a TODO to move these to private:?

[msw, 2011/03/31 14:01:18]

Done.

+  bool is_mouse_down_;
+
+  // The following are used to detect duplicate mouse move events and not
+  // deliver them. Displaying a window may result in the system generating
+  // duplicate move events even though the mouse hasn't moved.
+  bool last_mouse_event_was_move_;
+  gfx::Point last_mouse_event_position_;
+
  private:
   // Refresh the compositor tree. This is called by a View whenever its texture
   // is updated.
   void RefreshCompositeTree();
 
   // Try to create a compositor if one hasn't been created yet. Returns false if
   // a compositor couldn't be created.
   bool EnsureCompositor();
 
   NativeWidget* native_widget_;
@@ skipping 22 matching lines @@
 
   // The compositor for accelerated drawing.
   scoped_refptr<ui::Compositor> compositor_;
 
   DISALLOW_COPY_AND_ASSIGN(Widget);
 };
 
 }  // namespace views
 
 #endif // VIEWS_WIDGET_WIDGET_H_