Handle the mouse back/forward button in the browser directly
Ensure that the result of mouse up is passed back to the browser so
that it can be executed on the browser side.
BUG=850847,705583
Change-Id: Iebf871c43a9957af1f66a398344efcddc47d7293
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1769525
Commit-Queue: Dave Tapuska <[email protected]>
Reviewed-by: Charlie Reis <[email protected]>
Reviewed-by: Navid Zolghadr <[email protected]>
Cr-Commit-Position: refs/heads/master@{#694770}
diff --git a/content/browser/renderer_host/input/mouse_latency_browsertest.cc b/content/browser/renderer_host/input/mouse_latency_browsertest.cc
index 23caf0a..41ffc93 100644
--- a/content/browser/renderer_host/input/mouse_latency_browsertest.cc
+++ b/content/browser/renderer_host/input/mouse_latency_browsertest.cc
@@ -352,7 +352,7 @@
GetWidgetHost(), blink::WebInputEvent::kMouseUp);
StartTracing();
DoSyncClick(gfx::PointF(100, 100));
- EXPECT_EQ(INPUT_EVENT_ACK_STATE_CONSUMED,
+ EXPECT_EQ(INPUT_EVENT_ACK_STATE_NOT_CONSUMED,
filter->GetAckStateWaitIfNecessary());
const base::Value& trace_data = StopTracing();
diff --git a/content/browser/renderer_host/render_widget_host_delegate.cc b/content/browser/renderer_host/render_widget_host_delegate.cc
index 03f964a..fe8609c 100644
--- a/content/browser/renderer_host/render_widget_host_delegate.cc
+++ b/content/browser/renderer_host/render_widget_host_delegate.cc
@@ -22,6 +22,11 @@
return false;
}
+bool RenderWidgetHostDelegate::HandleMouseEvent(
+ const blink::WebMouseEvent& event) {
+ return false;
+}
+
bool RenderWidgetHostDelegate::HandleWheelEvent(
const blink::WebMouseWheelEvent& event) {
return false;
diff --git a/content/browser/renderer_host/render_widget_host_delegate.h b/content/browser/renderer_host/render_widget_host_delegate.h
index d073e85d7..7894b65 100644
--- a/content/browser/renderer_host/render_widget_host_delegate.h
+++ b/content/browser/renderer_host/render_widget_host_delegate.h
@@ -108,6 +108,11 @@
// Callback to inform the browser that the renderer did not process the
// specified events. This gives an opportunity to the browser to process the
+ // back/forward mouse buttons.
+ virtual bool HandleMouseEvent(const blink::WebMouseEvent& event);
+
+ // Callback to inform the browser that the renderer did not process the
+ // specified events. This gives an opportunity to the browser to process the
// event (used for keyboard shortcuts).
virtual bool HandleKeyboardEvent(const NativeWebKeyboardEvent& event);
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
index d7055ce..1fb9487 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1972,6 +1972,11 @@
for (auto& input_event_observer : input_event_observers_)
input_event_observer.OnInputEventAck(ack_source, ack_result,
mouse_event.event);
+
+ // Give the delegate the ability to handle a mouse event that wasn't consumed
+ // by the renderer. eg. Back/Forward mouse buttons.
+ if (delegate_ && ack_result != INPUT_EVENT_ACK_STATE_CONSUMED && !is_hidden())
+ delegate_->HandleMouseEvent(mouse_event.event);
}
bool RenderWidgetHostImpl::IsMouseLocked() const {
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index ddd8255..6ab3420 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -2372,6 +2372,25 @@
: KeyboardEventProcessingResult::NOT_HANDLED;
}
+bool WebContentsImpl::HandleMouseEvent(const blink::WebMouseEvent& event) {
+ // Handle mouse button back/forward in the browser process after the render
+ // process is done with the event. This ensures all renderer-initiated history
+ // navigations can be treated consistently.
+ if (event.GetType() == blink::WebInputEvent::Type::kMouseUp) {
+ WebContentsImpl* outermost = GetOutermostWebContents();
+ if (event.button == blink::WebPointerProperties::Button::kBack &&
+ outermost->controller_.CanGoBack()) {
+ outermost->controller_.GoBack();
+ return true;
+ } else if (event.button == blink::WebPointerProperties::Button::kForward &&
+ outermost->controller_.CanGoForward()) {
+ outermost->controller_.GoForward();
+ return true;
+ }
+ }
+ return false;
+}
+
bool WebContentsImpl::HandleKeyboardEvent(const NativeWebKeyboardEvent& event) {
if (browser_plugin_embedder_ &&
browser_plugin_embedder_->HandleKeyboardEvent(event)) {
diff --git a/content/browser/web_contents/web_contents_impl.h b/content/browser/web_contents/web_contents_impl.h
index b256fe6..3121f105 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -763,6 +763,7 @@
KeyboardEventProcessingResult PreHandleKeyboardEvent(
const NativeWebKeyboardEvent& event) override;
+ bool HandleMouseEvent(const blink::WebMouseEvent& event) override;
bool HandleKeyboardEvent(const NativeWebKeyboardEvent& event) override;
bool HandleWheelEvent(const blink::WebMouseWheelEvent& event) override;
bool PreHandleGestureEvent(const blink::WebGestureEvent& event) override;
diff --git a/content/browser/web_contents/web_contents_impl_browsertest.cc b/content/browser/web_contents/web_contents_impl_browsertest.cc
index 7ba29e5..5550c69 100644
--- a/content/browser/web_contents/web_contents_impl_browsertest.cc
+++ b/content/browser/web_contents/web_contents_impl_browsertest.cc
@@ -17,6 +17,7 @@
#include "base/strings/pattern.h"
#include "base/strings/utf_string_conversions.h"
#include "base/test/scoped_feature_list.h"
+#include "base/test/test_timeouts.h"
#include "base/threading/thread_restrictions.h"
#include "base/time/time.h"
#include "base/values.h"
@@ -3906,4 +3907,110 @@
EXPECT_THAT(deleted_routing_ids, testing::Contains(frame_routing_id));
}
+IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest, MouseButtonsNavigate) {
+ ASSERT_TRUE(embedded_test_server()->Start());
+ WebContentsImpl* web_contents =
+ static_cast<WebContentsImpl*>(shell()->web_contents());
+
+ GURL url_a = embedded_test_server()->GetURL("a.com", "/title1.html");
+ GURL url_b = embedded_test_server()->GetURL("b.com", "/title1.html");
+
+ EXPECT_TRUE(NavigateToURL(shell(), url_a));
+ EXPECT_TRUE(NavigateToURL(shell(), url_b));
+
+ {
+ TestNavigationObserver back_observer(web_contents);
+ web_contents->GetRenderWidgetHostWithPageFocus()->ForwardMouseEvent(
+ blink::WebMouseEvent(
+ blink::WebInputEvent::kMouseUp, blink::WebFloatPoint(51, 50),
+ blink::WebFloatPoint(51, 50),
+ blink::WebPointerProperties::Button::kBack, 0,
+ blink::WebInputEvent::kNoModifiers, base::TimeTicks::Now()));
+ back_observer.Wait();
+ ASSERT_EQ(url_a, web_contents->GetLastCommittedURL());
+ }
+
+ {
+ TestNavigationObserver forward_observer(web_contents);
+ web_contents->GetRenderWidgetHostWithPageFocus()->ForwardMouseEvent(
+ blink::WebMouseEvent(
+ blink::WebInputEvent::kMouseUp, blink::WebFloatPoint(51, 50),
+ blink::WebFloatPoint(51, 50),
+ blink::WebPointerProperties::Button::kForward, 0,
+ blink::WebInputEvent::kNoModifiers, base::TimeTicks::Now()));
+ forward_observer.Wait();
+ ASSERT_EQ(url_b, web_contents->GetLastCommittedURL());
+ }
+}
+
+IN_PROC_BROWSER_TEST_F(WebContentsImplBrowserTest, MouseButtonsDontNavigate) {
+ // This test injects mouse event listeners in javascript that will
+ // preventDefault the action causing the default navigation action not to be
+ // taken.
+
+ ASSERT_TRUE(embedded_test_server()->Start());
+ WebContentsImpl* web_contents =
+ static_cast<WebContentsImpl*>(shell()->web_contents());
+
+ GURL url_a = embedded_test_server()->GetURL("a.com", "/title1.html");
+ GURL url_b = embedded_test_server()->GetURL("b.com", "/title1.html");
+
+ EXPECT_TRUE(NavigateToURL(shell(), url_a));
+ EXPECT_TRUE(NavigateToURL(shell(), url_b));
+
+ // Prevent default the action.
+ EXPECT_TRUE(content::ExecuteScript(shell(),
+ "document.addEventListener('mouseup', "
+ "event => event.preventDefault());"));
+
+ RenderWidgetHostImpl* render_widget_host =
+ web_contents->GetRenderWidgetHostWithPageFocus();
+ render_widget_host->ForwardMouseEvent(blink::WebMouseEvent(
+ blink::WebInputEvent::kMouseUp, blink::WebFloatPoint(51, 50),
+ blink::WebFloatPoint(51, 50), blink::WebPointerProperties::Button::kBack,
+ 0, blink::WebInputEvent::kNoModifiers, base::TimeTicks::Now()));
+ RunUntilInputProcessed(render_widget_host);
+
+ // Wait an action timeout and assert the URL is correct.
+ {
+ base::RunLoop run_loop;
+ base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
+ FROM_HERE, run_loop.QuitClosure(), TestTimeouts::action_timeout());
+ run_loop.Run();
+ }
+ ASSERT_EQ(url_b, web_contents->GetLastCommittedURL());
+
+ // Move back so we have a forward entry in the history stack so we
+ // can test forward getting canceled.
+ {
+ TestNavigationObserver back_observer(web_contents);
+ web_contents->GetController().GoBack();
+ back_observer.Wait();
+ ASSERT_EQ(url_a, web_contents->GetLastCommittedURL());
+ }
+
+ // Now test the forward button by going back, and injecting the prevention
+ // script.
+ // Prevent default the action.
+ EXPECT_TRUE(content::ExecuteScript(shell(),
+ "document.addEventListener('mouseup', "
+ "event => event.preventDefault());"));
+
+ render_widget_host = web_contents->GetRenderWidgetHostWithPageFocus();
+ render_widget_host->ForwardMouseEvent(blink::WebMouseEvent(
+ blink::WebInputEvent::kMouseUp, blink::WebFloatPoint(51, 50),
+ blink::WebFloatPoint(51, 50),
+ blink::WebPointerProperties::Button::kForward, 0,
+ blink::WebInputEvent::kNoModifiers, base::TimeTicks::Now()));
+ RunUntilInputProcessed(render_widget_host);
+ // Wait an action timeout and assert the URL is correct.
+ {
+ base::RunLoop run_loop;
+ base::ThreadTaskRunnerHandle::Get()->PostDelayedTask(
+ FROM_HERE, run_loop.QuitClosure(), TestTimeouts::action_timeout());
+ run_loop.Run();
+ }
+ ASSERT_EQ(url_a, web_contents->GetLastCommittedURL());
+}
+
} // namespace content
diff --git a/content/renderer/input/render_widget_input_handler.cc b/content/renderer/input/render_widget_input_handler.cc
index 68a99ade..364073b 100644
--- a/content/renderer/input/render_widget_input_handler.cc
+++ b/content/renderer/input/render_widget_input_handler.cc
@@ -376,6 +376,7 @@
}
bool prevent_default = false;
+ bool show_virtual_keyboard_for_mouse = false;
if (WebInputEvent::IsMouseEventType(input_event.GetType())) {
const WebMouseEvent& mouse_event =
static_cast<const WebMouseEvent&>(input_event);
@@ -388,6 +389,11 @@
// time that the mouse enters we always set the cursor accordingly.
if (mouse_event.GetType() == WebInputEvent::kMouseLeave)
current_cursor_.reset();
+
+ if (mouse_event.button == WebPointerProperties::Button::kLeft &&
+ mouse_event.GetType() == WebInputEvent::kMouseUp) {
+ show_virtual_keyboard_for_mouse = true;
+ }
}
if (WebInputEvent::IsKeyboardEventType(input_event.GetType())) {
@@ -494,9 +500,9 @@
// Show the virtual keyboard if enabled and a user gesture triggers a focus
// change.
- if (processed != WebInputEventResult::kNotHandled &&
- (input_event.GetType() == WebInputEvent::kTouchEnd ||
- input_event.GetType() == WebInputEvent::kMouseUp)) {
+ if ((processed != WebInputEventResult::kNotHandled &&
+ input_event.GetType() == WebInputEvent::kTouchEnd) ||
+ show_virtual_keyboard_for_mouse) {
delegate_->ShowVirtualKeyboard();
}
diff --git a/third_party/blink/renderer/core/dom/node.cc b/third_party/blink/renderer/core/dom/node.cc
index e84f01a..fc0b74b3b 100644
--- a/third_party/blink/renderer/core/dom/node.cc
+++ b/third_party/blink/renderer/core/dom/node.cc
@@ -2850,21 +2850,6 @@
frame->GetEventHandler().StartMiddleClickAutoscroll(layout_object);
}
}
- } else if (event_type == event_type_names::kMouseup && event.IsMouseEvent()) {
- auto& mouse_event = ToMouseEvent(event);
- if (mouse_event.button() ==
- static_cast<int16_t>(WebPointerProperties::Button::kBack)) {
- if (LocalFrame* frame = GetDocument().GetFrame()) {
- if (frame->Client()->NavigateBackForward(-1, false))
- event.SetDefaultHandled();
- }
- } else if (mouse_event.button() ==
- static_cast<int16_t>(WebPointerProperties::Button::kForward)) {
- if (LocalFrame* frame = GetDocument().GetFrame()) {
- if (frame->Client()->NavigateBackForward(1, false))
- event.SetDefaultHandled();
- }
- }
}
}
diff --git a/third_party/blink/renderer/core/exported/web_view_impl.cc b/third_party/blink/renderer/core/exported/web_view_impl.cc
index 58e4fbc..4f59a7d 100644
--- a/third_party/blink/renderer/core/exported/web_view_impl.cc
+++ b/third_party/blink/renderer/core/exported/web_view_impl.cc
@@ -439,9 +439,10 @@
// implementation...
}
-void WebViewImpl::HandleMouseUp(LocalFrame& main_frame,
- const WebMouseEvent& event) {
- PageWidgetEventHandler::HandleMouseUp(main_frame, event);
+WebInputEventResult WebViewImpl::HandleMouseUp(LocalFrame& main_frame,
+ const WebMouseEvent& event) {
+ WebInputEventResult result =
+ PageWidgetEventHandler::HandleMouseUp(main_frame, event);
if (GetPage()->GetSettings().GetShowContextMenuOnMouseUp()) {
// Dispatch the contextmenu event regardless of if the click was swallowed.
@@ -449,6 +450,7 @@
if (event.button == WebMouseEvent::Button::kRight)
MouseContextMenu(event);
}
+ return result;
}
WebInputEventResult WebViewImpl::HandleMouseWheel(
diff --git a/third_party/blink/renderer/core/exported/web_view_impl.h b/third_party/blink/renderer/core/exported/web_view_impl.h
index 61d54c30..d1b07190 100644
--- a/third_party/blink/renderer/core/exported/web_view_impl.h
+++ b/third_party/blink/renderer/core/exported/web_view_impl.h
@@ -523,7 +523,7 @@
// PageWidgetEventHandler functions
void HandleMouseLeave(LocalFrame&, const WebMouseEvent&) override;
void HandleMouseDown(LocalFrame&, const WebMouseEvent&) override;
- void HandleMouseUp(LocalFrame&, const WebMouseEvent&) override;
+ WebInputEventResult HandleMouseUp(LocalFrame&, const WebMouseEvent&) override;
WebInputEventResult HandleMouseWheel(LocalFrame&,
const WebMouseWheelEvent&) override;
WebInputEventResult HandleGestureEvent(const WebGestureEvent&) override;
diff --git a/third_party/blink/renderer/core/frame/web_frame_widget_impl.cc b/third_party/blink/renderer/core/frame/web_frame_widget_impl.cc
index 3adb6359..75d1868 100644
--- a/third_party/blink/renderer/core/frame/web_frame_widget_impl.cc
+++ b/third_party/blink/renderer/core/frame/web_frame_widget_impl.cc
@@ -755,9 +755,11 @@
// implementation...
}
-void WebFrameWidgetImpl::HandleMouseUp(LocalFrame& main_frame,
- const WebMouseEvent& event) {
- PageWidgetEventHandler::HandleMouseUp(main_frame, event);
+WebInputEventResult WebFrameWidgetImpl::HandleMouseUp(
+ LocalFrame& main_frame,
+ const WebMouseEvent& event) {
+ WebInputEventResult result =
+ PageWidgetEventHandler::HandleMouseUp(main_frame, event);
if (GetPage()->GetSettings().GetShowContextMenuOnMouseUp()) {
// Dispatch the contextmenu event regardless of if the click was swallowed.
@@ -765,6 +767,7 @@
if (event.button == WebMouseEvent::Button::kRight)
MouseContextMenu(event);
}
+ return result;
}
WebInputEventResult WebFrameWidgetImpl::HandleMouseWheel(
diff --git a/third_party/blink/renderer/core/frame/web_frame_widget_impl.h b/third_party/blink/renderer/core/frame/web_frame_widget_impl.h
index 3ac90b55..e111f8d7 100644
--- a/third_party/blink/renderer/core/frame/web_frame_widget_impl.h
+++ b/third_party/blink/renderer/core/frame/web_frame_widget_impl.h
@@ -158,7 +158,7 @@
// PageWidgetEventHandler functions
void HandleMouseLeave(LocalFrame&, const WebMouseEvent&) override;
void HandleMouseDown(LocalFrame&, const WebMouseEvent&) override;
- void HandleMouseUp(LocalFrame&, const WebMouseEvent&) override;
+ WebInputEventResult HandleMouseUp(LocalFrame&, const WebMouseEvent&) override;
WebInputEventResult HandleMouseWheel(LocalFrame&,
const WebMouseWheelEvent&) override;
WebInputEventResult HandleGestureEvent(const WebGestureEvent&) override;
diff --git a/third_party/blink/renderer/core/input/event_handler.cc b/third_party/blink/renderer/core/input/event_handler.cc
index 1826a4b..5930a3c 100644
--- a/third_party/blink/renderer/core/input/event_handler.cc
+++ b/third_party/blink/renderer/core/input/event_handler.cc
@@ -2335,11 +2335,7 @@
WebInputEventResult EventHandler::PassMouseReleaseEventToSubframe(
MouseEventWithHitTestResults& mev,
LocalFrame* subframe) {
- WebInputEventResult result =
- subframe->GetEventHandler().HandleMouseReleaseEvent(mev.Event());
- if (result != WebInputEventResult::kNotHandled)
- return result;
- return WebInputEventResult::kHandledSystem;
+ return subframe->GetEventHandler().HandleMouseReleaseEvent(mev.Event());
}
void EventHandler::CaptureMouseEventsToWidget(bool capture) {
diff --git a/third_party/blink/renderer/core/input/event_handler_test.cc b/third_party/blink/renderer/core/input/event_handler_test.cc
index dabf396..92406c1 100644
--- a/third_party/blink/renderer/core/input/event_handler_test.cc
+++ b/third_party/blink/renderer/core/input/event_handler_test.cc
@@ -1044,96 +1044,6 @@
ASSERT_FALSE(chrome_client_->ReceivedRequestForUnbufferedInput());
}
-class NavigationCapturingFrameClient : public EmptyLocalFrameClient {
- public:
- NavigationCapturingFrameClient() = default;
-
- bool NavigateBackForward(int offset, bool from_script) const override {
- offset_ = offset;
- return true;
- }
-
- int Offset() const { return offset_; }
-
- private:
- mutable int offset_ = 0;
-};
-
-class EventHandlerNavigationTest : public EventHandlerTest {
- public:
- EventHandlerNavigationTest() = default;
-
- void SetUp() override {
- frame_client_ = MakeGarbageCollected<NavigationCapturingFrameClient>();
- Page::PageClients clients;
- FillWithEmptyClients(clients);
- SetupPageWithClients(&clients, frame_client_);
- }
-
- int Offset() { return frame_client_->Offset(); }
-
- private:
- Persistent<NavigationCapturingFrameClient> frame_client_;
-};
-
-TEST_F(EventHandlerNavigationTest, MouseButtonsNavigate) {
- SetHtmlInnerHTML("<div>");
-
- EXPECT_EQ(0, Offset());
-
- WebMouseEvent mouse_back_event(
- WebInputEvent::kMouseUp, WebFloatPoint(51, 50), WebFloatPoint(51, 50),
- WebPointerProperties::Button::kBack, 0, WebInputEvent::kNoModifiers,
- base::TimeTicks::Now());
- mouse_back_event.SetFrameScale(1);
- GetDocument().GetFrame()->GetEventHandler().HandleMouseReleaseEvent(
- mouse_back_event);
-
- EXPECT_EQ(-1, Offset());
-
- WebMouseEvent mouse_forward_event(
- WebInputEvent::kMouseUp, WebFloatPoint(51, 50), WebFloatPoint(51, 50),
- WebPointerProperties::Button::kForward, 0, WebInputEvent::kNoModifiers,
- base::TimeTicks::Now());
- mouse_forward_event.SetFrameScale(1);
- GetDocument().GetFrame()->GetEventHandler().HandleMouseReleaseEvent(
- mouse_forward_event);
-
- EXPECT_EQ(1, Offset());
-}
-
-TEST_F(EventHandlerNavigationTest, MouseButtonsDontNavigate) {
- SetHtmlInnerHTML("<div>");
- GetDocument().GetSettings()->SetScriptEnabled(true);
- Element* script = GetDocument().CreateRawElement(html_names::kScriptTag);
- script->SetInnerHTMLFromString(
- "document.addEventListener('mouseup', event => "
- "event.preventDefault());");
- GetDocument().body()->AppendChild(script);
-
- EXPECT_EQ(0, Offset());
-
- WebMouseEvent mouse_back_event(
- WebInputEvent::kMouseUp, WebFloatPoint(51, 50), WebFloatPoint(51, 50),
- WebPointerProperties::Button::kBack, 0, WebInputEvent::kNoModifiers,
- base::TimeTicks::Now());
- mouse_back_event.SetFrameScale(1);
- GetDocument().GetFrame()->GetEventHandler().HandleMouseReleaseEvent(
- mouse_back_event);
-
- EXPECT_EQ(0, Offset());
-
- WebMouseEvent mouse_forward_event(
- WebInputEvent::kMouseUp, WebFloatPoint(51, 50), WebFloatPoint(51, 50),
- WebPointerProperties::Button::kForward, 0, WebInputEvent::kNoModifiers,
- base::TimeTicks::Now());
- mouse_forward_event.SetFrameScale(1);
- GetDocument().GetFrame()->GetEventHandler().HandleMouseReleaseEvent(
- mouse_forward_event);
-
- EXPECT_EQ(0, Offset());
-}
-
class EventHandlerScrollbarGestureInjectionTest : public SimTest {
public:
EventHandlerScrollbarGestureInjectionTest() = default;
diff --git a/third_party/blink/renderer/core/page/page_widget_delegate.cc b/third_party/blink/renderer/core/page/page_widget_delegate.cc
index caf005f..6a25df5 100644
--- a/third_party/blink/renderer/core/page/page_widget_delegate.cc
+++ b/third_party/blink/renderer/core/page/page_widget_delegate.cc
@@ -142,8 +142,8 @@
case WebInputEvent::kMouseUp:
if (!root || !root->View())
return WebInputEventResult::kHandledSuppressed;
- handler.HandleMouseUp(*root, static_cast<const WebMouseEvent&>(event));
- return WebInputEventResult::kHandledSystem;
+ return handler.HandleMouseUp(*root,
+ static_cast<const WebMouseEvent&>(event));
case WebInputEvent::kMouseWheel:
if (!root || !root->View())
return WebInputEventResult::kNotHandled;
@@ -239,11 +239,13 @@
main_frame.GetEventHandler().HandleMousePressEvent(transformed_event);
}
-void PageWidgetEventHandler::HandleMouseUp(LocalFrame& main_frame,
- const WebMouseEvent& event) {
+WebInputEventResult PageWidgetEventHandler::HandleMouseUp(
+ LocalFrame& main_frame,
+ const WebMouseEvent& event) {
WebMouseEvent transformed_event =
TransformWebMouseEvent(main_frame.View(), event);
- main_frame.GetEventHandler().HandleMouseReleaseEvent(transformed_event);
+ return main_frame.GetEventHandler().HandleMouseReleaseEvent(
+ transformed_event);
}
WebInputEventResult PageWidgetEventHandler::HandleMouseWheel(
diff --git a/third_party/blink/renderer/core/page/page_widget_delegate.h b/third_party/blink/renderer/core/page/page_widget_delegate.h
index 645d4ac..996e0dc 100644
--- a/third_party/blink/renderer/core/page/page_widget_delegate.h
+++ b/third_party/blink/renderer/core/page/page_widget_delegate.h
@@ -54,7 +54,8 @@
const WebVector<const WebInputEvent*>&);
virtual void HandleMouseLeave(LocalFrame& main_frame, const WebMouseEvent&);
virtual void HandleMouseDown(LocalFrame& main_frame, const WebMouseEvent&);
- virtual void HandleMouseUp(LocalFrame& main_frame, const WebMouseEvent&);
+ virtual WebInputEventResult HandleMouseUp(LocalFrame& main_frame,
+ const WebMouseEvent&);
virtual WebInputEventResult HandleMouseWheel(LocalFrame& main_frame,
const WebMouseWheelEvent&);
virtual WebInputEventResult HandleKeyEvent(const WebKeyboardEvent&) = 0;
