Add a boolean histogram "Chrome.CommandLineUseSystemSSL" to keep track of

Add a boolean histogram "Chrome.CommandLineUseSystemSSL" to keep track of
"--use-system-ssl" usage. The flag forces the user to use SSLClientSocket[Mac,
  Win], instead of the default NSS SSL implementation.

I also changed the order of initialize callings in
BrowserMainRunnerImpl::Initialize(), because originally the StatisticsRecorder
was created after main loop flag parsing so the new histogram can not be
recorded. Since StatisticsRecorder has few dependencies, I think this will
not have negative effect.

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

[kaiwang, 2012-03-02 00:54:16]

Hi Marshall,
Could you take a look at my change of browser_main_runner.cc? I've changed the
initialized order. StatisticsRecorder has few dependencies, so I think this will
not cause problems.

[Marshall, 2012-03-02 02:14:06]

http://codereview.chromium.org/9570052/diff/4/content/browser/browser_main_ru...
File content/browser/browser_main_runner.cc (right):

http://codereview.chromium.org/9570052/diff/4/content/browser/browser_main_ru...
content/browser/browser_main_runner.cc:44: statistics_.reset(new
base::StatisticsRecorder);
This should probably come after the WaitForDebugger line. Also, please maintain
the reverse order for calling reset() in the Shutdown method.

[kaiwang, 2012-03-02 04:26:56]

http://codereview.chromium.org/9570052/diff/4/content/browser/browser_main_ru...
File content/browser/browser_main_runner.cc (right):

http://codereview.chromium.org/9570052/diff/4/content/browser/browser_main_ru...
content/browser/browser_main_runner.cc:44: statistics_.reset(new
base::StatisticsRecorder);
On 2012/03/02 02:14:06, Marshall wrote:
> This should probably come after the WaitForDebugger line. Also, please
maintain
> the reverse order for calling reset() in the Shutdown method.

Done.

[Marshall, 2012-03-02 17:00:45]

lgtm but an owner will need to approve.

[jar (doing other things), 2012-03-02 18:34:51]

LGTM

You'll probably need a OWNERS review from content... so I'm adding Jam (to get a
fancy rubber stamp).

[jam, 2012-03-02 19:13:10]

lgtm

[kaiwang, 2012-03-02 21:25:21]

Trybot now passed my change (seems last error was a flacky error).
I'm going to commit the change.

[commit-bot: I haz the power, 2012-03-02 21:25:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kaiwang@chromium.org/9570052/11001

[commit-bot: I haz the power, 2012-03-02 21:27:37]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kaiwang@chromium.org/9570052/11001

[commit-bot: I haz the power, 2012-03-02 21:29:09]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/kaiwang@chromium.org/9570052/11001

[commit-bot: I haz the power, 2012-03-02 23:34:35]

Change committed as 124770