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

Issue 620945 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 2
Type: Feature



Sign in to add a comment

log every time we kill the prompt for a user after X negative actions

Project Member Reported by juliecattiau@google.com, Jun 17 2016

Issue description

In the existing Translate Chrome bubble UI, we kill the prompt after 2 negative actions. In the new experiment, we kill the bubble after 3 negative actions.

In both cases we want to log every time we kill the prompt, due to a certain number of negative actions.
 
Cc: groby@chromium.org chaotian@chromium.org

Comment 2 by groby@chromium.org, Jun 17 2016

Status: Assigned (was: Untriaged)
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 17 2016

Labels: Hotlist-Google

Comment 4 by ftang@chromium.org, Jun 29 2016

Components: UI>Browser>Translate
Labels: Hotlist-Translate
Owner: napper@chromium.org
Components: -UI>Browser>Translate UI>Browser>Language>Translate
Project Member

Comment 7 by bugdroid1@chromium.org, Jun 28 2017

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

commit 145524431de20a7e4d8738065b7855132c02e250
Author: Jon Napper <napper@chromium.org>
Date: Tue Jun 27 23:59:56 2017

Added translate logging for too often denied / matches previous language

Added missing translation logging for the cases where the language
matches the previous language, or the language has been denied too many
times. Also updated the translate browser metrics unit test to include
missing metrics.

Bug:  620945 
Change-Id: I4a1d3a4700012cd6de9938b5e57f58a64dceec25
Reviewed-on: https://chromium-review.googlesource.com/547177
Commit-Queue: Jon Napper <napper@chromium.org>
Reviewed-by: Patricia Lor <patricialor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#482802}
[modify] https://crrev.com/145524431de20a7e4d8738065b7855132c02e250/components/translate/core/browser/translate_browser_metrics.h
[modify] https://crrev.com/145524431de20a7e4d8738065b7855132c02e250/components/translate/core/browser/translate_browser_metrics_unittest.cc
[modify] https://crrev.com/145524431de20a7e4d8738065b7855132c02e250/components/translate/core/browser/translate_manager.cc
[modify] https://crrev.com/145524431de20a7e4d8738065b7855132c02e250/components/translate/core/browser/translate_manager_unittest.cc
[modify] https://crrev.com/145524431de20a7e4d8738065b7855132c02e250/tools/metrics/histograms/enums.xml

Comment 8 by napper@chromium.org, Jun 28 2017

Status: Fixed (was: Assigned)

Sign in to add a comment