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

Stop the location bar from flashing white when navigating from one HTTPS site to another.  To do this, we have to make sure the SSLManager sees the DidCommitProvisionalLoad event before the location bar draws itself to the screen.

BUG=11157
TEST=Navigate from one HTTPS site to another and watch the location bar carefully.  It shouln't flash anymore.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=15020

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

LGTM

http://codereview.chromium.org/99247/diff/1/4
File chrome/browser/tab_contents/navigation_controller.cc (right):

http://codereview.chromium.org/99247/diff/1/4#newcode869
Line 869: NotificationDetails out_details =
Details<LoadCommittedDetails>(details);
Nit: Perhaps |notification_details| instead of |out_details|?

http://codereview.chromium.org/99247/diff/1/4#newcode873
Line 873: // This is to work around the fact that parts of the location bar draw
I would just eliminate these last two lines and append "when it wants to draw"
to the end of the previous sentence.