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

Issue 2450103003: Update CTS zip file format and version (Closed)

Created:
4 years, 1 month ago by the real yoland
Modified:
4 years, 1 month ago
CC:
android-webview-reviews_chromium.org, chromium-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Update CTS zip file format and version Having only CTS zip file's path with the build id in the name can be confusing. Adding "_origin" as a comment shows where the cts zip was downloaded from CL for recipe change: https://codereview.chromium.org/2452843002 BUG=659072 Committed: https://crrev.com/ccba3490c66d514298731511e1d10378e93b7273 Cr-Commit-Position: refs/heads/master@{#427710}

Patch Set 1 #

Patch Set 2 : transitioning #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -0 lines) Patch
A android_webview/tools/cts_config/webview_cts_gcs_path_new.json View 1 1 chunk +12 lines, -0 lines 0 comments Download

Messages

Total messages: 15 (8 generated)
the real yoland
4 years, 1 month ago (2016-10-26 01:10:17 UTC) #4
the real yoland
4 years, 1 month ago (2016-10-26 01:30:23 UTC) #6
Torne
Can you update the description to make it clearer that you are actually switching to ...
4 years, 1 month ago (2016-10-26 10:10:22 UTC) #7
the real yoland
On 2016/10/26 at 10:10:22, torne wrote: > Can you update the description to make it ...
4 years, 1 month ago (2016-10-26 16:16:17 UTC) #9
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/2450103003/20001
4 years, 1 month ago (2016-10-26 16:16:45 UTC) #12
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 1 month ago (2016-10-26 16:26:16 UTC) #13
commit-bot: I haz the power
4 years, 1 month ago (2016-10-26 17:12:33 UTC) #15
Message was sent while issue was closed.
Patchset 2 (id:??) landed as
https://crrev.com/ccba3490c66d514298731511e1d10378e93b7273
Cr-Commit-Position: refs/heads/master@{#427710}

Powered by Google App Engine
This is Rietveld 408576698