Media Documents should disable View [Page] Source

chrome/browser/tab_contents/view_source_browsertest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/notification_service.h"
 #include "content/public/browser/notification_types.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/test/browser_test_utils.h"
 #include "googleurl/src/gurl.h"
 #include "net/test/test_server.h"
 
 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;
 
 // This test renders a page in view-source and then checks to see if the title
 // set in the html was set successfully (it shouldn't because we rendered the
 // page in view source).
 // Flaky; see http://crbug.com/72201.
 IN_PROC_BROWSER_TEST_F(ViewSourceTest, DoesBrowserRenderInViewSource) {
   ASSERT_TRUE(test_server()->Start());
@@ skipping 34 matching lines @@
 // from the menu.
 IN_PROC_BROWSER_TEST_F(ViewSourceTest, ViewSourceInMenuEnabledOnANormalPage) {
   ASSERT_TRUE(test_server()->Start());
 
   GURL url(test_server()->GetURL(kTestHtml));
   ui_test_utils::NavigateToURL(browser(), url);
 
   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,
                        ViewSourceInMenuDisabledWhileViewingSource) {
   ASSERT_TRUE(test_server()->Start());
 
   GURL url_viewsource(chrome::kViewSourceScheme + std::string(":") +
       test_server()->GetURL(kTestHtml).spec());
   ui_test_utils::NavigateToURL(browser(), url_viewsource);
 
@@ skipping 18 matching lines @@
   ASSERT_TRUE(content::ExecuteScript(chrome::GetWebContentsAt(browser(), 0),
                                      "window.location.reload();"));
 
   content::WindowedNotificationObserver observer2(
       content::NOTIFICATION_LOAD_STOP,
       content::NotificationService::AllSources());
   observer2.Wait();
   ASSERT_TRUE(chrome::GetWebContentsAt(browser(), 0)->GetController().
                   GetActiveEntry()->IsViewSourceMode());
 }