Cope with a race condition in NetworkActionPredictor and NetworkActionPredictorDB initialization.

Cope with a race condition in NetworkActionPredictor and NetworkActionPredictorDB initialization.

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

[dominich, 2011-11-14 22:25:53]

This was happening as the NAP destructor was being triggered from the browser
test before the task to call NAPDB::Initialize could complete. Replacing the
CHECK with an if() return; is safe.

[Peter Kasting, 2011-11-14 22:49:13]

LGTM

http://codereview.chromium.org/8566018/diff/1/chrome/browser/autocomplete/net...
File chrome/browser/autocomplete/network_action_predictor_database.cc (right):

http://codereview.chromium.org/8566018/diff/1/chrome/browser/autocomplete/net...
chrome/browser/autocomplete/network_action_predictor_database.cc:92: if
(canceled_.IsSet())
Nit: Looks like your practice elsewhere is to add a newline above this.

[commit-bot: I haz the power, 2011-11-15 15:32:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dominich@chromium.org/8566018/3001

[commit-bot: I haz the power, 2011-11-15 17:01:47]

Try job failure for 8566018-3001 (retry) on mac_rel for step "ui_tests".
It's a second try, previously, step "ui_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2011-11-15 17:24:39]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dominich@chromium.org/8566018/3001

[commit-bot: I haz the power, 2011-11-15 18:23:45]

Change committed as 110119