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

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

Issue 1277663002: Allow only one bookmark to be added for multiple fast starring (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@2454
Patch Set: Created 5 years, 4 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/enhancedbookmarks/EnhancedBookmarkUtils.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/ChromeActivity.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java b/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
index c7e8d3593157126d6ab9fd3dd1fe54e1f4a3f13c..10c0543023482fc294795ffbaa83867ba45c051d 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java
@@ -115,8 +115,6 @@ import org.chromium.chrome.browser.toolbar.ToolbarManager;
import org.chromium.chrome.browser.util.FeatureUtilities;
import org.chromium.chrome.browser.webapps.AddToHomescreenDialog;
import org.chromium.chrome.browser.widget.ControlContainer;
-import org.chromium.components.bookmarks.BookmarkId;
-import org.chromium.components.bookmarks.BookmarkType;
import org.chromium.content.browser.ContentReadbackHandler;
import org.chromium.content.browser.ContentReadbackHandler.GetBitmapCallback;
import org.chromium.content.browser.ContentViewCore;
@@ -193,6 +191,7 @@ public abstract class ChromeActivity extends AsyncInitializationActivity
private ChromeAppMenuPropertiesDelegate mAppMenuPropertiesDelegate;
private AppMenuHandler mAppMenuHandler;
private ToolbarManager mToolbarManager;
+ private BookmarkModelObserver mBookmarkObserver;
// Time in ms that it took took us to inflate the initial layout
private long mInflateInitialLayoutDurationMs;
@@ -914,28 +913,22 @@ public abstract class ChromeActivity extends AsyncInitializationActivity
if (EnhancedBookmarkUtils.isEnhancedBookmarkEnabled(tabToBookmark.getProfile())) {
final EnhancedBookmarksModel bookmarkModel = new EnhancedBookmarksModel();
+ if (bookmarkModel.isBookmarkModelLoaded()) {
+ EnhancedBookmarkUtils.addOrEditBookmark(bookmarkId, bookmarkModel,
+ tabToBookmark, getSnackbarManager(), ChromeActivity.this);
+ } else if (mBookmarkObserver == null) {
+ mBookmarkObserver = new BookmarkModelObserver() {
+ @Override
+ public void bookmarkModelChanged() {}
- BookmarkModelObserver modelObserver = new BookmarkModelObserver() {
- @Override
- public void bookmarkModelChanged() {}
-
- @Override
- public void bookmarkModelLoaded() {
- if (bookmarkId == ChromeBrowserProviderClient.INVALID_BOOKMARK_ID) {
- EnhancedBookmarkUtils.addBookmarkAndShowSnackbar(bookmarkModel,
+ @Override
+ public void bookmarkModelLoaded() {
+ EnhancedBookmarkUtils.addOrEditBookmark(bookmarkId, bookmarkModel,
tabToBookmark, getSnackbarManager(), ChromeActivity.this);
- } else {
- EnhancedBookmarkUtils.startEditActivity(ChromeActivity.this,
- new BookmarkId(bookmarkId, BookmarkType.NORMAL));
+ bookmarkModel.removeModelObserver(this);
}
- bookmarkModel.removeModelObserver(this);
- }
- };
-
- if (bookmarkModel.isBookmarkModelLoaded()) {
- modelObserver.bookmarkModelLoaded();
- } else {
- bookmarkModel.addModelObserver(modelObserver);
+ };
+ bookmarkModel.addModelObserver(mBookmarkObserver);
}
} else {
Intent intent = new Intent(this, ManageBookmarkActivity.class);
« no previous file with comments | « no previous file | chrome/android/java/src/org/chromium/chrome/browser/enhancedbookmarks/EnhancedBookmarkUtils.java » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698