New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 734656 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Translate option menu should be dismissed after a new intent

Project Member Reported by klo...@chromium.org, Jun 19 2017

Issue description

Make sure you have the new translate UI on.

Load a page to trigger the new translation infobar. Click on the three dots to show the translate overflow menu.

Leave Chrome, go to another app, click a web link to trigger an external intent.

Noticed the infobar is not there, but translate overflow menu stays.
 
Screenshot_20170618-123806.png
397 KB View Download
Owner: goog...@chromium.org

Comment 2 by martiw@chromium.org, Jun 20 2017

Owner: martiw@chromium.org

Comment 3 by martiw@chromium.org, Jun 20 2017

Status: Started (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a1d3a65da2d4e255a001fbcbdde583938d919fc3

commit a1d3a65da2d4e255a001fbcbdde583938d919fc3
Author: martiw <martiw@chromium.org>
Date: Fri Jul 07 00:04:54 2017

Dismiss menu when infobar's host tab is hidden. (new translate infobar)

When host tab of infobar is being hidden (this occurs when Chrome app
lost focus or the tab is being switched out), dismiss all existing
overflow menus and snackbars.
It is to prevent the overflow menu (or snackbar) from keep showing when
the chrome app is re-open with a new tab.

BUG= 734656 

Review-Url: https://codereview.chromium.org/2952553002
Cr-Commit-Position: refs/heads/master@{#484775}

[modify] https://crrev.com/a1d3a65da2d4e255a001fbcbdde583938d919fc3/chrome/android/java/src/org/chromium/chrome/browser/infobar/TranslateCompactInfoBar.java

Comment 5 by martiw@chromium.org, Jul 10 2017

Status: Fixed (was: Started)

Sign in to add a comment