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

Side by Side Diff: ui/views/widget/root_view.h

Issue 169443005: Fix crash which occurs when a widget destroys itself as a result of ET_GESTURE_TAP_DOWN (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 UI_VIEWS_WIDGET_ROOT_VIEW_H_ 5 #ifndef UI_VIEWS_WIDGET_ROOT_VIEW_H_
6 #define UI_VIEWS_WIDGET_ROOT_VIEW_H_ 6 #define UI_VIEWS_WIDGET_ROOT_VIEW_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
(...skipping 127 matching lines...) Expand 10 before | Expand all | Expand 10 after
138 // cursor during drag operations. The location of the mouse should be in the 138 // cursor during drag operations. The location of the mouse should be in the
139 // current coordinate system (i.e. any necessary transformation should be 139 // current coordinate system (i.e. any necessary transformation should be
140 // applied to the point prior to calling this). 140 // applied to the point prior to calling this).
141 void UpdateCursor(const ui::MouseEvent& event); 141 void UpdateCursor(const ui::MouseEvent& event);
142 142
143 // Updates the last_mouse_* fields from e. The location of the mouse should be 143 // Updates the last_mouse_* fields from e. The location of the mouse should be
144 // in the current coordinate system (i.e. any necessary transformation should 144 // in the current coordinate system (i.e. any necessary transformation should
145 // be applied to the point prior to calling this). 145 // be applied to the point prior to calling this).
146 void SetMouseLocationAndFlags(const ui::MouseEvent& event); 146 void SetMouseLocationAndFlags(const ui::MouseEvent& event);
147 147
148 void DispatchEventToTarget(View* target, ui::Event* event); 148 ui::EventDispatchDetails DispatchEventToTarget(View* target,
149 ui::Event* event);
sadrul 2014/02/19 18:32:34 You should add WARN_UNUSED_RESULT here.
149 150
150 // |view| is the view receiving |event|. This function sends the event to all 151 // |view| is the view receiving |event|. This function sends the event to all
151 // the Views up the hierarchy that has |notify_enter_exit_on_child_| flag 152 // the Views up the hierarchy that has |notify_enter_exit_on_child_| flag
152 // turned on, but does not contain |sibling|. 153 // turned on, but does not contain |sibling|.
153 void NotifyEnterExitOfDescendant(const ui::MouseEvent& event, 154 void NotifyEnterExitOfDescendant(const ui::MouseEvent& event,
154 ui::EventType type, 155 ui::EventType type,
155 View* view, 156 View* view,
156 View* sibling); 157 View* sibling);
157 158
158 // Dispatches the KeyEvent to |view| and ancestors until the event is 159 // Dispatches the KeyEvent to |view| and ancestors until the event is
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
223 // Tracks drag state for a view. 224 // Tracks drag state for a view.
224 View::DragInfo drag_info_; 225 View::DragInfo drag_info_;
225 226
226 DISALLOW_IMPLICIT_CONSTRUCTORS(RootView); 227 DISALLOW_IMPLICIT_CONSTRUCTORS(RootView);
227 }; 228 };
228 229
229 } // namespace internal 230 } // namespace internal
230 } // namespace views 231 } // namespace views
231 232
232 #endif // UI_VIEWS_WIDGET_ROOT_VIEW_H_ 233 #endif // UI_VIEWS_WIDGET_ROOT_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698