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

Unified Diff: views/events/event_aura.cc

Issue 7942004: Consolidate/cleanup event cracking code; single out GdkEvents. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Merge removal of compact nav. Created 9 years, 3 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/events/event.cc ('k') | views/events/event_gtk.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: views/events/event_aura.cc
diff --git a/views/events/event_aura.cc b/views/events/event_aura.cc
index 57c9d973e21756bcbd33fc1231b3a43394605bdb..f5bda061798edb3db9a925dce0d86c4e61b99419 100644
--- a/views/events/event_aura.cc
+++ b/views/events/event_aura.cc
@@ -9,84 +9,24 @@
namespace views {
-namespace {
-
-int GetKeyStateFlags() {
- NOTIMPLEMENTED();
- return 0;
-}
-
-ui::EventType EventTypeFromNative(NativeEvent native_event) {
- return native_event->type();
-}
-
-int EventFlagsFromNative(NativeEvent native_event) {
- return native_event->flags();
-}
-
-}
-
-bool IsClientMouseEvent(const views::NativeEvent& native_event) {
- return true;
-}
-
-bool IsNonClientMouseEvent(const views::NativeEvent& native_event) {
- return false;
-}
-
-////////////////////////////////////////////////////////////////////////////////
-// Event, private:
-
-void Event::Init() {
-}
-
-void Event::InitWithNativeEvent(NativeEvent native_event) {
- native_event_ = native_event;
- // TODO(beng): remove once we rid views of Gtk/Gdk.
- native_event_2_ = NULL;
-}
-
-void Event::InitWithNativeEvent2(NativeEvent2 native_event_2,
- FromNativeEvent2) {
- // No one should ever call this on Aura.
- // TODO(beng): remove once we rid views of Gtk/Gdk.
- NOTREACHED();
- native_event_2_ = NULL;
-}
-
////////////////////////////////////////////////////////////////////////////////
// LocatedEvent, protected:
-LocatedEvent::LocatedEvent(NativeEvent native_event)
+LocatedEvent::LocatedEvent(const NativeEvent& native_event)
: Event(native_event, native_event->type(), native_event->flags()),
location_(static_cast<aura::LocatedEvent*>(native_event)->location()) {
}
-LocatedEvent::LocatedEvent(NativeEvent2 native_event_2,
- FromNativeEvent2 from_native)
- : Event(native_event_2, ui::ET_UNKNOWN, 0, from_native) {
- // No one should ever call this on Windows.
- // TODO(msw): remove once we rid views of Gtk/Gdk.
- NOTREACHED();
-}
-
////////////////////////////////////////////////////////////////////////////////
// KeyEvent, public:
-KeyEvent::KeyEvent(NativeEvent native_event)
- : Event(native_event, native_event->type(), GetKeyStateFlags()),
+KeyEvent::KeyEvent(const NativeEvent& native_event)
+ : Event(native_event, native_event->type(), native_event->flags()),
key_code_(static_cast<aura::KeyEvent*>(native_event)->key_code()),
character_(0),
unmodified_character_(0) {
}
-KeyEvent::KeyEvent(NativeEvent2 native_event_2, FromNativeEvent2 from_native)
- : Event(native_event_2, ui::ET_UNKNOWN, 0, from_native) {
- // No one should ever call this on Windows.
- // TODO(beng): remove once we rid views of Gtk/Gdk.
- NOTREACHED();
-}
-
uint16 KeyEvent::GetCharacter() const {
NOTIMPLEMENTED();
return key_code_;
@@ -98,34 +38,11 @@ uint16 KeyEvent::GetUnmodifiedCharacter() const {
}
////////////////////////////////////////////////////////////////////////////////
-// MouseEvent, public:
-
-MouseEvent::MouseEvent(NativeEvent native_event)
- : LocatedEvent(native_event) {
-}
-
-MouseEvent::MouseEvent(NativeEvent2 native_event_2,
- FromNativeEvent2 from_native)
- : LocatedEvent(native_event_2, from_native) {
- // No one should ever call this on Windows.
- // TODO(msw): remove once we rid views of Gtk/Gdk.
- NOTREACHED();
-}
-
-////////////////////////////////////////////////////////////////////////////////
// MouseWheelEvent, public:
-MouseWheelEvent::MouseWheelEvent(NativeEvent native_event)
+MouseWheelEvent::MouseWheelEvent(const NativeEvent& native_event)
: MouseEvent(native_event),
offset_(0 /* TODO(beng): obtain */) {
}
-MouseWheelEvent::MouseWheelEvent(NativeEvent2 native_event_2,
- FromNativeEvent2 from_native)
- : MouseEvent(native_event_2, from_native) {
- // No one should ever call this on Windows.
- // TODO(msw): remove once we rid views of Gtk/Gdk.
- NOTREACHED();
-}
-
} // namespace views
« no previous file with comments | « views/events/event.cc ('k') | views/events/event_gtk.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698