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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/browseractions/BrowserActionActivity.java

Issue 2963583003: Remove special casing for context menu animation for BrowserActions. (Closed)
Patch Set: Fix findbugs Created 3 years, 6 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
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/widget/ContextMenuDialog.java » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/browseractions/BrowserActionActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/browseractions/BrowserActionActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/browseractions/BrowserActionActivity.java
index de634b39efa9930ba3656cc24a0e8b28903f20f4..3f963c9312124bc2387c6e2ae0e14ffa443fa7ab 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/browseractions/BrowserActionActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/browseractions/BrowserActionActivity.java
@@ -7,6 +7,7 @@ package org.chromium.chrome.browser.browseractions;
import android.app.PendingIntent;
import android.content.Intent;
import android.graphics.Bitmap;
+import android.graphics.Point;
import android.net.Uri;
import android.os.Bundle;
import android.support.customtabs.browseractions.BrowserActionItem;
@@ -99,9 +100,16 @@ public class BrowserActionActivity extends AsyncInitializationActivity {
*/
private ContextMenuParams createContextMenuParams() {
Referrer referrer = IntentHandler.constructValidReferrerForAuthority(mCreatorPackageName);
+
+ Point displaySize = new Point();
+ getWindowManager().getDefaultDisplay().getSize(displaySize);
+ float density = getResources().getDisplayMetrics().density;
+ int touchX = (int) ((displaySize.x / 2f) / density);
+ int touchY = (int) ((displaySize.y / 2f) / density);
+
return new ContextMenuParams(mType, mUri.toString(), mUri.toString(), mUri.toString(),
mUri.toString(), mUri.toString(), mUri.toString(), false /* imageWasFetchedLoFi */,
- referrer, false /* canSaveMedia */, 0 /* touchPointXDp */, 0 /* touchPointYDp */);
+ referrer, false /* canSaveMedia */, touchX, touchY);
}
@Override
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/widget/ContextMenuDialog.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698