Adding Chrome Frame metric for SetSite performance.

Adding Chrome Frame metric for SetSite performance.

BUG=134640
TEST=NONE


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=145086

[grt (UTC plus 2), 2012-06-28 13:55:04]

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc
File chrome_frame/bho.cc (right):

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc#ne...
chrome_frame/bho.cc:1: // Copyright (c) 2011 The Chromium Authors. All rights
reserved.
2012

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc#ne...
chrome_frame/bho.cc:96: base::TimeTicks start = base::TimeTicks::Now();
move this down to line 100?

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc#ne...
chrome_frame/bho.cc:133: UMA_HISTOGRAM_TIMES("ChromeFrame.BhoLoadSetSite",
delta);
delta.InMilliseconds()

[robertshield, 2012-06-28 14:11:13]

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc
File chrome_frame/bho.cc (right):

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc#ne...
chrome_frame/bho.cc:1: // Copyright (c) 2011 The Chromium Authors. All rights
reserved.
On 2012/06/28 13:55:04, grt wrote:
> 2012

Done.

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc#ne...
chrome_frame/bho.cc:96: base::TimeTicks start = base::TimeTicks::Now();
On 2012/06/28 13:55:04, grt wrote:
> move this down to line 100?

Done.

https://chromiumcodereview.appspot.com/10704017/diff/1/chrome_frame/bho.cc#ne...
chrome_frame/bho.cc:133: UMA_HISTOGRAM_TIMES("ChromeFrame.BhoLoadSetSite",
delta);
On 2012/06/28 13:55:04, grt wrote:
> delta.InMilliseconds()

The macro already does that.

[grt (UTC plus 2), 2012-06-28 14:16:02]

lgtm

[commit-bot: I haz the power, 2012-06-28 18:23:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robertshield@chromium.org/10704017/9001

[commit-bot: I haz the power, 2012-06-28 18:23:55]

Failed to apply patch for chrome_frame/bho.cc:
While running patch -p1 --forward --force;
patching file chrome_frame/bho.cc
Hunk #1 FAILED at 1.
Hunk #3 succeeded at 73 (offset -22 lines).
Hunk #4 succeeded at 103 (offset -24 lines).
1 out of 4 hunks FAILED -- saving rejects to file chrome_frame/bho.cc.rej

[commit-bot: I haz the power, 2012-06-30 18:39:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/robertshield@chromium.org/10704017/3002

[commit-bot: I haz the power, 2012-06-30 19:34:17]

Change committed as 145086