Make all of our unit tests be PlatformTests to get an autorelease pool....

Make all of our unit tests be PlatformTests to get an autorelease pool.
BUG=none
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=25035

[Mark Mentovai, 2009-08-31 19:03:23]

See, I actually read this.

LGTM, but check that you intend to disable the ones you're disabling.

http://codereview.chromium.org/178037/diff/1/23
File chrome/browser/cocoa/bookmark_editor_controller_unittest.mm (right):

http://codereview.chromium.org/178037/diff/1/23#newcode63
Line 63: TEST_F(BookmarkEditorControllerTest, DISABLED_UserEditsStuff) {
Intentionally disabling this?

http://codereview.chromium.org/178037/diff/1/6
File chrome/browser/cocoa/tab_controller_unittest.mm (right):

http://codereview.chromium.org/178037/diff/1/6#newcode154
Line 154: TEST_F(TabControllerTest, DISABLED_UserSelection) {
This too.

[pink (ping after 24hrs), 2009-08-31 19:06:04]

Thanks for catching these. Leftovers from debugging on SL last week.

http://codereview.chromium.org/178037/diff/1/23
File chrome/browser/cocoa/bookmark_editor_controller_unittest.mm (right):

http://codereview.chromium.org/178037/diff/1/23#newcode63
Line 63: TEST_F(BookmarkEditorControllerTest, DISABLED_UserEditsStuff) {
On 2009/08/31 19:03:24, Mark Mentovai wrote:
> Intentionally disabling this?

Done.

http://codereview.chromium.org/178037/diff/1/6
File chrome/browser/cocoa/tab_controller_unittest.mm (right):

http://codereview.chromium.org/178037/diff/1/6#newcode154
Line 154: TEST_F(TabControllerTest, DISABLED_UserSelection) {
On 2009/08/31 19:03:24, Mark Mentovai wrote:
> This too.

Done.