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

Issue 393033009: roll gyp 1951:1953 (Closed)

Created:
6 years, 5 months ago by Nico
Modified:
6 years, 5 months ago
Reviewers:
scottmg
CC:
chromium-reviews
Project:
chromium
Visibility:
Public.

Description

roll gyp 1951:1953 1952: Add test files for https://codereview.chromium.org/406523005/ 1953: ninja/win: Put common msvs_system_include_dirs into %INCLUDE% BUG=395405 R=scottmg@chromium.org TBR=scottmg Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=284370

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (0 generated)
Nico
6 years, 5 months ago (2014-07-20 16:18:27 UTC) #1
Nico
The CQ bit was checked by thakis@chromium.org
6 years, 5 months ago (2014-07-20 16:18:30 UTC) #2
Nico
The CQ bit was unchecked by thakis@chromium.org
6 years, 5 months ago (2014-07-20 16:18:43 UTC) #3
Nico
The CQ bit was checked by thakis@chromium.org
6 years, 5 months ago (2014-07-20 16:19:16 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/393033009/1
6 years, 5 months ago (2014-07-20 16:20:04 UTC) #5
scottmg
lgtm
6 years, 5 months ago (2014-07-20 16:38:02 UTC) #6
Nico
Committed patchset #1 manually as r284370 (tree was closed).
6 years, 5 months ago (2014-07-20 17:28:21 UTC) #7
Nico
6 years, 5 months ago (2014-07-20 17:28:59 UTC) #8
Message was sent while issue was closed.
On 2014/07/20 17:28:21, Nico (away) wrote:
> Committed patchset #1 manually as r284370 (tree was closed).

(in part to check if the content_tests failure on the android bot will stay
around for another round; in that case the v8 roll probably needs to be
reverted.)

Powered by Google App Engine
This is Rietveld 408576698