Use the last committed entry for View Source.

chrome/browser/ui/browser_commands.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 "chrome/browser/ui/browser_commands.h"
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_editor.h"
@@ skipping 858 matching lines @@
 
 bool IsDebuggerAttachedToCurrentTab(Browser* browser) {
   WebContents* contents = chrome::GetActiveWebContents(browser);
   return contents ?
       content::DevToolsAgentHostRegistry::IsDebuggerAttached(contents) : false;
 }
 
 void ViewSource(Browser* browser, TabContents* contents) {
   DCHECK(contents);
 
-  NavigationEntry* active_entry =
-    contents->web_contents()->GetController().GetActiveEntry();
-  if (!active_entry)
+  // Use the last committed entry, since the pending entry hasn't loaded yet and
+  // won't be copied into the cloned tab.
+  NavigationEntry* entry =
+    contents->web_contents()->GetController().GetLastCommittedEntry();
+  if (!entry)
     return;
 
-  ViewSource(browser, contents, active_entry->GetURL(),
-             active_entry->GetContentState());
+  ViewSource(browser, contents, entry->GetURL(), entry->GetContentState());
 }
 
 void ViewSource(Browser* browser,
                 TabContents* contents,
                 const GURL& url,
                 const std::string& content_state) {
   content::RecordAction(UserMetricsAction("ViewSource"));
   DCHECK(contents);
 
+  // Note that Clone does not copy the pending or transient entries, so the
+  // active entry in view_source_contents will be the last committed entry.
   TabContents* view_source_contents = contents->Clone();
   view_source_contents->web_contents()->GetController().PruneAllButActive();
   NavigationEntry* active_entry =
       view_source_contents->web_contents()->GetController().GetActiveEntry();
   if (!active_entry)
     return;
 
   GURL view_source_url = GURL(kViewSourceScheme + std::string(":") +
       url.spec());
   active_entry->SetVirtualURL(view_source_url);
@@ skipping 74 matching lines @@
   if (!tab_contents)
     tab_contents = new TabContents(contents);
   app_browser->tab_strip_model()->AppendTabContents(tab_contents, true);
 
   contents->GetMutableRendererPrefs()->can_accept_load_drops = false;
   contents->GetRenderViewHost()->SyncRendererPrefs();
   app_browser->window()->Show();
 }
 
 }  // namespace chrome