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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java

Issue 2459033002: 📰 Add a context menu to remove status items (Closed)
Patch Set: close the menu when the view is detached Created 4 years, 1 month 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/ntp/MostVisitedItem.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java
index 27b248c76b0f03b39f3a81fc470fb204de106c1f..f4d0792b83f16578397e53d543fd37ca73945064 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/MostVisitedItem.java
@@ -40,6 +40,7 @@
private int mSource;
private View mView;
private TouchDisableableView mWrapperView;
+ private NewTabPageLayout mNewTabPageLayout;
/**
* Constructs a MostVisitedItem with the given manager, title, URL, whitelist icon path, index,
@@ -70,10 +71,12 @@ public MostVisitedItem(NewTabPageManager manager, String title, String url,
* Sets the view that will display this item. MostVisitedItem will handle clicks on the view.
* This should be called exactly once.
*/
- public void initView(View view, TouchDisableableView parent) {
+ public void initView(
+ View view, TouchDisableableView parent, NewTabPageLayout newTabPageLayout) {
assert mView == null;
mView = view;
mWrapperView = parent;
+ mNewTabPageLayout = newTabPageLayout;
mView.setOnClickListener(this);
mView.setOnCreateContextMenuListener(this);
}
@@ -154,7 +157,7 @@ public int getSource() {
@Override
public void onCreateContextMenu(ContextMenu menu, View v, ContextMenuInfo menuInfo) {
new ContextMenuHandler(mManager, mWrapperView, this)
- .onCreateContextMenu(menu);
+ .onCreateContextMenu(menu, mNewTabPageLayout);
}
@Override

Powered by Google App Engine
This is Rietveld 408576698