Add elapsed time and timeout value to feedback data.

Add elapsed time and timeout value to feedback data.

To help identifying issues with the connectivity feedback data, this CL
adds data about timeouts and the elapsed time before the data was
gathered.

Some of the flows where this data is used, there is a very short or no
delay between starting the connectivity checks and gathering the results
so this should help identify such cases.

BUG=386395
Committed: https://crrev.com/94cf3373f7a8d3d40f47be0f25bd0f30275a38d2
Cr-Commit-Position: refs/heads/master@{#334084}

[nyquist, 2015-06-11 18:59:22]

nyquist@chromium.org changed reviewers:
+ dfalcantara@chromium.org

[nyquist, 2015-06-11 18:59:23]

dfalcantara: PTAL

[gone, 2015-06-11 20:21:38]

On 2015/06/11 18:59:23, nyquist wrote:
> dfalcantara: PTAL

Remove the @deprecated in this CL?

[gone, 2015-06-11 20:26:08]

lgtm

https://codereview.chromium.org/1178703010/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/feedback/ConnectivityCheckerCollector.java
(right):

https://codereview.chromium.org/1178703010/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/feedback/ConnectivityCheckerCollector.java:109:
private final Map<Type, Result> mResult = new EnumMap<>(Type.class);
Does findbugs not complain here about unchecked somesuch?

[nyquist, 2015-06-11 23:09:49]

https://codereview.chromium.org/1178703010/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/feedback/ConnectivityCheckerCollector.java
(right):

https://codereview.chromium.org/1178703010/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/feedback/ConnectivityCheckerCollector.java:109:
private final Map<Type, Result> mResult = new EnumMap<>(Type.class);
On 2015/06/11 20:26:08, dfalcantara wrote:
> Does findbugs not complain here about unchecked somesuch?

Oh, maybe that was why I kept it in. Changed it back since it's not really
needed for this change anyway.

[nyquist, 2015-06-11 23:10:00]

On 2015/06/11 20:21:38, dfalcantara wrote:
> On 2015/06/11 18:59:23, nyquist wrote:
> > dfalcantara: PTAL
> 
> Remove the @deprecated in this CL?

I'll do this in a separate CL.

[nyquist, 2015-06-11 23:10:26]

The CQ bit was checked by nyquist@chromium.org

[nyquist, 2015-06-11 23:10:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
dfalcantara@chromium.org
Link to the patchset: https://codereview.chromium.org/1178703010/#ps20001
(title: "Removed unnecessary change to EnumMap creation.")

[commit-bot: I haz the power, 2015-06-11 23:10:53]

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

[commit-bot: I haz the power, 2015-06-11 23:45:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-06-11 23:46:42]

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