Comment #9 on issue 9310 by bugdro...@chromium.org: Progress circle does  
not immediately appear when navigating away from new tab
http://code.google.com/p/chromium/issues/detail?id=9310

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=13567

------------------------------------------------------------------------
r13567 | lafo...@chromium.org | 2009-04-11 14:08:45 -0700 (Sat, 11 Apr  
2009) | 12 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/browser.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/dom_ui/dom_ui_unittest.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/tab_contents/web_contents.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/tabs/tab_strip_model.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/tabs/tab_strip_model.h?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/tabs/tab_strip_model_unittest.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/views/tabs/dragged_tab_view.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/views/tabs/tab_renderer.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/views/tabs/tab_renderer.h?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/views/tabs/tab_strip.cc?r1=13567&r2=13566
    M  
http://src.chromium.org/viewvc/chrome/branches/172/src/chrome/browser/views/tabs/tab_strip.h?r1=13567&r2=13566

Merge 13131 - Make the throbber throb sooner after you navigate. This fixes  
the new tab page,
which would not start throbbing until the load committed. I think this was  
always
broken, but switching the tab contents types covered it up.

Now I have a flag that goes along with the tab updating that indicates if  
it's
a load update or a full update. This is necessary to avoid updating the  
title
to the page's URL until it does actually commit.

BUG=9310
Review URL: http://codereview.chromium.org/60066
tbr=bre...@chromium.org
Review URL: http://codereview.chromium.org/67070
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to