Issue 147123010: Re-enable the two blink heap unit tests.

Issue 147123010: Re-enable the two blink heap unit tests. (Closed)

Created:
6 years, 10 months ago by wibling-chromium

Modified:
6 years, 10 months ago

Reviewers:
haraken, Vyacheslav Egorov (Chromium), Erik Corry, zerny-chromium, Mads Ager (chromium)

CC:
chromium-reviews, craigdh+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org, klundberg+watch_chromium.org, frankf+watch_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

Re-enable the two blink heap unit tests. These should work now after fixing bug 340591. R=ager@chromium.org, erik.corry@gmail.com, haraken@chromium.org, vegorov@chromium.org, zerny@chromium.org BUG=340591 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=249388

Patch Set 1 #

Created: 6 years, 10 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+0 lines, -3 lines)			Patch
	M	build/android/pylib/gtest/filter/blink_heap_unittests_disabled	View		1 chunk	+0 lines, -3 lines	0 comments	Download

Messages

Total messages: 9 (0 generated)

Expand Messages | Collapse Messages

Mads Ager (chromium)

Could you add the bug number to the BUG= line?

6 years, 10 months ago (2014-02-06 08:51:40 UTC) #3

wibling-chromium

On 2014/02/06 08:51:40, Mads Ager (chromium) wrote: > Could you add the bug number to ...

6 years, 10 months ago (2014-02-06 08:57:27 UTC) #5

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/wibling@chromium.org/147123010/1

6 years, 10 months ago (2014-02-06 09:02:29 UTC) #8

Message was sent while issue was closed.

Change committed as 249388

Expand Messages | Collapse Messages