Use completion_time to call LoadingFinished of DevTools's Network event for navigation preload

Use completion_time to call LoadingFinished of DevTools's Network event for navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.

I found this difference while investigating the issue 712809. |completion_time|
is set when the request finished but, NavigationPreloadCompleted() isn't called
before the service worker finishes starting up.

BUG=712809
Review-Url: https://codereview.chromium.org/2844133005
Cr-Commit-Position: refs/heads/master@{#467896}
Committed: https://chromium.googlesource.com/chromium/src/+/99d003caf2d7876a7e402a07caa94a5c24ccb922

[horo, 2017-04-28 02:49:02]

The CQ bit was checked by horo@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-04-28 02:49:39]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[horo, 2017-04-28 02:54:00]

Description was changed from

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

BUG=
==========

to

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.


BUG=
==========

[horo, 2017-04-28 02:54:41]

Description was changed from

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.


BUG=
==========

to

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.

(I found this difference while investigating the issue 712809) 

BUG=712809
==========

[horo, 2017-04-28 03:00:41]

Description was changed from

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.

(I found this difference while investigating the issue 712809) 

BUG=712809
==========

to

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.

I found this difference while investigating the issue 712809. |completion_time|
is set when the request finished but, NavigationPreloadCompleted() isn't called
before the service worker finishes starting up.

BUG=712809
==========

[horo, 2017-04-28 03:01:38]

horo@chromium.org changed reviewers:
+ kinuko@chromium.org

[horo, 2017-04-28 03:01:38]

kinuko@
Could you please review?

[kinuko, 2017-04-28 03:09:35]

lgtm

[commit-bot: I haz the power, 2017-04-28 04:11:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-04-28 04:11:35]

Dry run: This issue passed the CQ dry run.

[horo, 2017-04-28 04:20:13]

The CQ bit was checked by horo@chromium.org

[commit-bot: I haz the power, 2017-04-28 04:20:47]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-28 05:22:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493353213647100, "parent_rev":
"08879a1263f332c02ff6ad2d6179fc8a20af4901", "commit_rev":
"99d003caf2d7876a7e402a07caa94a5c24ccb922"}

[commit-bot: I haz the power, 2017-04-28 05:22:31]

Description was changed from

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.

I found this difference while investigating the issue 712809. |completion_time|
is set when the request finished but, NavigationPreloadCompleted() isn't called
before the service worker finishes starting up.

BUG=712809
==========

to

==========
Use completion_time to call LoadingFinished of DevTools's Network event for
navigation preload

Currently NetworkHandler::NavigationPreloadCompleted() is using TimeTicks::Now()
to call LoadingFinished() of DevTools's Network event. But for normal requests
from the renderer, FrameFetchContext::DispatchDidFinishLoading() is using the
|finish_time| which is originated from ResourceRequestCompletionStatus's
completion_time.

NetworkHandler::NavigationPreloadCompleted() should use the same timing
information.

I found this difference while investigating the issue 712809. |completion_time|
is set when the request finished but, NavigationPreloadCompleted() isn't called
before the service worker finishes starting up.

BUG=712809

Review-Url: https://codereview.chromium.org/2844133005
Cr-Commit-Position: refs/heads/master@{#467896}
Committed:
https://chromium.googlesource.com/chromium/src/+/99d003caf2d7876a7e402a07caa9...
==========

[commit-bot: I haz the power, 2017-04-28 05:22:33]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/99d003caf2d7876a7e402a07caa9...