diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-16 23:22:42 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-16 23:22:42 +0000 |
| commit | 69fe7af6d2ffcc52d4539d624eca48524e7ca2b3 (patch) | |
| tree | 8e7c52fe7fe4e356bc0f405ac92bf9b96e7bcf6e /chrome/browser/dom_ui/passwords_exceptions_handler.cc | |
| parent | f53846073932e5f6bcd01ffca11a317796fceb59 (diff) | |
| download | chromium_src-69fe7af6d2ffcc52d4539d624eca48524e7ca2b3.zip chromium_src-69fe7af6d2ffcc52d4539d624eca48524e7ca2b3.tar.gz chromium_src-69fe7af6d2ffcc52d4539d624eca48524e7ca2b3.tar.bz2 | |
Preserve the correct number of visible browser actions across restarts.
The problem here was that in the case where the container was sized all the way open, we never saved the result, because we never called Animate(). Having the size saving happen at the end of animation seemed subtle, so this:
* Saves at the beginning, not end, of animation (so shutdowns during animation don't forget the right size)
* Renames Animate() to make it clearer that it's also saving the desired size
* In general simplifies things by basing animation on a target number of icons instead of a target width
BUG=51179
TEST=Size browser actions container to hide one action (and show the rest); restart; size to show all; restart. On both restarts, you should see the same number of visible actions as at shut down.
Review URL: http://codereview.chromium.org/3114014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@56258 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/dom_ui/passwords_exceptions_handler.cc')
0 files changed, 0 insertions, 0 deletions