New Tab: fix tab title flickering.

content/browser/web_contents/web_contents_impl.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 "content/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
@@ skipping 732 matching lines @@
     // Don't override the title in view source mode.
     entry = controller_.GetVisibleEntry();
     if (!(entry && entry->IsViewSourceMode())) {
       // Give the Web UI the chance to override our title.
       const string16& title = our_web_ui->GetOverriddenTitle();
       if (!title.empty())
         return title;
     }
   }
 
+  // For initial navigations, use the title from the initial pending entry if it
+  // has a title set. This can happen, for example, when doing a history
+  // navigation in a new tab, such as Ctrl+Back.
+  entry = controller_.GetVisibleEntry();
+  if (entry && controller_.IsInitialNavigation()) {
+    const string16& title = entry->GetTitleForDisplay(accept_languages);

[Charlie Reis, 2013/11/22 23:19:12]

This isn't quite right.  GetTitleForDisplay will return the URL if no title is
set, so the empty check below won't ever fail.

We should be doing something closer to:
if (!entry->GetTitle().empty())
  return entry->GetTitleForDisplay(accept_languages);

Of course, that changes the Ctrl+Back case, so I'd rather not reorder this code.
 I think we want the following, down below the entry = LastCommitted line.

  // We make an exception for initial navigations.
  if (controller_.IsInitialNavigation()) {
    // We only want to use the title from the visible entry in one of two cases:
    // 1. There's already a committed entry for an initial navigation, in which
    //    case we are doing a history navigation in a new tab (e.g., Ctrl+Back).
    // 2. The pending entry has been explicitly assigned a title to display.
    //
    // If there's no last committed entry and no assigned title, we should fall
    // back to |page_title_when_no_navigation_entry_| rather than showing the
    // URL.
    if (entry ||
        (controller_.GetVisibleEntry() &&
         !controller_.GetVisibleEntry()->GetTitle().empty())) {
      entry = controller_.GetVisibleEntry();
    }
  }

[samarth, 2013/11/25 15:32:13]

Done, thanks.

+    if (!title.empty())
+      return title;
+  }
+
   // We use the title for the last committed entry rather than a pending
   // navigation entry. For example, when the user types in a URL, we want to
   // keep the old page's title until the new load has committed and we get a new
   // title.
   entry = controller_.GetLastCommittedEntry();
 
-  // We make an exception for initial navigations, because we can have a
-  // committed entry for an initial navigation when doing a history navigation
-  // in a new tab, such as Ctrl+Back.
-  if (entry && controller_.IsInitialNavigation())
-    entry = controller_.GetVisibleEntry();
-
   if (entry) {
     return entry->GetTitleForDisplay(accept_languages);
   }
 
   // |page_title_when_no_navigation_entry_| is finally used
   // if no title cannot be retrieved.
   return page_title_when_no_navigation_entry_;
 }
 
 int32 WebContentsImpl::GetMaxPageID() {
@@ skipping 2907 matching lines @@
 }
 
 void WebContentsImpl::OnFrameRemoved(
     RenderViewHostImpl* render_view_host,
     int64 frame_id) {
    FOR_EACH_OBSERVER(WebContentsObserver, observers_,
                      FrameDetached(render_view_host, frame_id));
 }
 
 }  // namespace content