SafeBrowsingSystemTest: use notification for end of update instead of polling.

SafeBrowsingSystemTest: use notification for end of update instead of polling.

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

[Brian Ryner, 2012-08-09 18:10:36]

lgtm

http://codereview.chromium.org/10827189/diff/7001/chrome/browser/safe_browsin...
File chrome/browser/safe_browsing/safe_browsing_test.cc (right):

http://codereview.chromium.org/10827189/diff/7001/chrome/browser/safe_browsin...
chrome/browser/safe_browsing/safe_browsing_test.cc:440: // Wait for a given
period to get safebrowsing status updated.
Is the "for a given period" part of this comment still relevant?

[commit-bot: I haz the power, 2012-08-10 00:12:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mattm@chromium.org/10827189/4004

[mattm, 2012-08-10 00:12:53]

http://codereview.chromium.org/10827189/diff/7001/chrome/browser/safe_browsin...
File chrome/browser/safe_browsing/safe_browsing_test.cc (right):

http://codereview.chromium.org/10827189/diff/7001/chrome/browser/safe_browsin...
chrome/browser/safe_browsing/safe_browsing_test.cc:440: // Wait for a given
period to get safebrowsing status updated.
On 2012/08/09 18:10:36, Brian Ryner wrote:
> Is the "for a given period" part of this comment still relevant?

nope, removed. thanks

[commit-bot: I haz the power, 2012-08-10 01:51:07]

Change committed as 150970