tree 3671e944c1922fa4aaf2185d999c5155532d92f6
parent d6440156201ba7d0229b5e2dc5019d0b6667830e
author Todd Volkert <tvolkert@users.noreply.github.com> 1561161692 -0700
committer GitHub <noreply@github.com> 1561161692 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJdDW/cCRBK7hj4Ov3rIwAAdHIIAAGaqvm23fWb7v+HUkzbxMzM
 eXEG3BF4V2uEwkUnTV0GnvWFuEx1Bt189VNrwFK70483JAqasZ4pfH9i7NeB+rBI
 sfTw2bvPw1SU5vl9M+YthWcwxPeQlKUUv7XrzUsDIrxtwB7uLrJjflQNLaZeAhdx
 /Ctu5WQkoqP0YPWX7beiHWn453RLAtf8MSe5Z03W+MawylOADLX4xQi02pXAxHOM
 lwlrTOAx4UH0wAl1kl51afZTPHdEzbC5QTyJg89NlOvywLOilvTrYSbW6kKWDAwY
 AOrKS7vCUEZaFlAJyq9qUozk/fo1Pg0rJSRHzIc7Wkb9C2/zK0NBByFpxwbwHlM=
 =dx/U
 -----END PGP SIGNATURE-----
 

Prepare for HttpClientResponse Uint8List SDK change (#34863)

* Prepare for HttpClientResponse Uint8List SDK change

An upcoming change in the Dart SDK will change `HttpClientResponse`
from implementing `Stream<List<int>>` to instead implement
`Stream<Uint8List>`.

This forwards-compatible change to `_MockHttpClientResponse` is being
made to allow for a smooth rollout of that SDK breaking change. The
current structure of the class is as follows:

```dart
_MockHttpClientResponse extends Stream<List<int>> implements HttpClientResponse {
  ...
}
```

This structure would require that the Dart SDK change land atomically
a change to the class (`extends Stream<Uint8List>`). This atomic landing
requirement doesn't play well at all with Flutter's roll model vis-a-vis
the Dart SDK's roll model to Google's internal repo.  As such, this commit
changes the structure of `_MockHttpClientResponse` to be:

```dart
_MockHttpClientResponse implements HttpClientResponse {
  final Stream<Uint8List> _delegate;

  ...
}
```

Once the Dart SDK change has fully rolled out, we can simplify this class
back to its former structure.

https://github.com/dart-lang/sdk/issues/36900

* Review comment