Chrome OS "open with" picker allowing Web Intents

net/base/mime_util_unittest.cc

Index: net/base/mime_util_unittest.cc
diff --git a/net/base/mime_util_unittest.cc b/net/base/mime_util_unittest.cc
index 8210b0c9d954d9d9f3f7d14c6a774093abd2ac6e..af214332cf2ef5bf029bc2c170f73117b799a06d 100644
--- a/net/base/mime_util_unittest.cc
+++ b/net/base/mime_util_unittest.cc
@@ -80,6 +80,7 @@ TEST(MimeUtilTest, LookupTypes) {
 
 TEST(MimeUtilTest, MatchesMimeType) {
   EXPECT_TRUE(MatchesMimeType("*", "video/x-mpeg"));
+  EXPECT_TRUE(MatchesMimeType("*", ""));

[benwells, 2012/08/27 08:02:59]

Is there a reason to add this test?

[thorogood, 2012/08/28 00:04:04]

The reason for this is that it demonstrates that if a Web Intent handler
supports "*", it can accept all files, even those without extensions.

We weren't showing that this was the case through a test, and I added this
mostly to confirm this behaviour (as opposed to the alternative, which was that
a blank mime-type could not be matched by "*").

   EXPECT_TRUE(MatchesMimeType("video/*", "video/x-mpeg"));
   EXPECT_TRUE(MatchesMimeType("video/x-mpeg", "video/x-mpeg"));
   EXPECT_TRUE(MatchesMimeType("application/*+xml",