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

Issue 3008009: DISABLE WrenchMenuControllerTest.Initialized. (Closed)

Created:
10 years, 5 months ago by cbentzel
Modified:
9 years, 7 months ago
Reviewers:
Robert Sesek
CC:
chromium-reviews, John Grabowski, ben+cc_chromium.org, pam+watch_chromium.org, Paweł Hajdan Jr.
Visibility:
Public.

Description

DISABLE WrenchMenuControllerTest.Initialized. Also made other DISABLED tests consistent with this one. BUG=49206 TEST=None Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52914

Patch Set 1 #

Patch Set 2 : Change bug that comment points to. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -5 lines) Patch
M chrome/browser/cocoa/wrench_menu_controller_unittest.mm View 1 2 chunks +4 lines, -5 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
cbentzel
10 years, 5 months ago (2010-07-18 12:56:51 UTC) #1
cbentzel
On 2010/07/18 12:56:51, cbentzel wrote: > I'll update the comment to r49206
10 years, 5 months ago (2010-07-18 13:28:56 UTC) #2
cbentzel
Updated comment - should I land this?
10 years, 5 months ago (2010-07-19 16:48:17 UTC) #3
Robert Sesek
10 years, 5 months ago (2010-07-19 16:49:57 UTC) #4
I thought you TBRed it -- sorry. LG.

rsesek / @chromium.org

On Mon, Jul 19, 2010 at 12:48 PM, <cbentzel@chromium.org> wrote:

> Updated comment - should I land this?
>
>
>
>
> http://codereview.chromium.org/3008009/show
>

Powered by Google App Engine
This is Rietveld 408576698