Issue 3297: Fix retardation in app window throbber updating
http://code.google.com/p/chromium/issues/detail?id=3297
New issue report by [EMAIL PROTECTED]:
To update the throbber on an app window, the following steps occur:
Browser::LoadingStateChanged
TabStripModel::UpdateTabContentsLoadingAnimations
TabStrip::TabValidateAnimations
TabStrip::LoadingAnimationCallback
Browser::ValidateLoadingAnimations
BrowserView2::ValidateThrobber
OpaqueFrame::UpdateThrobber
... (updates throbber)
I think we can compress this to:
Browser::LoadingStateChanged
BrowserView2::ValidateLoadingAnimations
BrowserView2::LoadingAnimationCallback
OpaqueFrame::UpdateThrobber
... (updates throbber)
... for the app window case. There's no reason to involve the TabStrip.
Rather, if we want to continue with this single source of animation timer,
we should make BrowserView2 be that source, since it's constant in both
cases (windows with or without TabStrips). The flow for windows with
TabStrips would look like this:
Browser::LoadingStateChanged
BrowserView2::ValidateLoadingAnimations
BrowserView2::LoadingAnimationCallback
TabStrip::UpdateLoadingAnimations
... (updates throbber)
This approach avoids involving the TabStrip where it's not required, and a
roundtrip through Browser (!)
This is probably most easily accomplished once the old frame code is
removed.
Issue attributes:
Status: Assigned
Owner: [EMAIL PROTECTED]
Labels: Type-Bug Pri-2 OS-Windows Area-BrowserUI
--
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
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Chromium-bugs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/chromium-bugs?hl=en
-~----------~----~----~----~------~----~------~--~---
