Adding QuotaManager logic to TestingProfile.

Adding QuotaManager logic to TestingProfile.

BUG=63700
TEST=unit_tests

[Mike West, 2011-07-21 14:50:32]

marja: Can you review the change to the appcache tests in the
BrowsingDataRemoverTest code?

Not sure who should check the TestingProfile change. Looks like either michaeln
or kinuko wrote most of the code I copied from ProfileImpl, so... I hope one of
you can take a look at these pastes. :)

Thanks so much!

-Mike

[marja, 2011-07-21 14:59:05]

On 2011/07/21 14:50:32, Mike West wrote:
> marja: Can you review the change to the appcache tests in the
> BrowsingDataRemoverTest code?

BrowsingDataRemoverTest changes LGTM.

[Mike West, 2011-07-21 19:37:22]

Yaks need shaving with regard to threading in the testing profile; I'm doing my
best to avoid them. :)

[michaeln, 2011-07-22 00:09:58]

Please see my comments in http://codereview.chromium.org/7129018/, i think
changes along these line may not be necessary to support BDR unit tests.

http://codereview.chromium.org/7464029/diff/1/chrome/browser/browsing_data_re...
File chrome/browser/browsing_data_remover_unittest.cc (left):

http://codereview.chromium.org/7464029/diff/1/chrome/browser/browsing_data_re...
chrome/browser/browsing_data_remover_unittest.cc:379:
MessageLoop::current()->RunAllPending();
the call is RunAllPending is gone now... is that ok?

http://codereview.chromium.org/7464029/diff/1/chrome/browser/browsing_data_re...
File chrome/browser/browsing_data_remover_unittest.cc (right):

http://codereview.chromium.org/7464029/diff/1/chrome/browser/browsing_data_re...
chrome/browser/browsing_data_remover_unittest.cc:373:
profile->SetExtensionSpecialStoragePolicy(mock_policy);
since the policy is now set up on line 368, no longer needed?

http://codereview.chromium.org/7464029/diff/1/chrome/browser/browsing_data_re...
chrome/browser/browsing_data_remover_unittest.cc:390: profile,
BrowsingDataRemover::EVERYTHING, base::Time::Now());
maybe use GetProfile() here now instead of 'profile' and get rid of the local

[Mike West, 2011-07-26 07:53:20]

Rolling this into http://codereview.chromium.org/7129018/, closing the review.

Marja, you should probably take a look at that review, as I've thrown away all
the appcache-specific code in the BrowsingDataRemover: QuotaManager is handling
it now. :)