Issue 10837103: Fix deallocation bug in TestingAutomationProvider

Issue 10837103: Fix deallocation bug in TestingAutomationProvider (Closed)

Created:
8 years, 4 months ago by Russ Harmon

Modified:
8 years, 4 months ago

Reviewers:
craigdh, darin (slow to review), Nirnimesh

CC:
chromium-reviews, robertshield, kkania

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

Visibility:
Public.

Fix deallocation bug in TestingAutomationProvider TBR=craigdh,nirnimesh Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=150413

Created: 8 years, 4 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+2 lines, -2 lines)			Patch
	M	chrome/browser/automation/testing_automation_provider.cc	View		1 chunk	+2 lines, -2 lines	0 comments	Download

Total messages: 10 (0 generated)

craigdh

My bad -- a DCHECK fails when reply goes out of scope. Thanks Russell.

8 years, 4 months ago (2012-08-03 21:04:30 UTC) #1

commit-bot: I haz the power

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

8 years, 4 months ago (2012-08-03 22:23:18 UTC) #7

commit-bot: I haz the power

Try job failure for 10837103-1 (retry) on win_rel for step "compile" (clobber build). It's a ...

8 years, 4 months ago (2012-08-03 22:49:48 UTC) #8

commit-bot: I haz the power

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

8 years, 4 months ago (2012-08-07 19:25:49 UTC) #9

Change committed as 150413