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

Issue 8387059: Enable default object tracking no Mac (Closed)

Created:
9 years, 1 month ago by jar (doing other things)
Modified:
9 years, 1 month ago
CC:
chromium-reviews, brettw-cc_chromium.org
Visibility:
Public.

Description

Enable default object tracking no Mac We had trouble with base_unittests before, so we ifdef'ed out the larger landing. This tiny switch CL will turn it on for the Mac, but can be quickly reverted if there is a problem. Scott Hess was able to debug and identify the cause of the problem (yeah Scott!!!!). I think the code we landed corrects for the issue he identified, and this should prove it (or get reverted trying). r=rtenneti,shess bug=101856 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108072

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -5 lines) Patch
M base/tracked_objects.cc View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 11 (0 generated)
jar (doing other things)
9 years, 1 month ago (2011-10-31 23:31:26 UTC) #1
Scott Hess - ex-Googler
So this'll work off trunk if I patch it in and try it?
9 years, 1 month ago (2011-10-31 23:33:48 UTC) #2
jar (doing other things)
For sure! On Mon, Oct 31, 2011 at 4:33 PM, <shess@chromium.org> wrote: > So this'll ...
9 years, 1 month ago (2011-10-31 23:43:49 UTC) #3
Scott Hess - ex-Googler
On 2011/10/31 23:33:48, shess wrote: > So this'll work off trunk if I patch it ...
9 years, 1 month ago (2011-10-31 23:49:41 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jar@chromium.org/8387059/1
9 years, 1 month ago (2011-10-31 23:55:38 UTC) #5
Scott Hess - ex-Googler
On 2011/10/31 23:49:41, shess wrote: > On 2011/10/31 23:33:48, shess wrote: > > So this'll ...
9 years, 1 month ago (2011-10-31 23:56:51 UTC) #6
commit-bot: I haz the power
Try job failure for 8387059-1 (retry) on linux_rel for step "ui_tests". It's a second try, ...
9 years, 1 month ago (2011-11-01 00:57:28 UTC) #7
Scott Hess - ex-Googler
On 2011/11/01 00:57:28, I haz the power (commit-bot) wrote: > Try job failure for 8387059-1 ...
9 years, 1 month ago (2011-11-01 04:18:49 UTC) #8
Scott Hess - ex-Googler
On 2011/11/01 04:18:49, shess wrote: > On 2011/11/01 00:57:28, I haz the power (commit-bot) wrote: ...
9 years, 1 month ago (2011-11-01 04:20:56 UTC) #9
jar (doing other things)
I manually committed it... so there is a good excuse for precluding complying with additional ...
9 years, 1 month ago (2011-11-01 04:50:39 UTC) #10
Scott Hess - ex-Googler
9 years, 1 month ago (2011-11-01 05:03:25 UTC) #11
On 2011/11/01 04:50:39, jar wrote:
> I manually committed it... so there is a good excuse for precluding
> complying with additional checkboxes ;-).

hahahah.  did not even occur to me to look.

Powered by Google App Engine
This is Rietveld 408576698