Expose net::LoadTimingInfo through content_shell

Expose net::LoadTimingInfo through content_shell

Since blink use content_shell to run layout test, we need to expose net::LoadTimingInfo through content_shell

Contributed by pan.deng@intel.com

BUG=246117
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=203959

[Pan, 2013-06-03 09:31:38]

Hi James,

I noticed blink switched to content_shell to run layout tests, without this
change, net::LoadTimingInfo sent to blink is always null, which blocks
https://codereview.chromium.org/13912021/

please review :)

thanks
Pan

[James Simonsen, 2013-06-03 23:34:14]

You should remove the other callers in
chrome_resource_dispatcher_host_delegate.cc, otherwise we end up doing this
multiple times in Chrome. Also, move the comment from those callsites to this
file.

https://codereview.chromium.org/16324002/diff/1/content/browser/loader/resour...
File content/browser/loader/resource_loader.cc (right):

https://codereview.chromium.org/16324002/diff/1/content/browser/loader/resour...
content/browser/loader/resource_loader.cc:58:
request->GetLoadTimingInfo(&(response->head.load_timing));
Inner () not needed.

[Pan, 2013-06-04 00:51:49]

removed duplicate GetLoadTiming, 
thanks :)

Pan

https://codereview.chromium.org/16324002/diff/1/content/browser/loader/resour...
File content/browser/loader/resource_loader.cc (right):

https://codereview.chromium.org/16324002/diff/1/content/browser/loader/resour...
content/browser/loader/resource_loader.cc:58:
request->GetLoadTimingInfo(&(response->head.load_timing));
On 2013/06/03 23:34:14, James Simonsen wrote:
> Inner () not needed.

right, thanks!

[James Simonsen, 2013-06-04 00:54:59]

lgtm

You'll need to track down a chrome/ owner now too though.

[Pan, 2013-06-04 01:06:46]

Hi, @jochen, @darin, @jam 

could you please help review this patch :)

thanks!
Pan

[darin (slow to review), 2013-06-04 05:21:54]

LGTM

[commit-bot: I haz the power, 2013-06-04 06:06:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/pan.deng@intel.com/16324002/7001

[commit-bot: I haz the power, 2013-06-04 13:51:22]

Change committed as 203959