Commit e25cfc08 authored by Xida Chen's avatar Xida Chen Committed by Commit Bot

[Code cleanup] Change screenPosition to screen_position in RenderWidgetHostViewGuest

This CL changes the variable name only, should not introduce any behavior
change.

TBR=alexmos@chromium.org

Bug: None
Change-Id: I0e249eb585bc8e2953bef51f93995eab18d73937
Reviewed-on: https://chromium-review.googlesource.com/1176272Reviewed-by: default avatarXida Chen <xidachen@chromium.org>
Commit-Queue: Xida Chen <xidachen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#584291}
parent 14be2cfe
......@@ -622,15 +622,15 @@ RenderWidgetHostViewGuest::GetOwnerRenderWidgetHostView() const {
void RenderWidgetHostViewGuest::MaybeSendSyntheticTapGestureForTest(
const blink::WebFloatPoint& position,
const blink::WebFloatPoint& screenPosition) const {
MaybeSendSyntheticTapGesture(position, screenPosition);
const blink::WebFloatPoint& screen_position) const {
MaybeSendSyntheticTapGesture(position, screen_position);
}
// TODO(wjmaclean): When we remove BrowserPlugin, delete this code.
// http://crbug.com/533069
void RenderWidgetHostViewGuest::MaybeSendSyntheticTapGesture(
const blink::WebFloatPoint& position,
const blink::WebFloatPoint& screenPosition) const {
const blink::WebFloatPoint& screen_position) const {
if (!HasFocus()) {
// We need to a account for the position of the guest view within the
// embedder, as well as the fact that the embedder's host will add its
......@@ -646,7 +646,7 @@ void RenderWidgetHostViewGuest::MaybeSendSyntheticTapGesture(
blink::kWebGestureDeviceTouchscreen);
gesture_tap_event.SetPositionInWidget(
blink::WebFloatPoint(position.x + offset.x(), position.y + offset.y()));
gesture_tap_event.SetPositionInScreen(screenPosition);
gesture_tap_event.SetPositionInScreen(screen_position);
// The touch action may not be set yet because this is still at the
// Pre-processing stage of a mouse or a touch event. In this case, set the
// touch action to Auto to prevent crashing.
......
......@@ -164,7 +164,7 @@ class CONTENT_EXPORT RenderWidgetHostViewGuest
void MaybeSendSyntheticTapGestureForTest(
const blink::WebFloatPoint& position,
const blink::WebFloatPoint& screenPosition) const;
const blink::WebFloatPoint& screen_position) const;
private:
friend class RenderWidgetHostView;
......@@ -187,7 +187,7 @@ class CONTENT_EXPORT RenderWidgetHostViewGuest
// http://crbug.com/533069
void MaybeSendSyntheticTapGesture(
const blink::WebFloatPoint& position,
const blink::WebFloatPoint& screenPosition) const;
const blink::WebFloatPoint& screen_position) const;
void OnHandleInputEvent(RenderWidgetHostImpl* embedder,
int browser_plugin_instance_id,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment