[extensions] Remove unnecessary checks in IsSensitiveURL

extensions/test/test_extensions_client.cc

Index: extensions/test/test_extensions_client.cc
diff --git a/extensions/test/test_extensions_client.cc b/extensions/test/test_extensions_client.cc
index 5d928a5762aa1530904c6f5c37a133943214af56..600bc2d9892887766c68e2ec78d8b57c11b8fcfd 100644
--- a/extensions/test/test_extensions_client.cc
+++ b/extensions/test/test_extensions_client.cc
@@ -156,7 +156,7 @@ std::string TestExtensionsClient::GetWebstoreUpdateURL() const {
 }
 
 bool TestExtensionsClient::IsBlacklistUpdateURL(const GURL& url) const {
-  return true;
+  return false;

[Charlie Harrison, 2016/11/10 22:10:20]

unit tests pass with this but I'm not sure if browser tests will. Does this seem
safe to you or should I find another way to test this function?

[Devlin, 2016/11/10 22:26:40]

I think this should be fine.  Not sure why the default would be true.

 }
 
 std::set<base::FilePath> TestExtensionsClient::GetBrowserImagePaths(