Report PasswordManager.OsPasswordStatus on a background thread with a delay.

Report PasswordManager.OsPasswordStatus on a background thread with a delay.

The histogram was revived in https://codereview.chromium.org/772413004/ but caused jankiness on the UI thread.

BUG=441428
Committed: https://crrev.com/08cd707154a8b1ca7869c5b410d98acb8a6ed3e7
Cr-Commit-Position: refs/heads/master@{#308128}

[vasilii, 2014-12-12 17:25:31]

vasilii@chromium.org changed reviewers:
+ vabr@chromium.org

[vasilii, 2014-12-12 17:25:45]

Hi Vaclav,

please review.

[vabr (Chromium), 2014-12-12 17:34:02]

lgtm

[vasilii, 2014-12-12 17:39:00]

The CQ bit was checked by vasilii@chromium.org

[commit-bot: I haz the power, 2014-12-12 17:39:37]

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

[commit-bot: I haz the power, 2014-12-12 18:41:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2014-12-12 18:42:11]

Patchset 1 (id:??) landed as
https://crrev.com/08cd707154a8b1ca7869c5b410d98acb8a6ed3e7
Cr-Commit-Position: refs/heads/master@{#308128}

[grt (UTC plus 2), 2014-12-16 18:21:16]

This change introduced a DCHECK because CheckBlankPassword accesses the browser
process's local_state() off the main thread.

(180.23c0): Break instruction exception - code 80000003 (first chance)
eax=05c3ec00 ebx=05c3f0ec ecx=00000000 edx=00000000 esi=05c3f0f8 edi=05c3f0f4
eip=6fc808e1 esp=05c3ec8c ebp=05c3f0dc iopl=0         nv up ei pl zr na pe nc
cs=0023  ss=002b  ds=002b  es=002b  fs=0053  gs=002b             efl=00000246
base!base::debug::BreakDebugger+0x11:
6fc808e1 cc              int     3
*** WARNING: Unable to verify checksum for
E:\src\chrome\shared\src\out\Release\content.dll
0:012> k
ChildEBP RetAddr  
05c3ec88 6fcb7e3a base!base::debug::BreakDebugger+0x11
[e:\src\chrome\shared\src\base\debug\debugger_win.cc @ 21]
05c3f0dc 6d11111e base!logging::LogMessage::~LogMessage+0x23a
[e:\src\chrome\shared\src\base\logging.cc @ 643]
05c3f1a0 6d1ff1a3 chrome_6cd60000!BrowserProcessImpl::local_state+0x5e
[e:\src\chrome\shared\src\chrome\browser\browser_process_impl.cc @ 547]
05c3f1e0 6d1ff508 chrome_6cd60000!password_manager_util::CheckBlankPassword+0x23
[e:\src\chrome\shared\src\chrome\browser\password_manager\password_manager_util_win.cc
@ 66]
05c3f5f8 6d1fff17
chrome_6cd60000!password_manager_util::GetOsPasswordStatus+0x98
[e:\src\chrome\shared\src\chrome\browser\password_manager\password_manager_util_win.cc
@ 137]
05c3f62c 6fc84b4c chrome_6cd60000!`anonymous namespace'::ReportOsPassword+0x17
[e:\src\chrome\shared\src\chrome\browser\password_manager\password_store_factory.cc
@ 60]

[grt (UTC plus 2), 2014-12-16 18:28:45]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/802923003/ by grt@chromium.org.

The reason for reverting is: Reverting because DCHECK is hit 40s after startup..