Fix crbug.com/140910 (BrowsingDataRemover not working).

Fix crbug.com/140910 (BrowsingDataRemover not working).

TBR=estade@chromium.org
BUG=140910
TEST=Manual


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

[Mike West, 2012-08-08 07:55:33]

I love you, Marja. Good catch.

LGTM. Land it, please. :)

[commit-bot: I haz the power, 2012-08-08 07:57:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/marja@chromium.org/10830210/1

[commit-bot: I haz the power, 2012-08-08 07:57:42]

Presubmit check for 10830210-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for files in these directories:
    chrome
    chrome/browser/ui/webui/options2

[battre, 2012-08-08 07:59:54]

https://chromiumcodereview.appspot.com/10830210/diff/1/chrome/browser/chrome_...
File chrome/browser/chrome_content_browser_client.cc (left):

https://chromiumcodereview.appspot.com/10830210/diff/1/chrome/browser/chrome_...
chrome/browser/chrome_content_browser_client.cc:1563: BrowsingDataRemover*
remover = new BrowsingDataRemover(profile,
There are more calls of this constructor with base::Time() as the third
parameter:
- chrome/browser/ui/browser_commands.cc
- chrome/browser/automation/testing_automation_provider.cc
- chrome/browser/automation/automation_provider.cc
- chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc
- chrome/browser/prerender/prerender_browsertest.cc
- chrome/browser/ui/webui/net_internals/net_internals_ui.cc
-
chrome/browser/ui/webui/chromeos/login/enterprise_oauth_enrollment_screen_handler.cc

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

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/marja@chromium.org/10830210/1

[marja, 2012-08-08 08:40:20]

A new patch set fixes the other usages, and adds a DCHECK and conversion
(base::Time() -> base::Time::Now()) to BrowsingDataRemover.

[Mike West, 2012-08-08 08:41:13]

On 2012/08/08 08:40:20, marja wrote:
> A new patch set fixes the other usages, and adds a DCHECK and conversion
> (base::Time() -> base::Time::Now()) to BrowsingDataRemover.

LGTM. Thanks for the quick fix.

[commit-bot: I haz the power, 2012-08-08 08:42:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/marja@chromium.org/10830210/7002

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

Change committed as 150531