Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(30)

Issue 2767703002: [Android] Record user actions of new context menus for CCT (Closed)

Created:
3 years, 9 months ago by ltian
Modified:
3 years, 9 months ago
Reviewers:
Ted C, jwd
CC:
chromium-reviews, srahim+watch_chromium.org, asvitkine+watch_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

[Android] Record user actions of new context menus for CCT Add the logic to record user actions for the newly added context menus ("Open in new Chrome tab", "Open in Chrome incognito tab", "Open in browser") for CCT. Also record the user action of "Open in Chrome" for Fullscreen mode. Change the title from "Open in Chrome incognito tab" to "Open in incognito tab" since the translation for the former one needs too much characters to show on the small screen devices. BUG=703426 Review-Url: https://codereview.chromium.org/2767703002 Cr-Commit-Position: refs/heads/master@{#459178} Committed: https://chromium.googlesource.com/chromium/src/+/d8860857f5aa2a2a358c31c1122eccca7188faf9

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -2 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuPopulator.java View 2 chunks +9 lines, -1 line 0 comments Download
M chrome/android/java/strings/android_chrome_strings.grd View 1 chunk +1 line, -1 line 0 comments Download
M tools/metrics/histograms/histograms.xml View 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 11 (5 generated)
ltian
tedchoc@chromium.org: could you take a look of my change of java/ in this CL? Thanks!
3 years, 9 months ago (2017-03-21 19:09:31 UTC) #2
ltian
jwd@chromium.org: could you take a look of my change about histograms.xml in this CL? Thanks!
3 years, 9 months ago (2017-03-21 19:11:39 UTC) #4
Ted C
lgtm
3 years, 9 months ago (2017-03-21 23:32:13 UTC) #5
jwd
lgtm
3 years, 9 months ago (2017-03-23 14:09:37 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2767703002/1
3 years, 9 months ago (2017-03-23 18:07:02 UTC) #8
commit-bot: I haz the power
3 years, 9 months ago (2017-03-23 19:30:50 UTC) #11
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d8860857f5aa2a2a358c31c1122e...

Powered by Google App Engine
This is Rietveld 408576698