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

Side by Side Diff: components/cronet/android/java/src/org/chromium/net/impl/VersionSafeCallbacks.java

Issue 2514783002: [Cronet] Add callback wrapper classes to enforce API version checking. (Closed)
Patch Set: fix JNIAdditionalImport 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
« no previous file with comments | « components/cronet/android/java/src/org/chromium/net/impl/UrlResponseInfoImpl.java ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 package org.chromium.net.impl;
6
7 import org.chromium.net.BidirectionalStream;
8 import org.chromium.net.CronetEngine;
9 import org.chromium.net.CronetException;
10 import org.chromium.net.NetworkQualityRttListener;
11 import org.chromium.net.NetworkQualityThroughputListener;
12 import org.chromium.net.RequestFinishedInfo;
13 import org.chromium.net.UploadDataProvider;
14 import org.chromium.net.UploadDataSink;
15 import org.chromium.net.UrlRequest;
16 import org.chromium.net.UrlRequestException;
17 import org.chromium.net.UrlResponseInfo;
18
19 import java.io.IOException;
20 import java.nio.ByteBuffer;
21 import java.util.concurrent.Executor;
22
23 /**
24 * This class contains wrapper classes for all Cronet API callback/listener clas ses. These classes
25 * only permit callbacks that the version of the client API is known to support. For example, if
26 * version 2 of the API adds a callback onFoo() but the client API this class is implementing is
27 * version 1, these wrapper classes should not call {@code mWrappedCallback.onFo o()} and should
28 * instead silently drop the callback.
29 *
30 * When adding any callback wrapping here, be sure you add the proper version ch eck. Only callbacks
31 * supported in all versions of the API should forgo a version check.
32 */
33 public class VersionSafeCallbacks {
34 /**
35 * Wrap a {@link UrlRequest.Callback} in a version safe manner.
36 */
37 public static final class UrlRequestCallback extends UrlRequest.Callback {
38 private final UrlRequest.Callback mWrappedCallback;
39
40 public UrlRequestCallback(UrlRequest.Callback callback) {
41 mWrappedCallback = callback;
42 }
43
44 @Override
45 public void onRedirectReceived(
46 UrlRequest request, UrlResponseInfo info, String newLocationUrl) throws Exception {
47 mWrappedCallback.onRedirectReceived(request, info, newLocationUrl);
48 }
49
50 @Override
51 public void onResponseStarted(UrlRequest request, UrlResponseInfo info) throws Exception {
52 mWrappedCallback.onResponseStarted(request, info);
53 }
54
55 @Override
56 public void onReadCompleted(UrlRequest request, UrlResponseInfo info, By teBuffer byteBuffer)
57 throws Exception {
58 mWrappedCallback.onReadCompleted(request, info, byteBuffer);
59 }
60
61 @Override
62 public void onSucceeded(UrlRequest request, UrlResponseInfo info) {
63 mWrappedCallback.onSucceeded(request, info);
64 }
65
66 @Override
67 public void onFailed(UrlRequest request, UrlResponseInfo info, UrlReques tException error) {
68 mWrappedCallback.onFailed(request, info, error);
69 }
70
71 @Override
72 public void onCanceled(UrlRequest request, UrlResponseInfo info) {
73 mWrappedCallback.onCanceled(request, info);
74 }
75 }
76
77 /**
78 * Wrap a {@link UrlRequest.StatusListener} in a version safe manner.
79 */
80 public static final class UrlRequestStatusListener extends UrlRequest.Status Listener {
81 private final UrlRequest.StatusListener mWrappedListener;
82
83 public UrlRequestStatusListener(UrlRequest.StatusListener listener) {
84 mWrappedListener = listener;
85 }
86
87 @Override
88 public void onStatus(int status) {
89 mWrappedListener.onStatus(status);
90 }
91 }
92
93 /**
94 * Wrap a {@link BidirectionalStream.Callback} in a version safe manner.
95 */
96 public static final class BidirectionalStreamCallback extends BidirectionalS tream.Callback {
97 private final BidirectionalStream.Callback mWrappedCallback;
98
99 public BidirectionalStreamCallback(BidirectionalStream.Callback callback ) {
100 mWrappedCallback = callback;
101 }
102
103 @Override
104 public void onStreamReady(BidirectionalStream stream) {
105 mWrappedCallback.onStreamReady(stream);
106 }
107
108 @Override
109 public void onResponseHeadersReceived(BidirectionalStream stream, UrlRes ponseInfo info) {
110 mWrappedCallback.onResponseHeadersReceived(stream, info);
111 }
112
113 @Override
114 public void onReadCompleted(BidirectionalStream stream, UrlResponseInfo info,
115 ByteBuffer buffer, boolean endOfStream) {
116 mWrappedCallback.onReadCompleted(stream, info, buffer, endOfStream);
117 }
118
119 @Override
120 public void onWriteCompleted(BidirectionalStream stream, UrlResponseInfo info,
121 ByteBuffer buffer, boolean endOfStream) {
122 mWrappedCallback.onWriteCompleted(stream, info, buffer, endOfStream) ;
123 }
124
125 @Override
126 public void onResponseTrailersReceived(BidirectionalStream stream, UrlRe sponseInfo info,
127 UrlResponseInfo.HeaderBlock trailers) {
128 mWrappedCallback.onResponseTrailersReceived(stream, info, trailers);
129 }
130
131 @Override
132 public void onSucceeded(BidirectionalStream stream, UrlResponseInfo info ) {
133 mWrappedCallback.onSucceeded(stream, info);
134 }
135
136 @Override
137 public void onFailed(
138 BidirectionalStream stream, UrlResponseInfo info, CronetExceptio n error) {
139 mWrappedCallback.onFailed(stream, info, error);
140 }
141
142 @Override
143 public void onCanceled(BidirectionalStream stream, UrlResponseInfo info) {
144 mWrappedCallback.onCanceled(stream, info);
145 }
146 }
147
148 /**
149 * Wrap a {@link UploadDataProvider} in a version safe manner.
150 */
151 public static final class UploadDataProviderWrapper extends UploadDataProvid er {
152 private final UploadDataProvider mWrappedProvider;
153
154 public UploadDataProviderWrapper(UploadDataProvider provider) {
155 mWrappedProvider = provider;
156 }
157
158 @Override
159 public long getLength() throws IOException {
160 return mWrappedProvider.getLength();
161 }
162
163 @Override
164 public void read(UploadDataSink uploadDataSink, ByteBuffer byteBuffer) t hrows IOException {
165 mWrappedProvider.read(uploadDataSink, byteBuffer);
166 }
167
168 @Override
169 public void rewind(UploadDataSink uploadDataSink) throws IOException {
170 mWrappedProvider.rewind(uploadDataSink);
171 }
172
173 @Override
174 public void close() throws IOException {
175 mWrappedProvider.close();
176 }
177 }
178
179 /**
180 * Wrap a {@link RequestFinishedInfo.Listener} in a version safe manner.
181 */
182 public static final class RequestFinishedInfoListener extends RequestFinishe dInfo.Listener {
183 private final RequestFinishedInfo.Listener mWrappedListener;
184
185 public RequestFinishedInfoListener(RequestFinishedInfo.Listener listener ) {
186 super(listener.getExecutor());
187 mWrappedListener = listener;
188 }
189
190 @Override
191 public void onRequestFinished(RequestFinishedInfo requestInfo) {
192 mWrappedListener.onRequestFinished(requestInfo);
193 }
194
195 @Override
196 public Executor getExecutor() {
197 return mWrappedListener.getExecutor();
198 }
199 }
200
201 /**
202 * Wrap a {@link NetworkQualityRttListener} in a version safe manner.
203 * NOTE(pauljensen): Delegates equals() and hashCode() to wrapped listener t o
204 * facilitate looking up by wrapped listener in an ArrayList.indexOf().
205 */
206 public static final class NetworkQualityRttListenerWrapper extends NetworkQu alityRttListener {
207 private final NetworkQualityRttListener mWrappedListener;
208
209 public NetworkQualityRttListenerWrapper(NetworkQualityRttListener listen er) {
210 super(listener.getExecutor());
211 mWrappedListener = listener;
212 }
213
214 @Override
215 public void onRttObservation(int rttMs, long whenMs, int source) {
216 mWrappedListener.onRttObservation(rttMs, whenMs, source);
217 }
218
219 @Override
220 public Executor getExecutor() {
221 return mWrappedListener.getExecutor();
222 }
223
224 @Override
225 public int hashCode() {
226 return mWrappedListener.hashCode();
227 }
228
229 @Override
230 public boolean equals(Object o) {
231 if (o == null || !(o instanceof NetworkQualityRttListenerWrapper)) {
232 return false;
233 }
234 return mWrappedListener.equals(((NetworkQualityRttListenerWrapper) o ).mWrappedListener);
235 }
236 }
237
238 /**
239 * Wrap a {@link NetworkQualityThroughputListener} in a version safe manner.
240 * NOTE(pauljensen): Delegates equals() and hashCode() to wrapped listener t o
241 * facilitate looking up by wrapped listener in an ArrayList.indexOf().
242 */
243 public static final class NetworkQualityThroughputListenerWrapper
244 extends NetworkQualityThroughputListener {
245 private final NetworkQualityThroughputListener mWrappedListener;
246
247 public NetworkQualityThroughputListenerWrapper(NetworkQualityThroughputL istener listener) {
248 super(listener.getExecutor());
249 mWrappedListener = listener;
250 }
251
252 @Override
253 public void onThroughputObservation(int throughputKbps, long whenMs, int source) {
254 mWrappedListener.onThroughputObservation(throughputKbps, whenMs, sou rce);
255 }
256
257 @Override
258 public Executor getExecutor() {
259 return mWrappedListener.getExecutor();
260 }
261
262 @Override
263 public int hashCode() {
264 return mWrappedListener.hashCode();
265 }
266
267 @Override
268 public boolean equals(Object o) {
269 if (o == null || !(o instanceof NetworkQualityThroughputListenerWrap per)) {
270 return false;
271 }
272 return mWrappedListener.equals(
273 ((NetworkQualityThroughputListenerWrapper) o).mWrappedListen er);
274 }
275 }
276
277 /**
278 * Wrap a {@link CronetEngine.Builder.LibraryLoader} in a version safe manne r.
279 */
280 public static final class LibraryLoader extends CronetEngine.Builder.Library Loader {
281 private final CronetEngine.Builder.LibraryLoader mWrappedLoader;
282
283 public LibraryLoader(CronetEngine.Builder.LibraryLoader libraryLoader) {
284 mWrappedLoader = libraryLoader;
285 }
286
287 @Override
288 public void loadLibrary(String libName) {
289 mWrappedLoader.loadLibrary(libName);
290 }
291 }
292 }
OLDNEW
« no previous file with comments | « components/cronet/android/java/src/org/chromium/net/impl/UrlResponseInfoImpl.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698