Skip to content

Commit

Permalink
content: Remove unused RenderViewObserver::DidHandleTouchEvent.
Browse files Browse the repository at this point in the history
BUG=none

Review URL: https://codereview.chromium.org/1101243002

Cr-Commit-Position: refs/heads/master@{#326806}
  • Loading branch information
sadrulhc authored and Commit bot committed Apr 24, 2015
1 parent 2f6155f commit 41b4ada
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 12 deletions.
1 change: 0 additions & 1 deletion content/public/renderer/render_view_observer.h
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,6 @@ class CONTENT_EXPORT RenderViewObserver : public IPC::Listener,

// These match the RenderView methods.
virtual void DidHandleMouseEvent(const blink::WebMouseEvent& event) {}
virtual void DidHandleTouchEvent(const blink::WebTouchEvent& event) {}
virtual void DidHandleGestureEvent(const blink::WebGestureEvent& event) {}

// These match incoming IPCs.
Expand Down
4 changes: 0 additions & 4 deletions content/renderer/render_view_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -3075,10 +3075,6 @@ void RenderViewImpl::DidHandleMouseEvent(const WebMouseEvent& event) {
FOR_EACH_OBSERVER(RenderViewObserver, observers_, DidHandleMouseEvent(event));
}

void RenderViewImpl::DidHandleTouchEvent(const WebTouchEvent& event) {
FOR_EACH_OBSERVER(RenderViewObserver, observers_, DidHandleTouchEvent(event));
}

bool RenderViewImpl::HasTouchEventHandlersAt(const gfx::Point& point) const {
if (!webview())
return false;
Expand Down
1 change: 0 additions & 1 deletion content/renderer/render_view_impl.h
Original file line number Diff line number Diff line change
Expand Up @@ -467,7 +467,6 @@ class CONTENT_EXPORT RenderViewImpl
bool WillHandleMouseEvent(const blink::WebMouseEvent& event) override;
bool WillHandleGestureEvent(const blink::WebGestureEvent& event) override;
void DidHandleMouseEvent(const blink::WebMouseEvent& event) override;
void DidHandleTouchEvent(const blink::WebTouchEvent& event) override;
bool HasTouchEventHandlersAt(const gfx::Point& point) const override;
void OnSetFocus(bool enable) override;
void OnWasHidden() override;
Expand Down
2 changes: 0 additions & 2 deletions content/renderer/render_widget.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1289,8 +1289,6 @@ void RenderWidget::OnHandleInputEvent(const blink::WebInputEvent* input_event,
DidHandleKeyEvent();
if (WebInputEvent::isMouseEventType(input_event->type))
DidHandleMouseEvent(*(static_cast<const WebMouseEvent*>(input_event)));
if (WebInputEvent::isTouchEventType(input_event->type))
DidHandleTouchEvent(*(static_cast<const WebTouchEvent*>(input_event)));
}

// TODO(rouslan): Fix ChromeOS and Windows 8 behavior of autofill popup with
Expand Down
4 changes: 0 additions & 4 deletions content/renderer/render_widget.h
Original file line number Diff line number Diff line change
Expand Up @@ -561,10 +561,6 @@ class CONTENT_EXPORT RenderWidget
// just handled.
virtual void DidHandleMouseEvent(const blink::WebMouseEvent& event) {}

// Called by OnHandleInputEvent() to notify subclasses that a touch event was
// just handled.
virtual void DidHandleTouchEvent(const blink::WebTouchEvent& event) {}

// Called by OnHandleInputEvent() to forward a mouse wheel event to the
// compositor thread, to effect the elastic overscroll effect.
void ObserveWheelEventAndResult(const blink::WebMouseWheelEvent& wheel_event,
Expand Down

0 comments on commit 41b4ada

Please sign in to comment.