Do not display warnings for file:/// extension permissions.

chrome/common/extensions/extension_permission_set_unittest.cc

Index: chrome/common/extensions/extension_permission_set_unittest.cc
diff --git a/chrome/common/extensions/extension_permission_set_unittest.cc b/chrome/common/extensions/extension_permission_set_unittest.cc
index 7bba33a7e17f263283d8549bfbd8f3c35eae8900..644965f9a32cda300e61b07fc34865f536016a1c 100644
--- a/chrome/common/extensions/extension_permission_set_unittest.cc
+++ b/chrome/common/extensions/extension_permission_set_unittest.cc
@@ -872,6 +872,21 @@ TEST(ExtensionPermissionSetTest, GetDistinctHostsForDisplay) {
         empty_perms, explicit_hosts, scriptable_hosts);
     EXPECT_EQ(expected, perm_set->GetDistinctHostsForDisplay());
   }
+
+  {
+    // We don't display warnings for file URLs because they are off by default.
+    SCOPED_TRACE("file urls");

[asargent_no_longer_on_chrome, 2011/08/05 22:56:37]

Whoa, SCOPED_TRACE is awesome! I had never heard of it before, and just
yesterday was writing a CL where I wished we had something like this. 

+    explicit_hosts.ClearPatterns();
+    scriptable_hosts.ClearPatterns();
+    expected.clear();
+
+    explicit_hosts.AddPattern(
+        URLPattern(URLPattern::SCHEME_FILE, "file:///*"));
+
+    perm_set = new ExtensionPermissionSet(
+        empty_perms, explicit_hosts, scriptable_hosts);
+    EXPECT_EQ(expected, perm_set->GetDistinctHostsForDisplay());
+  }
 }
 
 TEST(ExtensionPermissionSetTest, GetDistinctHostsForDisplay_ComIsBestRcd) {