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

Side by Side Diff: chrome/browser/android/offline_pages/offline_page_bridge.cc

Issue 2469933002: Offline Pages: Replace Observer::OfflinePageModelChanged with OfflinePageAdded. (Closed)
Patch Set: nits Created 4 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/android/offline_pages/offline_page_bridge.h" 5 #include "chrome/browser/android/offline_pages/offline_page_bridge.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <set> 8 #include <set>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 260 matching lines...) Expand 10 before | Expand all | Expand 10 after
271 271
272 // Native shutdown causes the destruction of |this|. 272 // Native shutdown causes the destruction of |this|.
273 Java_OfflinePageBridge_offlinePageBridgeDestroyed(env, java_ref_); 273 Java_OfflinePageBridge_offlinePageBridgeDestroyed(env, java_ref_);
274 } 274 }
275 275
276 void OfflinePageBridge::OfflinePageModelLoaded(OfflinePageModel* model) { 276 void OfflinePageBridge::OfflinePageModelLoaded(OfflinePageModel* model) {
277 DCHECK_EQ(offline_page_model_, model); 277 DCHECK_EQ(offline_page_model_, model);
278 NotifyIfDoneLoading(); 278 NotifyIfDoneLoading();
279 } 279 }
280 280
281 void OfflinePageBridge::OfflinePageModelChanged(OfflinePageModel* model) { 281 void OfflinePageBridge::OfflinePageAdded(OfflinePageModel* model,
282 const OfflinePageItem& added_page) {
282 DCHECK_EQ(offline_page_model_, model); 283 DCHECK_EQ(offline_page_model_, model);
283 JNIEnv* env = base::android::AttachCurrentThread(); 284 JNIEnv* env = base::android::AttachCurrentThread();
284 Java_OfflinePageBridge_offlinePageModelChanged(env, java_ref_); 285
286 Java_OfflinePageBridge_offlinePageAdded(
287 env, java_ref_, ToJavaOfflinePageItem(env, added_page));
285 } 288 }
286 289
287 void OfflinePageBridge::OfflinePageDeleted(int64_t offline_id, 290 void OfflinePageBridge::OfflinePageDeleted(int64_t offline_id,
288 const ClientId& client_id) { 291 const ClientId& client_id) {
289 JNIEnv* env = base::android::AttachCurrentThread(); 292 JNIEnv* env = base::android::AttachCurrentThread();
290 Java_OfflinePageBridge_offlinePageDeleted(env, java_ref_, offline_id, 293 Java_OfflinePageBridge_offlinePageDeleted(env, java_ref_, offline_id,
291 CreateClientId(env, client_id)); 294 CreateClientId(env, client_id));
292 } 295 }
293 296
294 void OfflinePageBridge::CheckPagesExistOffline( 297 void OfflinePageBridge::CheckPagesExistOffline(
(...skipping 273 matching lines...) Expand 10 before | Expand all | Expand 10 after
568 env, ConvertUTF8ToJavaString(env, client_id.name_space), 571 env, ConvertUTF8ToJavaString(env, client_id.name_space),
569 ConvertUTF8ToJavaString(env, client_id.id)); 572 ConvertUTF8ToJavaString(env, client_id.id));
570 } 573 }
571 574
572 bool RegisterOfflinePageBridge(JNIEnv* env) { 575 bool RegisterOfflinePageBridge(JNIEnv* env) {
573 return RegisterNativesImpl(env); 576 return RegisterNativesImpl(env);
574 } 577 }
575 578
576 } // namespace android 579 } // namespace android
577 } // namespace offline_pages 580 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698