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

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

Issue 2564163002: [Offline Pages] Remove load state from public OfflinePageModel API. (Closed)
Patch Set: Fix broken tests. 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 #ifndef CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_BRIDGE_H_ 5 #ifndef CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_BRIDGE_H_
6 #define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_BRIDGE_H_ 6 #define CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_BRIDGE_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include "base/android/jni_android.h" 10 #include "base/android/jni_android.h"
(...skipping 21 matching lines...) Expand all
32 static base::android::ScopedJavaLocalRef<jobject> ConvertToJavaOfflinePage( 32 static base::android::ScopedJavaLocalRef<jobject> ConvertToJavaOfflinePage(
33 JNIEnv* env, 33 JNIEnv* env,
34 const OfflinePageItem& offline_page); 34 const OfflinePageItem& offline_page);
35 35
36 OfflinePageBridge(JNIEnv* env, 36 OfflinePageBridge(JNIEnv* env,
37 content::BrowserContext* browser_context, 37 content::BrowserContext* browser_context,
38 OfflinePageModel* offline_page_model); 38 OfflinePageModel* offline_page_model);
39 ~OfflinePageBridge() override; 39 ~OfflinePageBridge() override;
40 40
41 // OfflinePageModel::Observer implementation. 41 // OfflinePageModel::Observer implementation.
42 void OfflinePageModelLoaded(OfflinePageModel* model) override;
43 void OfflinePageAdded(OfflinePageModel* model, 42 void OfflinePageAdded(OfflinePageModel* model,
44 const OfflinePageItem& added_page) override; 43 const OfflinePageItem& added_page) override;
45 void OfflinePageDeleted(int64_t offline_id, 44 void OfflinePageDeleted(int64_t offline_id,
46 const ClientId& client_id) override; 45 const ClientId& client_id) override;
47 46
48 void CheckPagesExistOffline( 47 void CheckPagesExistOffline(
49 JNIEnv* env, 48 JNIEnv* env,
50 const base::android::JavaParamRef<jobject>& obj, 49 const base::android::JavaParamRef<jobject>& obj,
51 const base::android::JavaParamRef<jobjectArray>& j_urls_array, 50 const base::android::JavaParamRef<jobjectArray>& j_urls_array,
52 const base::android::JavaParamRef<jobject>& j_callback_obj); 51 const base::android::JavaParamRef<jobject>& j_callback_obj);
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
116 115
117 void RemoveRequestsFromQueue( 116 void RemoveRequestsFromQueue(
118 JNIEnv* env, 117 JNIEnv* env,
119 const base::android::JavaParamRef<jobject>& obj, 118 const base::android::JavaParamRef<jobject>& obj,
120 const base::android::JavaParamRef<jlongArray>& j_request_ids_array, 119 const base::android::JavaParamRef<jlongArray>& j_request_ids_array,
121 const base::android::JavaParamRef<jobject>& j_callback_obj); 120 const base::android::JavaParamRef<jobject>& j_callback_obj);
122 121
123 base::android::ScopedJavaGlobalRef<jobject> java_ref() { return java_ref_; } 122 base::android::ScopedJavaGlobalRef<jobject> java_ref() { return java_ref_; }
124 123
125 private: 124 private:
126 void NotifyIfDoneLoading() const;
127
128 base::android::ScopedJavaLocalRef<jobject> CreateClientId( 125 base::android::ScopedJavaLocalRef<jobject> CreateClientId(
129 JNIEnv* env, 126 JNIEnv* env,
130 const ClientId& clientId) const; 127 const ClientId& clientId) const;
131 128
132 base::android::ScopedJavaGlobalRef<jobject> java_ref_; 129 base::android::ScopedJavaGlobalRef<jobject> java_ref_;
133 // Not owned. 130 // Not owned.
134 content::BrowserContext* browser_context_; 131 content::BrowserContext* browser_context_;
135 // Not owned. 132 // Not owned.
136 OfflinePageModel* offline_page_model_; 133 OfflinePageModel* offline_page_model_;
137 134
138 DISALLOW_COPY_AND_ASSIGN(OfflinePageBridge); 135 DISALLOW_COPY_AND_ASSIGN(OfflinePageBridge);
139 }; 136 };
140 137
141 bool RegisterOfflinePageBridge(JNIEnv* env); 138 bool RegisterOfflinePageBridge(JNIEnv* env);
142 139
143 } // namespace android 140 } // namespace android
144 } // namespace offline_pages 141 } // namespace offline_pages
145 142
146 #endif // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_BRIDGE_H_ 143 #endif // CHROME_BROWSER_ANDROID_OFFLINE_PAGES_OFFLINE_PAGE_BRIDGE_H_
147 144
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698