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

Issue 2503043002: Reland "[wrapper-tracing] Enable flag per default" (Closed)

Created:
4 years, 1 month ago by Michael Lippautz
Modified:
4 years, 1 month ago
CC:
apavlov+blink_chromium.org, blink-reviews, blink-reviews-bindings_chromium.org, caseq+blink_chromium.org, chromium-reviews, devtools-reviews_chromium.org, kozyatinskiy+blink_chromium.org, lushnikov+blink_chromium.org, pfeldman+blink_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reland "[wrapper-tracing] Enable flag per default" BUG=chromium:468240 Committed: https://crrev.com/e23de6962be8b4967127ae95040f32bbb533b4d1 Cr-Commit-Position: refs/heads/master@{#433823}

Patch Set 1 #

Messages

Total messages: 29 (17 generated)
Michael Lippautz
The incremental flag on V8 is now set to true, so we shouldn't see any ...
4 years, 1 month ago (2016-11-18 13:02:33 UTC) #12
jochen (gone - plz use gerrit)
lgtm
4 years, 1 month ago (2016-11-18 14:00:53 UTC) #13
haraken
LGTM
4 years, 1 month ago (2016-11-18 15:26:40 UTC) #14
Hannes Payer (out of office)
so LGTM
4 years, 1 month ago (2016-11-21 17:02:56 UTC) #15
Marcel Hlopko
lgtm
4 years, 1 month ago (2016-11-21 17:12:24 UTC) #16
Michael Lippautz
Back in office tomorrow, so I am trying to get it into a Canary.
4 years, 1 month ago (2016-11-21 18:46:55 UTC) #17
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/2503043002/20001
4 years, 1 month ago (2016-11-21 18:47:22 UTC) #19
commit-bot: I haz the power
Try jobs failed on following builders: win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_ng/builds/336226)
4 years, 1 month ago (2016-11-21 22:41:17 UTC) #21
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/2503043002/20001
4 years, 1 month ago (2016-11-22 07:58:30 UTC) #23
commit-bot: I haz the power
Committed patchset #1 (id:20001)
4 years, 1 month ago (2016-11-22 09:25:05 UTC) #26
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/e23de6962be8b4967127ae95040f32bbb533b4d1 Cr-Commit-Position: refs/heads/master@{#433823}
4 years, 1 month ago (2016-11-22 09:28:44 UTC) #28
Michael Lippautz
4 years ago (2016-11-28 10:41:44 UTC) #29
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:20001) has been created in
https://codereview.chromium.org/2528393002/ by mlippautz@chromium.org.

The reason for reverting is: Reverting because of upcomming Dev release. Will
re-enable tomorrow after the release is done..

Powered by Google App Engine
This is Rietveld 408576698