Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2688)

Unified Diff: chrome/browser/custom_handlers/protocol_handler_registry_factory.cc

Issue 10908088: Cleanup: Constify some ProfileKeyedBaseFactory methods and all overrides. Remove ProfileKeyedBaseFa… (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: rebase, remove a few more lines of code Created 8 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/custom_handlers/protocol_handler_registry_factory.cc
===================================================================
--- chrome/browser/custom_handlers/protocol_handler_registry_factory.cc (revision 154991)
+++ chrome/browser/custom_handlers/protocol_handler_registry_factory.cc (working copy)
@@ -33,19 +33,19 @@
// Will be created when initializing profile_io_data, so we might
// as well have the framework create this along with other
// PKSs to preserve orderly civic conduct :)
-bool ProtocolHandlerRegistryFactory::ServiceIsCreatedWithProfile() {
+bool ProtocolHandlerRegistryFactory::ServiceIsCreatedWithProfile() const {
return true;
}
// Allows the produced registry to be used in incognito mode.
-bool ProtocolHandlerRegistryFactory::ServiceRedirectedInIncognito() {
+bool ProtocolHandlerRegistryFactory::ServiceRedirectedInIncognito() const {
return true;
}
// Do not create this service for tests. MANY tests will fail
// due to the threading requirements of this service. ALSO,
// not creating this increases test isolation (which is GOOD!)
-bool ProtocolHandlerRegistryFactory::ServiceIsNULLWhileTesting() {
+bool ProtocolHandlerRegistryFactory::ServiceIsNULLWhileTesting() const {
return true;
}
« no previous file with comments | « chrome/browser/custom_handlers/protocol_handler_registry_factory.h ('k') | chrome/browser/download/download_service_factory.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698