Adding a BrowsingDataHelper class to hold some useful methods.

Adding a BrowsingDataHelper class to hold some useful methods.

There's simply no good reason to hardcode checks against
chrome::kExtensionScheme in various bits of BrowsingData*. BrowsingDataHelper
provides a static method that we can start using whenever we need to know about
browsing-data-relevant schemes.

BUG=121636
TEST=unit_tests:BrowsingDataRemover*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=130574

[Mike West, 2012-04-03 14:04:14]

Is this more or less what you were suggesting, Jochen?

-Mike

[jochen (gone - plz use gerrit), 2012-04-03 14:08:13]

lgtm

http://codereview.chromium.org/9958111/diff/7/chrome/browser/browsing_data_he...
File chrome/browser/browsing_data_helper.h (right):

http://codereview.chromium.org/9958111/diff/7/chrome/browser/browsing_data_he...
chrome/browser/browsing_data_helper.h:16: class WebString;
no indent in namespaces

[commit-bot: I haz the power, 2012-04-03 14:52:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mkwst@chromium.org/9958111/2015

[commit-bot: I haz the power, 2012-04-03 18:01:55]

Try job failure for 9958111-2015 (retry) on win_rel for steps "browser_tests,
unit_tests".
It's a second try, previously, steps "browser_tests, unit_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[commit-bot: I haz the power, 2012-04-03 19:01:47]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/mkwst@chromium.org/9958111/2015

[commit-bot: I haz the power, 2012-04-03 23:28:52]

Try job failure for 9958111-2015 (retry) on win_rel for step "browser_tests".
It's a second try, previously, steps "browser_tests, unit_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[Mike West, 2012-04-04 08:48:51]

I tried to be a good boy. Ended up landing it manually. :)