Replacing WM_TOUCH with WM_POINTER for touch events on Wins 8+

chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc

 // Copyright (c) 2012 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.
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
@@ skipping 38 matching lines @@
 #include "ui/base/clipboard/clipboard.h"
 #include "ui/base/test/ui_controls.h"
 #include "ui/events/keycodes/keyboard_codes.h"
 #include "ui/views/controls/button/menu_button.h"
 #include "ui/views/controls/menu/menu_controller.h"
 #include "ui/views/controls/menu/menu_item_view.h"
 #include "ui/views/controls/menu/submenu_view.h"
 #include "ui/views/widget/widget.h"
 
 #if defined(OS_WIN)
+#include "base/win/windows_version.h"
 #include "ui/aura/window_tree_host.h"
 #endif
 
 using base::ASCIIToUTF16;
 using bookmarks::BookmarkModel;
 using bookmarks::BookmarkNode;
 using content::BrowserThread;
 using content::OpenURLParams;
 using content::PageNavigator;
 using content::WebContents;
@@ skipping 153 matching lines @@
 
   const std::vector<GURL>& urls() const { return urls_; }
   GURL last_url() const { return urls_.empty() ? GURL() : urls_.back(); }
 
  private:
   std::vector<GURL> urls_;
 
   DISALLOW_COPY_AND_ASSIGN(TestingPageNavigator);
 };
 
+class TouchEventHandler : public ui::EventHandler {
+ public:
+  TouchEventHandler() : num_touch_presses_(0), num_touches_(0) {}
+
+  ~TouchEventHandler() override {}
+
+  void WaitForEvents() {
+    base::MessageLoopForUI* loop = base::MessageLoopForUI::current();
+    base::MessageLoopForUI::ScopedNestableTaskAllower allow_nested(loop);
+    base::RunLoop run_loop;
+    quit_closure_ = run_loop.QuitClosure();
+    run_loop.Run();
+  }
+
+  int num_touch_presses() const { return num_touch_presses_; }
+
+  int num_touches() const { return num_touches_; }
+
+ private:
+  // ui::EventHandler:
+  void OnTouchEvent(ui::TouchEvent* event) override {
+    switch (event->type()) {
+      case ui::ET_TOUCH_PRESSED:
+        num_touch_presses_++;
+        num_touches_++;
+        break;
+      case ui::ET_TOUCH_RELEASED:
+        num_touches_--;
+        if (!quit_closure_.is_null() && num_touches_ == 0)
+          quit_closure_.Run();
+        break;
+      default:
+        break;
+    }
+  }
+
+  int num_touch_presses_;
+  int num_touches_;
+  base::Closure quit_closure_;
+  DISALLOW_COPY_AND_ASSIGN(TouchEventHandler);
+};
+
 // TODO(erg): Fix bookmark DND tests on linux_aura. crbug.com/163931
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
 #define MAYBE(x) DISABLED_##x
 #else
 #define MAYBE(x) x
 #endif
 
 }  // namespace
 
 // Base class for event generating bookmark view tests. These test are intended
@@ skipping 2136 matching lines @@
     ASSERT_EQ(2u, navigator_.urls().size());
     EXPECT_EQ(navigator_.urls()[0],
               model_->bookmark_bar_node()->GetChild(0)->GetChild(0)->url());
     EXPECT_EQ(navigator_.urls()[1],
               model_->bookmark_bar_node()->GetChild(0)->GetChild(2)->url());
     Done();
   }
 };
 
 VIEW_TEST(BookmarkBarViewTest27, MiddleClickOnFolderOpensAllBookmarks);
+
+#if defined(OS_WIN)
+class BookmarkBarViewTest28 : public BookmarkBarViewEventTestBase {

[sky, 2017/06/22 20:54:21]

This test has *nothing* to do with bookmark bars, it should be in a file closer
to the changes. Ideally it would be in hwnd_message_handler_interactive_uitest,
but because you are using ViewEventTestBase that isn't possible. How about
putting this in chrome/browser/ui/views/touch_events_interactive_uitest_win.cc.

[lanwei, 2017/06/23 20:06:05]

Done.

+ protected:
+  void DoTestOnMessageLoop() override {
+    views::LabelButton* button = GetBookmarkButton(0);
+    GetWidget()->GetNativeWindow()->GetHost()->window()->AddPreTargetHandler(
+        &touch_event_handler_);
+
+    gfx::Point in_content(button->width() / 2, button->height() / 2);
+    views::View::ConvertPointToScreen(button, &in_content);
+    ui_controls::SendTouchEventsNotifyWhenDone(
+        ui_controls::PRESS, 3, in_content.x(), in_content.y(),
+        CreateEventTask(this, &BookmarkBarViewTest28::Step2));
+  }
+
+ private:
+  void Step2() {
+    if (base::win::GetVersion() <= base::win::VERSION_WIN7) {
+      Done();
+      return;
+    }
+
+    touch_event_handler_.WaitForEvents();
+    ASSERT_EQ(3, touch_event_handler_.num_touch_presses());
+    ASSERT_EQ(0, touch_event_handler_.num_touches());
+    GetWidget()->GetNativeWindow()->GetHost()->window()->RemovePreTargetHandler(
+        &touch_event_handler_);
+    Done();
+  }
+  TouchEventHandler touch_event_handler_;
+};
+
+VIEW_TEST(BookmarkBarViewTest28, CheckWindowsNativeMessageForTouchEvents);
+#endif