Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(797)

Issue 2760103002: Merge WebApkConstants.EXTRA_WEBAPK_PACKAGE_NAME and ShortcutHelper.EXTRA_WEBAPK_PACKAGE_NAME (Closed)

Created:
1 year, 1 month ago by pkotwicz
Modified:
1 year, 1 month ago
Reviewers:
dominickn
CC:
chromium-reviews, dominickn+watch_chromium.org, pkotwicz+watch_chromium.org, zpeng+watch_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Merge WebApkConstants.EXTRA_WEBAPK_PACKAGE_NAME and ShortcutHelper.EXTRA_WEBAPK_PACKAGE_NAME BUG=702027 Review-Url: https://codereview.chromium.org/2760103002 Cr-Commit-Position: refs/heads/master@{#458423} Committed: https://chromium.googlesource.com/chromium/src/+/763a6232141799360238b6f53b728525c171d7bf

Patch Set 1 #

Patch Set 2 : Merge branch 'master' into package_name #

Total comments: 1

Messages

Total messages: 8 (4 generated)
pkotwicz
Dominick, can you please take a look?
1 year, 1 month ago (2017-03-20 21:17:44 UTC) #2
dominickn
lgtm https://codereview.chromium.org/2760103002/diff/20001/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappInfo.java File chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappInfo.java (right): https://codereview.chromium.org/2760103002/diff/20001/chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappInfo.java#newcode18 chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappInfo.java:18: import org.chromium.webapk.lib.common.WebApkConstants; I think you can revert the ...
1 year, 1 month ago (2017-03-20 23:59:04 UTC) #3
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/2760103002/20001
1 year, 1 month ago (2017-03-21 14:59:36 UTC) #5
commit-bot: I haz the power
1 year, 1 month ago (2017-03-21 15:39:14 UTC) #8
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/763a6232141799360238b6f53b72...

Powered by Google App Engine
This is Rietveld 408576698