diff options
| author | jcivelli@chromium.org <jcivelli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-09 20:23:15 +0000 |
|---|---|---|
| committer | jcivelli@chromium.org <jcivelli@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-09 20:23:15 +0000 |
| commit | e88079e395bf606c55e746bd618414abbcc6bc31 (patch) | |
| tree | 83088d953b0c9cf512ebb62f89ffcbb605cc2416 /chrome/chrome_installer.gypi | |
| parent | 76b2ad7e89d95b6b3934563cf625dfc493fca4c5 (diff) | |
| download | chromium_src-e88079e395bf606c55e746bd618414abbcc6bc31.zip chromium_src-e88079e395bf606c55e746bd618414abbcc6bc31.tar.gz chromium_src-e88079e395bf606c55e746bd618414abbcc6bc31.tar.bz2 | |
Refactored the translate infobars.
Since some work is needed on Linux and Mac to use the new translate infobar delegate, I created a new version of the refactored classes instead of replacing them. Once Linux and Mac use the new classes, we can make the swicth.
The TranslateInfoBarDelegate now contains all states, so there is no more logic on the infobar classes.
I broke down the single infobar class into multiple ones (there is now an infobar for each state: before translate, translating, after translate, error): it makes the code simpler.
I had to fix ReplaceInfoBar on Windows as it was not working properly.
BUG=40828
TEST=Test thoroughly the translate feature.
Review URL: http://codereview.chromium.org/2602003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@49307 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/chrome_installer.gypi')
0 files changed, 0 insertions, 0 deletions