Media Documents should disable View [Page] Source

chrome/browser/tab_contents/view_source_browsertest.cc

Index: chrome/browser/tab_contents/view_source_browsertest.cc
diff --git a/chrome/browser/tab_contents/view_source_browsertest.cc b/chrome/browser/tab_contents/view_source_browsertest.cc
index 12367751323f24dfee1781214e15d23124488843..3d08b054cc9d0844e8cfd3c40b1d0b3ed2338ff0 100644
--- a/chrome/browser/tab_contents/view_source_browsertest.cc
+++ b/chrome/browser/tab_contents/view_source_browsertest.cc
@@ -21,6 +21,7 @@
 
 namespace {
 const char kTestHtml[] = "files/viewsource/test.html";
+const char kTestMedia[] = "files/media/pink_noise_140ms.wav";

[wolenetz, 2013/01/04 22:58:37]

@ jam@: Is referencing a test data file from media, not viewsource, OK or do we
need to duplicate such content into the viewsource tree for use here?  Lack of
duplication could lead to test fragility if media test data changes later.

[jam, 2013/01/04 23:48:41]

I'm not sure what you mean? pink_noise_140ms.wav is in
chrome/test/data/media/pink_noise_140ms.wav

[wolenetz, 2013/01/05 00:05:19]

Yes (and the test passes).  The question really is does this wav file need to be
copied into chrome/test/data/viewsource/ and the value of kTestMedia updated to
the viewsource location, or are references by this browsertest to
chrome/test/data/media/ files OK?

 }
 
 typedef InProcessBrowserTest ViewSourceTest;
@@ -75,6 +76,21 @@ IN_PROC_BROWSER_TEST_F(ViewSourceTest, ViewSourceInMenuEnabledOnANormalPage) {
   EXPECT_TRUE(chrome::CanViewSource(browser()));
 }
 
+// For page that is media content, make sure that we cannot select "View Source"
+// See http://crbug.com/83714
+IN_PROC_BROWSER_TEST_F(ViewSourceTest, ViewSourceInMenuDisabledOnAMediaPage) {
+  ASSERT_TRUE(test_server()->Start());
+
+  GURL url(test_server()->GetURL(kTestMedia));
+  ui_test_utils::NavigateToURL(browser(), url);
+
+  const char* mime_type = chrome::GetActiveWebContents(browser())->
+      GetContentsMimeType().c_str();
+
+  EXPECT_STREQ("audio/wav", mime_type);
+  EXPECT_FALSE(chrome::CanViewSource(browser()));
+}
+
 // Make sure that when looking at the page source, we can't select "View Source"
 // from the menu.
 IN_PROC_BROWSER_TEST_F(ViewSourceTest,