Enabling "Save as..." in view:source page....

chrome/browser/tab_contents/render_view_context_menu.cc

 // Copyright (c) 2006-2009 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 "chrome/browser/tab_contents/render_view_context_menu.h"
 
 #include "app/l10n_util.h"
 #include "base/clipboard.h"
 #include "base/command_line.h"
 #include "base/logging.h"
@@ skipping 289 matching lines @@
              URLRequest::IsHandledURL(params_.link_url);
 
     case IDS_CONTENT_CONTEXT_SAVEIMAGEAS:
       return params_.src_url.is_valid() &&
              URLRequest::IsHandledURL(params_.src_url);
 
     case IDS_CONTENT_CONTEXT_OPENIMAGENEWTAB:
       // The images shown in the most visited thumbnails do not currently open
       // in a new tab as they should. Disabling this context menu option for
       // now, as a quick hack, before we resolve this issue (Issue = 2608).
-      // TODO (sidchat): Enable this option once this issue is resolved.
+      // TODO(sidchat): Enable this option once this issue is resolved.
       if (params_.src_url.scheme() == chrome::kChromeUIScheme)
         return false;
       return true;
 
     case IDS_CONTENT_CONTEXT_FULLSCREEN:
       // TODO(ajwong): Enable fullscreen after we actually implement this.
       return false;
 
     // Media control commands should all be disabled if the player is in an
     // error state.
@@ skipping 24 matching lines @@
     case IDS_CONTENT_CONTEXT_SAVEVIDEOAS:
       return (params_.media_params.player_state &
               ContextMenuMediaParams::CAN_SAVE) &&
              params_.src_url.is_valid() &&
              URLRequest::IsHandledURL(params_.src_url);
 
     case IDS_CONTENT_CONTEXT_OPENAUDIONEWTAB:
     case IDS_CONTENT_CONTEXT_OPENVIDEONEWTAB:
       return true;
 
-    case IDS_CONTENT_CONTEXT_SAVEPAGEAS:
-      return SavePackage::IsSavableURL(source_tab_contents_->GetURL());
+    case IDS_CONTENT_CONTEXT_SAVEPAGEAS: {
+      // Instead of using GetURL here, we use url() (which is the "real" url of
+      // the page) from the NavigationEntry because its reflects their origin
+      // rather than the display one (returned by GetURL) which may be
+      // different (like having "view-source:" on the front).
+      NavigationEntry* active_entry =
+          source_tab_contents_->controller().GetActiveEntry();
+      GURL savable_url = (active_entry) ? active_entry->url() :
+                                          GURL::EmptyGURL();
+      return SavePackage::IsSavableURL(savable_url);
+    }
 
     case IDS_CONTENT_CONTEXT_OPENFRAMENEWTAB:
     case IDS_CONTENT_CONTEXT_OPENFRAMENEWWINDOW:
       return params_.frame_url.is_valid();
 
     case IDS_CONTENT_CONTEXT_UNDO:
       return !!(params_.edit_flags & ContextNodeType::CAN_UNDO);
 
     case IDS_CONTENT_CONTEXT_REDO:
       return !!(params_.edit_flags & ContextNodeType::CAN_REDO);
@@ skipping 415 matching lines @@
   WriteTextToClipboard(UTF8ToUTF16(utf8_text));
   DidWriteURLToClipboard(utf8_text);
 }
 
 void RenderViewContextMenu::MediaPlayerActionAt(
     int x,
     int y,
     const MediaPlayerAction& action) {
   source_tab_contents_->render_view_host()->MediaPlayerActionAt(x, y, action);
 }