Drop crash workaround for scrollbar clicking
An unrelated code change made bokan notice an old crash fix
workaround from 2007[1] that should not be needed anymore.
Save some code complexity by removing that workaround.
[1] https://chromium.googlesource.com/chromium/src/+/eeca720ce09f9b6b22f23041b332a46ab42030a1
Change-Id: Ie6de426e8c0c13c7ce6b90fa3554f4fbf7c03366
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1497142
Reviewed-by: David Bokan <bokan@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#637339}diff --git a/third_party/blink/renderer/core/input/event_handler.cc b/third_party/blink/renderer/core/input/event_handler.cc
index 1dc75d8..965e7a9 100644
--- a/third_party/blink/renderer/core/input/event_handler.cc
+++ b/third_party/blink/renderer/core/input/event_handler.cc
@@ -748,20 +748,6 @@
mouse_event_manager_->SetCapturesDragging(false);
}
- // If the hit testing originally determined the event was in a scrollbar,
- // refetch the MouseEventWithHitTestResults in case the scrollbar
- // EmbeddedContentView was destroyed when the mouse event was handled.
- if (mev.GetScrollbar()) {
- if (mev.GetScrollbar() == last_scrollbar_under_mouse_.Get()) {
- HitTestRequest read_only_request(HitTestRequest::kReadOnly |
- HitTestRequest::kActive);
- mev = frame_->GetDocument()->PerformMouseEventHitTest(
- read_only_request, document_point, mouse_event);
- if (mev.GetScrollbar() != last_scrollbar_under_mouse_.Get())
- last_scrollbar_under_mouse_ = nullptr;
- }
- }
-
// Scrollbars should get events anyway, even disabled controls might be
// scrollable.
if (PassMousePressEventToScrollbar(mev))
