Stop the location bar from flashing white when navigating from one HTTPS site...

chrome/browser/tab_contents/navigation_controller.cc

Index: chrome/browser/tab_contents/navigation_controller.cc
===================================================================
--- chrome/browser/tab_contents/navigation_controller.cc	(revision 14874)
+++ chrome/browser/tab_contents/navigation_controller.cc	(working copy)
@@ -865,17 +865,25 @@
 
 void NavigationController::NotifyNavigationEntryCommitted(
     LoadCommittedDetails* details) {
+  details->entry = GetActiveEntry();
+  NotificationDetails out_details = Details<LoadCommittedDetails>(details);

[Peter Kasting, 2009/04/30 22:07:54]

Nit: Perhaps |notification_details| instead of |out_details|?

+
+  // We need to notify the ssl_manager_ before the tab_contents_ so the
+  // location bar will have up-to-date information about the security style.
+  // This is to work around the fact that parts of the location bar draw

[Peter Kasting, 2009/04/30 22:07:54]

I would just eliminate these last two lines and append "when it wants to draw"
to the end of the previous sentence.

+  // synchronously.  See http://crbug.com/11157
+  ssl_manager_.DidCommitProvisionalLoad(out_details);
+
   // TODO(pkasting): http://b/1113079 Probably these explicit notification paths
   // should be removed, and interested parties should just listen for the
   // notification below instead.
   tab_contents_->NotifyNavigationStateChanged(
       TabContents::INVALIDATE_EVERYTHING);
 
-  details->entry = GetActiveEntry();
   NotificationService::current()->Notify(
       NotificationType::NAV_ENTRY_COMMITTED,
       Source<NavigationController>(this),
-      Details<LoadCommittedDetails>(details));
+      out_details);
 }
 
 // static