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

Issue 2640963007: [wrapper-tracing] Remove flag and object grouping entry points (Closed)

Created:
3 years, 11 months ago by Michael Lippautz
Modified:
3 years, 11 months ago
CC:
Mads Ager (chromium), blink-reviews, blink-reviews-bindings_chromium.org, chromium-reviews, kinuko+watch, kouhei+heap_chromium.org, oilpan-reviews
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[wrapper-tracing] Remove flag and object grouping entry points The flag has been switched to be the default in M57. This CL removes the possibility to turn wrapper tracing off and also removes the entry points for object grouping. BUG=chromium:682688 R=jochen@chromium.org,haraken@chromium.org Review-Url: https://codereview.chromium.org/2640963007 Cr-Commit-Position: refs/heads/master@{#446014} Committed: https://chromium.googlesource.com/chromium/src/+/6255303776961de8fd0a8f6884291372200f3051

Patch Set 1 #

Patch Set 2 : Remove major GC prologue #

Patch Set 3 : One more removal #

Patch Set 4 : Rebase #

Patch Set 5 : Fix merge artifact #

Messages

Total messages: 37 (30 generated)
Michael Lippautz
ptal, branch cut happened so we should start this? The idea is: (1) Remove flag ...
3 years, 11 months ago (2017-01-25 10:13:36 UTC) #27
Michael Lippautz
+hlopko@
3 years, 11 months ago (2017-01-25 10:14:21 UTC) #29
Marcel Hlopko
I love this cl :) LGTM
3 years, 11 months ago (2017-01-25 10:20:00 UTC) #30
haraken
LGTM!
3 years, 11 months ago (2017-01-25 10:44:18 UTC) #31
jochen (gone - plz use gerrit)
lgtm
3 years, 11 months ago (2017-01-25 12:24:41 UTC) #32
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/2640963007/80001
3 years, 11 months ago (2017-01-25 14:14:47 UTC) #34
commit-bot: I haz the power
3 years, 11 months ago (2017-01-25 14:19:39 UTC) #37
Message was sent while issue was closed.
Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/6255303776961de8fd0a8f688429...

Powered by Google App Engine
This is Rietveld 408576698