[Offline pages] Wiring saving and opening offline page

chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java

Index: chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java b/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
index de63c8576e09ada06a297e5f15e3c0eb107138c1..a5af774f336ec02128efa749c96aea6631d2c3a6 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/BookmarksBridge.java
@@ -565,13 +565,20 @@ public class BookmarksBridge {
         return nativeIsEnhancedBookmarksFeatureEnabled(profile);
     }
 
+    /**
+     * Notifies the observer that bookmark model has been loaded.
+     */
+    public void notifyBookmarkModelLoaded() {

[fgorski, 2015/08/13 17:41:50]

this should be protected not public. I think leaving it public allows anyone to
call it, and that will trigger bookmarkModelLoaded on observers regardless if it
is actually loaded.

[jianli, 2015/08/13 22:37:18]

Done.

+        for (BookmarkModelObserver observer : mObservers) {
+            observer.bookmarkModelLoaded();
+        }
+    }
+
     @CalledByNative
     private void bookmarkModelLoaded() {
         mIsNativeBookmarkModelLoaded = true;
 
-        for (BookmarkModelObserver observer : mObservers) {
-            observer.bookmarkModelLoaded();
-        }
+        notifyBookmarkModelLoaded();
 
         if (!mDelayedBookmarkCallbacks.isEmpty()) {
             for (int i = 0; i < mDelayedBookmarkCallbacks.size(); i++) {