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

Unified Diff: views/widget/widget.cc

Issue 6756043: Consolidate Widget Event code, other cleanup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Sync and merge. Created 9 years, 9 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
« no previous file with comments | « views/widget/widget.h ('k') | views/widget/widget_gtk.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: views/widget/widget.cc
diff --git a/views/widget/widget.cc b/views/widget/widget.cc
index 65382c49c766e43c83d13ec18563e76664e52a3a..09a9eb3f643382993adba2117b77339b2381b45d 100644
--- a/views/widget/widget.cc
+++ b/views/widget/widget.cc
@@ -47,7 +47,9 @@ Widget::CreateParams::CreateParams(Type type)
// Widget, public:
Widget::Widget()
- : native_widget_(NULL),
+ : is_mouse_button_pressed_(false),
+ last_mouse_event_was_move_(false),
+ native_widget_(NULL),
widget_delegate_(NULL),
dragged_view_(NULL) {
}
@@ -303,6 +305,56 @@ void Widget::OnNativeWidgetPaint(gfx::Canvas* canvas) {
RefreshCompositeTree();
}
+bool Widget::OnMouseEvent(const MouseEvent& event) {
+ switch (event.type()) {
+ case ui::ET_MOUSE_PRESSED:
+ last_mouse_event_was_move_ = false;
+ if (GetRootView()->OnMousePressed(event)) {
+ is_mouse_button_pressed_ = true;
+ if (!native_widget_->HasMouseCapture())
+ native_widget_->SetMouseCapture();
+ return true;
+ }
+ return false;
+ case ui::ET_MOUSE_RELEASED:
+ last_mouse_event_was_move_ = false;
+ is_mouse_button_pressed_ = false;
+ // Release capture first, to avoid confusion if OnMouseReleased blocks.
+ if (native_widget_->HasMouseCapture() &&
+ ShouldReleaseCaptureOnMouseReleased()) {
+ native_widget_->ReleaseMouseCapture();
+ }
+ GetRootView()->OnMouseReleased(event);
+ return (event.flags() & ui::EF_IS_NON_CLIENT) ? false : true;
+ case ui::ET_MOUSE_MOVED:
+ case ui::ET_MOUSE_DRAGGED:
+ if (native_widget_->HasMouseCapture() && is_mouse_button_pressed_) {
+ last_mouse_event_was_move_ = false;
+ GetRootView()->OnMouseDragged(event);
+ } else if (!last_mouse_event_was_move_ ||
+ last_mouse_event_position_ != event.location()) {
+ last_mouse_event_position_ = event.location();
+ last_mouse_event_was_move_ = true;
+ GetRootView()->OnMouseMoved(event);
+ }
+ return false;
+ case ui::ET_MOUSE_EXITED:
+ last_mouse_event_was_move_ = false;
+ GetRootView()->OnMouseExited(event);
+ return false;
+ default:
+ return false;
+ }
+ return true;
+}
+
+void Widget::OnMouseCaptureLost() {
+ if (is_mouse_button_pressed_)
+ GetRootView()->OnMouseCaptureLost();
+ is_mouse_button_pressed_ = false;
+}
+
+
////////////////////////////////////////////////////////////////////////////////
// Widget, FocusTraversable implementation:
« no previous file with comments | « views/widget/widget.h ('k') | views/widget/widget_gtk.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698