Remove abstract Clone and Cancel methods from MotionEvent

ui/events/test/motion_event_test_utils.h

Index: ui/events/test/motion_event_test_utils.h
diff --git a/ui/events/test/motion_event_test_utils.h b/ui/events/test/motion_event_test_utils.h
new file mode 100644
index 0000000000000000000000000000000000000000..707c5cbae26717146b79dd8b411a14285103eacc
--- /dev/null
+++ b/ui/events/test/motion_event_test_utils.h
@@ -0,0 +1,24 @@
+// Copyright 2014 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 UI_EVENTS_TEST_MOTION_EVENT_TEST_UTILS_H_
+#define UI_EVENTS_TEST_MOTION_EVENT_TEST_UTILS_H_
+
+#include <ostream>
+#include <string>
+
+namespace ui {
+
+class MotionEvent;
+
+// The methods below are explicitly not defined in production code as they can
+// be relatively expensive and, in the case of equality, contextually confusing.
+
+bool operator==(const MotionEvent& lhs, const MotionEvent& rhs);

[sky, 2014/10/22 00:01:04]

I would rather you have add an explicit ToString function and if tests want to
compare that for equality, go for it.

[jdduke (slow), 2014/10/22 00:15:56]

Sure, that's the behavior I had in PS7 but changed per tdresser@'s
recommendation. I'll change it back.

+
+std::ostream& operator<<(std::ostream& os, const MotionEvent& event);
+
+}  // namespace ui
+
+#endif  // UI_EVENTS_TEST_EVENTS_TEST_UTILS_H_