Make Mac pinch thresholding apply only to zoom (Chromium side)

content/browser/renderer_host/render_widget_host_view_mac_unittest.mm

 // 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 "content/browser/renderer_host/render_widget_host_view_mac.h"
 
 #include <Cocoa/Cocoa.h>
 
 #include "base/mac/mac_util.h"
 #include "base/mac/scoped_nsautorelease_pool.h"
@@ skipping 848 matching lines @@
   EXPECT_TRUE([view->cocoa_view() isOpaque]);
   set_background = process_host->sink().GetUniqueMessageMatching(
       ViewMsg_SetBackgroundOpaque::ID);
   ASSERT_TRUE(set_background);
   ViewMsg_SetBackgroundOpaque::Read(set_background, &sent_background);
   EXPECT_TRUE(get<0>(sent_background));
 
   host->Shutdown();
 }
 
-TEST_F(RenderWidgetHostViewMacTest, PinchThresholding) {
+class RenderWidgetHostViewMacPinchTest : public RenderWidgetHostViewMacTest {
+ public:
+  RenderWidgetHostViewMacPinchTest() : process_host_(NULL) {}
+
+  bool ZoomDisabledForPinchUpdateMessage() {
+    const IPC::Message* message = NULL;
+    // The first message may be a PinchBegin. Go for the second message if
+    // there are two.
+    switch (process_host_->sink().message_count()) {
+      case 1:
+        message = process_host_->sink().GetMessageAt(0);
+        break;
+      case 2:
+        message = process_host_->sink().GetMessageAt(1);
+        break;
+      default:
+        NOTREACHED();
+        break;
+    }
+    DCHECK(message);
+    Tuple<IPC::WebInputEventPointer, ui::LatencyInfo, bool> data;
+    InputMsg_HandleInputEvent::Read(message, &data);
+    IPC::WebInputEventPointer ipc_event = get<0>(data);
+    const blink::WebGestureEvent* gesture_event =
+        static_cast<const blink::WebGestureEvent*>(ipc_event);
+    return gesture_event->data.pinchUpdate.zoomDisabled;
+  }
+
+  MockRenderProcessHost* process_host_;
+};
+
+TEST_F(RenderWidgetHostViewMacPinchTest, PinchThresholding) {
   // This tests Lion+ functionality, so don't run the test pre-Lion.
   if (!base::mac::IsOSLionOrLater())
     return;
 
   // Initialize the view associated with a MockRenderWidgetHostImpl, rather than
   // the MockRenderProcessHost that is set up by the test harness which mocks
   // out |OnMessageReceived()|.
   TestBrowserContext browser_context;
-  MockRenderProcessHost* process_host =
-      new MockRenderProcessHost(&browser_context);
+  process_host_ = new MockRenderProcessHost(&browser_context);
   MockRenderWidgetHostDelegate delegate;
   MockRenderWidgetHostImpl* host = new MockRenderWidgetHostImpl(
-      &delegate, process_host, MSG_ROUTING_NONE);
+      &delegate, process_host_, MSG_ROUTING_NONE);
   RenderWidgetHostViewMac* view = new RenderWidgetHostViewMac(host, false);
 
   // We'll use this IPC message to ack events.
   InputHostMsg_HandleInputEvent_ACK_Params ack;
   ack.type = blink::WebInputEvent::GesturePinchUpdate;
   ack.state = INPUT_EVENT_ACK_STATE_CONSUMED;
   scoped_ptr<IPC::Message> response(
       new InputHostMsg_HandleInputEvent_ACK(0, ack));
 
   // Do a gesture that crosses the threshold.
   {
     NSEvent* pinchBeginEvent =
         MockGestureEvent(NSEventTypeBeginGesture, 100.1, 0);
     NSEvent* pinchUpdateEvents[3] = {
         MockGestureEvent(NSEventTypeMagnify, 100.2, 0.25),
         MockGestureEvent(NSEventTypeMagnify, 100.3, 0.25),
         MockGestureEvent(NSEventTypeMagnify, 100.4, 0.25),
     };
     NSEvent* pinchEndEvent =
         MockGestureEvent(NSEventTypeEndGesture, 100.5, 0);
 
-    // No messages are sent for the pinch begin and the first update event.
     [view->cocoa_view() beginGestureWithEvent:pinchBeginEvent];
+    EXPECT_EQ(0U, process_host_->sink().message_count());
+
+    // No zoom is sent for the first update event.
     [view->cocoa_view() magnifyWithEvent:pinchUpdateEvents[0]];
-    ASSERT_EQ(0U, process_host->sink().message_count());
+    host->OnMessageReceived(*response);
+    EXPECT_EQ(2U, process_host_->sink().message_count());
+    EXPECT_TRUE(ZoomDisabledForPinchUpdateMessage());
+    process_host_->sink().ClearMessages();
 
-    // The second update event crosses the threshold of 0.4, and so a begin
-    // and update are sent.
+    // The second update event crosses the threshold of 0.4, and so zoom is no
+    // longer disabled.
     [view->cocoa_view() magnifyWithEvent:pinchUpdateEvents[1]];
-    ASSERT_EQ(2U, process_host->sink().message_count());
+    EXPECT_FALSE(ZoomDisabledForPinchUpdateMessage());
     host->OnMessageReceived(*response);
+    EXPECT_EQ(1U, process_host_->sink().message_count());
+    process_host_->sink().ClearMessages();
 
-    // The third update only causes one event to be sent.
+    // The third update still has zoom enabled.
     [view->cocoa_view() magnifyWithEvent:pinchUpdateEvents[2]];
-    ASSERT_EQ(3U, process_host->sink().message_count());
+    EXPECT_FALSE(ZoomDisabledForPinchUpdateMessage());
     host->OnMessageReceived(*response);
+    EXPECT_EQ(1U, process_host_->sink().message_count());
+    process_host_->sink().ClearMessages();
 
-    // As does the end.
     [view->cocoa_view() endGestureWithEvent:pinchEndEvent];
-    ASSERT_EQ(4U, process_host->sink().message_count());
-
-    process_host->sink().ClearMessages();
+    EXPECT_EQ(1U, process_host_->sink().message_count());
+    process_host_->sink().ClearMessages();
   }
 
   // Do a gesture that doesn't cross the threshold, but happens within 1 second,
   // so it should be sent to the renderer.
   {
     NSEvent* pinchBeginEvent =
         MockGestureEvent(NSEventTypeBeginGesture, 101.0, 0);
     NSEvent* pinchUpdateEvent =
         MockGestureEvent(NSEventTypeMagnify, 101.1, 0.25);
     NSEvent* pinchEndEvent =
         MockGestureEvent(NSEventTypeEndGesture, 101.2, 0);
 
-    // No message comes for the begin event.
     [view->cocoa_view() beginGestureWithEvent:pinchBeginEvent];
-    ASSERT_EQ(0U, process_host->sink().message_count());
+    EXPECT_EQ(0U, process_host_->sink().message_count());
 
-    // Two messages come for the first update event.
+    // Expect that a zoom happen because the time threshold has not passed.
     [view->cocoa_view() magnifyWithEvent:pinchUpdateEvent];
-    ASSERT_EQ(2U, process_host->sink().message_count());
+    EXPECT_FALSE(ZoomDisabledForPinchUpdateMessage());
     host->OnMessageReceived(*response);
+    EXPECT_EQ(2U, process_host_->sink().message_count());
+    process_host_->sink().ClearMessages();
 
-    // The end event sends one message.
     [view->cocoa_view() endGestureWithEvent:pinchEndEvent];
-    ASSERT_EQ(3U, process_host->sink().message_count());
-
-    process_host->sink().ClearMessages();
+    EXPECT_EQ(1U, process_host_->sink().message_count());
+    process_host_->sink().ClearMessages();
   }
 
   // Do a gesture that doesn't cross the threshold and happens more than one
   // second later.
   {
     NSEvent* pinchBeginEvent =
         MockGestureEvent(NSEventTypeBeginGesture, 103.0, 0);
     NSEvent* pinchUpdateEvent =
         MockGestureEvent(NSEventTypeMagnify, 103.1, 0.25);
     NSEvent* pinchEndEvent =
         MockGestureEvent(NSEventTypeEndGesture, 103.2, 0);
 
-    // No message comes for the begin event.
     [view->cocoa_view() beginGestureWithEvent:pinchBeginEvent];
-    ASSERT_EQ(0U, process_host->sink().message_count());
+    EXPECT_EQ(0U, process_host_->sink().message_count());
 
-    // Two messages come for the first update event.
+    // Expect that zoom be disabled because the time threshold has passed.
     [view->cocoa_view() magnifyWithEvent:pinchUpdateEvent];
-    ASSERT_EQ(0U, process_host->sink().message_count());
+    EXPECT_EQ(2U, process_host_->sink().message_count());
+    EXPECT_TRUE(ZoomDisabledForPinchUpdateMessage());
+    host->OnMessageReceived(*response);
+    process_host_->sink().ClearMessages();
 
-    // As does the end.
     [view->cocoa_view() endGestureWithEvent:pinchEndEvent];
-    ASSERT_EQ(0U, process_host->sink().message_count());
-
-    process_host->sink().ClearMessages();
+    EXPECT_EQ(1U, process_host_->sink().message_count());
+    process_host_->sink().ClearMessages();
   }
 
   // Clean up.
   host->Shutdown();
 }
 
 
 }  // namespace content