Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(106)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java

Issue 2868403003: added scale animation for context menu (Closed)
Patch Set: y Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java
index 58ae91dc2b7201840ecdd89b6f6820b0564e36c3..7607962707d8c40fd88f4f95ae020e9516d6da3c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ContextMenuParams.java
@@ -43,6 +43,9 @@ public class ContextMenuParams {
private final boolean mIsVideo;
private final boolean mCanSavemedia;
+ private final int mX;
+ private final int mY;
+
/**
* @return The URL associated with the main frame of the page that triggered the context menu.
*/
@@ -134,10 +137,26 @@ public class ContextMenuParams {
return false;
}
+ /**
+ * @return The x-coordinate of the touch that triggered the context menu in dp relative to the
+ * content window; 0 corresponds to the left edge.
+ */
+ public int getX() {
Theresa 2017/05/24 17:53:33 nit: getTriggeringTouchX() or something else more
Daniel Park 2017/05/24 19:44:39 Done.
+ return mX;
+ }
+
+ /**
+ * @return The y-coordinate of the touch that triggered the context menu in dp relative to the
+ * content window; 0 corresponds to the left edge.
+ */
+ public int getY() {
+ return mY;
+ }
+
@VisibleForTesting
ContextMenuParams(int mediaType, String pageUrl, String linkUrl, String linkText,
String unfilteredLinkUrl, String srcUrl, String titleText, boolean imageWasFetchedLoFi,
- Referrer referrer, boolean canSavemedia) {
+ Referrer referrer, boolean canSavemedia, int x, int y) {
mPageUrl = pageUrl;
mLinkUrl = linkUrl;
mLinkText = linkText;
@@ -151,16 +170,18 @@ public class ContextMenuParams {
mIsImage = mediaType == MediaType.MEDIA_TYPE_IMAGE;
mIsVideo = mediaType == MediaType.MEDIA_TYPE_VIDEO;
mCanSavemedia = canSavemedia;
+ mX = x;
+ mY = y;
}
@CalledByNative
private static ContextMenuParams create(int mediaType, String pageUrl, String linkUrl,
String linkText, String unfilteredLinkUrl, String srcUrl, String titleText,
boolean imageWasFetchedLoFi, String sanitizedReferrer, int referrerPolicy,
- boolean canSavemedia) {
+ boolean canSavemedia, int x, int y) {
Referrer referrer = TextUtils.isEmpty(sanitizedReferrer)
? null : new Referrer(sanitizedReferrer, referrerPolicy);
return new ContextMenuParams(mediaType, pageUrl, linkUrl, linkText, unfilteredLinkUrl,
- srcUrl, titleText, imageWasFetchedLoFi, referrer, canSavemedia);
+ srcUrl, titleText, imageWasFetchedLoFi, referrer, canSavemedia, x, y);
}
}

Powered by Google App Engine
This is Rietveld 408576698