[Offline Pages] Remove load state from public OfflinePageModel API.

chrome/browser/android/offline_pages/evaluation/offline_page_evaluation_bridge.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/offline_pages/evaluation/offline_page_evaluation_bridge.h"
 
 #include "base/android/callback_android.h"
 #include "base/android/jni_android.h"
 #include "base/android/jni_array.h"
 #include "base/android/jni_string.h"
@@ skipping 194 matching lines @@
     JNIEnv* env,
     content::BrowserContext* browser_context,
     OfflinePageModel* offline_page_model,
     RequestCoordinator* request_coordinator)
     : browser_context_(browser_context),
       offline_page_model_(offline_page_model),
       request_coordinator_(request_coordinator) {
   java_ref_.Reset(Java_OfflinePageEvaluationBridge_create(
       env, reinterpret_cast<jlong>(this)));
 
-  NotifyIfDoneLoading();
   offline_page_model_->AddObserver(this);
   request_coordinator_->AddObserver(this);
   offline_page_model_->GetLogger()->SetClient(this);
   request_coordinator_->GetLogger()->SetClient(this);
 }
 
 OfflinePageEvaluationBridge::~OfflinePageEvaluationBridge() {
   JNIEnv* env = base::android::AttachCurrentThread();
   Java_OfflinePageEvaluationBridge_offlinePageEvaluationBridgeDestroyed(
       env, java_ref_);
 }
 
 // Implement OfflinePageModel::Observer
-void OfflinePageEvaluationBridge::OfflinePageModelLoaded(
-    OfflinePageModel* model) {
-  DCHECK_EQ(offline_page_model_, model);
-  NotifyIfDoneLoading();
-}
-
 void OfflinePageEvaluationBridge::OfflinePageAdded(
     OfflinePageModel* model,
     const OfflinePageItem& added_page) {}
 
 void OfflinePageEvaluationBridge::OfflinePageDeleted(
     int64_t offline_id,
     const ClientId& client_id) {}
 
 // Implement RequestCoordinator::Observer
 void OfflinePageEvaluationBridge::OnAdded(const SavePageRequest& request) {
@@ skipping 85 matching lines @@
     const JavaParamRef<jobject>& obj,
     const JavaParamRef<jlongArray>& j_request_ids,
     const JavaParamRef<jobject>& j_callback_obj) {
   std::vector<int64_t> request_ids;
   base::android::JavaLongArrayToInt64Vector(env, j_request_ids, &request_ids);
   ScopedJavaGlobalRef<jobject> j_callback_ref(j_callback_obj);
   request_coordinator_->RemoveRequests(
       request_ids, base::Bind(&OnRemoveRequestsDone, j_callback_ref));
 }
 
-void OfflinePageEvaluationBridge::NotifyIfDoneLoading() const {
-  if (!offline_page_model_->is_loaded())
-    return;
-  JNIEnv* env = base::android::AttachCurrentThread();
-  Java_OfflinePageEvaluationBridge_offlinePageModelLoaded(env, java_ref_);
-}
-
 }  // namespace android
 }  // namespace offline_pages