Move check whether desktop notifications can be shown to browser

content/browser/renderer_host/render_view_host.cc

 // Copyright (c) 2011 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 "content/browser/renderer_host/render_view_host.h"
 
 #include <string>
 #include <utility>
 #include <vector>
 
@@ skipping 1353 matching lines @@
 }
 
 void RenderViewHost::OnRequestDesktopNotificationPermission(
     const GURL& source_origin, int callback_context) {
   content::GetContentClient()->browser()->RequestDesktopNotificationPermission(
       source_origin, callback_context, process()->id(), routing_id());
 }
 
 void RenderViewHost::OnShowDesktopNotification(
     const DesktopNotificationHostMsg_Show_Params& params) {
-  // Disallow HTML notifications from unwanted schemes.  javascript:
-  // in particular allows unwanted cross-domain access.
+  // Disallow HTML notifications from javascript: and file: schemes as this
+  // allows unwanted cross-domain access.
   GURL url = params.contents_url;
   if (params.is_html &&
-      !url.SchemeIs(chrome::kHttpScheme) &&
-      !url.SchemeIs(chrome::kHttpsScheme) &&
-      !url.SchemeIs(chrome::kExtensionScheme) &&
-      !url.SchemeIs(chrome::kDataScheme)) {
+      (url.SchemeIs(chrome::kJavaScriptScheme) ||
+       url.SchemeIs(chrome::kFileScheme))) {
     return;
   }
 
   content::GetContentClient()->browser()->ShowDesktopNotification(
       params, process()->id(), routing_id(), false);
 }
 
 void RenderViewHost::OnCancelDesktopNotification(int notification_id) {
   content::GetContentClient()->browser()->CancelDesktopNotification(
       process()->id(), routing_id(), notification_id);
@@ skipping 17 matching lines @@
 void RenderViewHost::OnRunFileChooser(
     const ViewHostMsg_RunFileChooser_Params& params) {
   delegate_->RunFileChooser(this, params);
 }
 
 void RenderViewHost::OnWebUISend(const GURL& source_url,
                                  const std::string& name,
                                  const base::ListValue& args) {
   delegate_->WebUISend(this, source_url, name, args);
 }