Commit 6441331e authored by Megan Jablonski's avatar Megan Jablonski Committed by Commit Bot

Fix Translate Desktop UI Return action

Ensure that the focus is traversing on the language in which the page is
translated.

Bug: 1105807
Change-Id: I14a2b7a957c5ad8b581ca25a0d89a406ba3bfc52
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2300321Reviewed-by: default avataranthonyvd <anthonyvd@chromium.org>
Commit-Queue: Megan Jablonski <megjablon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#790082}
parent 62e2df93
......@@ -379,7 +379,6 @@ bool TranslateBubbleView::AcceleratorPressed(
break;
}
case TranslateBubbleModel::VIEW_STATE_TRANSLATING:
break;
case TranslateBubbleModel::VIEW_STATE_AFTER_TRANSLATE: {
if (accelerator.key_code() == ui::VKEY_RETURN) {
ShowOriginal();
......@@ -591,12 +590,14 @@ views::View* TranslateBubbleView::GetCurrentView() const {
void TranslateBubbleView::Translate() {
model_->Translate();
model_->SetViewState(TranslateBubbleModel::VIEW_STATE_TRANSLATING);
SwitchView(TranslateBubbleModel::VIEW_STATE_TRANSLATING);
translate::ReportUiAction(translate::TRANSLATE_BUTTON_CLICKED);
}
void TranslateBubbleView::ShowOriginal() {
model_->RevertTranslation();
model_->SetViewState(TranslateBubbleModel::VIEW_STATE_BEFORE_TRANSLATE);
SwitchView(TranslateBubbleModel::VIEW_STATE_BEFORE_TRANSLATE);
translate::ReportUiAction(translate::SHOW_ORIGINAL_BUTTON_CLICKED);
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment