Translate new UX: Fix the visual and behavior of 'Done' button

Translate new UX: Fix the visual and behavior of 'Done' button

The label is 'Translate' only when the page is not translated in the current language pair, otherwise 'Done'.

When the label is 'Translate', the view state transits to 'Translating' from 'Advanced' after clicking the button, while when 'Done', this transits to the previous view state.

BUG=324696
TEST=unit_tests --gtest_filter=TranslateBubbleViewTest.*

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238597

[hajimehoshi, 2013-12-02 08:46:37]

Can you take a look?  Thank you in advance.

[sky, 2013-12-02 21:43:58]

https://codereview.chromium.org/98643003/diff/1/chrome/browser/ui/translate/t...
File chrome/browser/ui/translate/translate_bubble_model.h (right):

https://codereview.chromium.org/98643003/diff/1/chrome/browser/ui/translate/t...
chrome/browser/ui/translate/translate_bubble_model.h:89: virtual bool
IsPageTranslatedInCurrentPair() const = 0;
'pair'? No where do you talk about what 'pair' means in this context.

[hajimehoshi, 2013-12-03 03:26:05]

Thank you!

https://codereview.chromium.org/98643003/diff/1/chrome/browser/ui/translate/t...
File chrome/browser/ui/translate/translate_bubble_model.h (right):

https://codereview.chromium.org/98643003/diff/1/chrome/browser/ui/translate/t...
chrome/browser/ui/translate/translate_bubble_model.h:89: virtual bool
IsPageTranslatedInCurrentPair() const = 0;
On 2013/12/02 21:43:58, sky wrote:
> 'pair'? No where do you talk about what 'pair' means in this context.

Sure, I avoided to use the word 'pair'.

[sky, 2013-12-03 17:23:34]

LGTM

[commit-bot: I haz the power, 2013-12-04 03:54:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hajimehoshi@chromium.org/98643003/20001

[commit-bot: I haz the power, 2013-12-04 06:13:31]

Change committed as 238597