Makes GetComponentExtensionResourceManager() thread-safe.
As is noted in a bug, this method can be called on various thread.
Actually, the instance itself holds simple mapping and its method
can be const, therefore it's safer to early-initialize and the
method just returns the already initialized object.
Also this should be const and the method does not change its
internal state, therefore it's thread safe.
BUG=445747
[email protected]
TEST=no functional changes / build succeeds
Review URL: https://codereview.chromium.org/830833002
Cr-Commit-Position: refs/heads/master@{#310029}
diff --git a/extensions/browser/component_extension_resource_manager.h b/extensions/browser/component_extension_resource_manager.h
index 1f82381e..6495f10 100644
--- a/extensions/browser/component_extension_resource_manager.h
+++ b/extensions/browser/component_extension_resource_manager.h
@@ -22,9 +22,9 @@
// resources. Otherwise fills |resource_id|. This doesn't check if the
// extension the resource is in is actually a component extension.
virtual bool IsComponentExtensionResource(
- const base::FilePath& extension_path,
- const base::FilePath& resource_path,
- int* resource_id) = 0;
+ const base::FilePath& extension_path,
+ const base::FilePath& resource_path,
+ int* resource_id) const = 0;
};
} // namespace extensions
