[Cronet] Make UrlResponseInfo constuctor public for testing

[Cronet] Make UrlResponseInfo constuctor public for testing

Also, add a test for the UrlResponseInfo API.

R=mef

Committed: https://crrev.com/826bb4293eb8e01c5fa6c7fe6d4a44de6e5b5a53
Cr-Commit-Position: refs/heads/master@{#357109}

[pauljensen, 2015-10-30 15:14:10]

Misha, PTAL.

[mef, 2015-10-30 15:24:19]

lgtm with nits.

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
File components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java
(right):

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java:80: *
Creates a {@link UrlResponseInfo} object.  Provied to facilitate testing.
Provied -> Provided or Public?

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java:82: *
@param urlChain the URL chain. The first entry is the origianlly requested URL;
origianlly

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java:115: *
Returns the URL chain. The first entry is the origianlly requested URL;
origianlly -> originally

[pauljensen, 2015-10-30 15:29:19]

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
File components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java
(right):

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java:80: *
Creates a {@link UrlResponseInfo} object.  Provied to facilitate testing.
On 2015/10/30 15:24:19, mef wrote:
> Provied -> Provided or Public?

Done.

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java:82: *
@param urlChain the URL chain. The first entry is the origianlly requested URL;
On 2015/10/30 15:24:19, mef wrote:
> origianlly

Done.

https://codereview.chromium.org/1419033006/diff/1/components/cronet/android/a...
components/cronet/android/api/src/org/chromium/net/UrlResponseInfo.java:115: *
Returns the URL chain. The first entry is the origianlly requested URL;
On 2015/10/30 15:24:19, mef wrote:
> origianlly -> originally

Done.

[pauljensen, 2015-10-30 15:29:27]

The CQ bit was checked by pauljensen@chromium.org

[pauljensen, 2015-10-30 15:29:27]

The patchset sent to the CQ was uploaded after l-g-t-m from mef@chromium.org
Link to the patchset: https://codereview.chromium.org/1419033006/#ps20001
(title: "address nits")

[commit-bot: I haz the power, 2015-10-30 15:29:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1419033006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1419033006/20001

[commit-bot: I haz the power, 2015-10-30 15:41:13]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-30 15:41:55]

Patchset 2 (id:??) landed as
https://crrev.com/826bb4293eb8e01c5fa6c7fe6d4a44de6e5b5a53
Cr-Commit-Position: refs/heads/master@{#357109}