diff options
| author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-27 12:22:05 +0000 |
|---|---|---|
| committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-27 12:22:05 +0000 |
| commit | cb793485ece8cf39cd13d1b365f8c570c6775ca3 (patch) | |
| tree | a6ec918306db0c9526ba1fd5ad913d269d5e484a /chrome/browser/browser_browsertest.cc | |
| parent | e8fa42e23401cded7df5bc4da7fd166bbebb413c (diff) | |
| download | chromium_src-cb793485ece8cf39cd13d1b365f8c570c6775ca3.zip chromium_src-cb793485ece8cf39cd13d1b365f8c570c6775ca3.tar.gz chromium_src-cb793485ece8cf39cd13d1b365f8c570c6775ca3.tar.bz2 | |
[Mac] Fix a memory corruption when closing a JS modal dialog.
JSModalDialogCocoa::CloseAppModalDialog was calling [NSApp endSheet],
which in turn was deleting the JSModalDialogCocoa (via a scoped_ptr
in alertDidEnd).
After that, the CloseAppModalDialog was writing NULL to the alert_
member variable, but at that point the object was deleted,
so it was corrupting memory.
GuardMalloc detected this corruption very quickly.
It was also causing BrowserTest.JavascriptAlertActivatesTab to crash
flakily on Mac. This change undisables that test.
The bug contains more details, context, and stack traces.
BUG=54896
TEST=browser_tests
Review URL: http://codereview.chromium.org/5318007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67493 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_browsertest.cc')
| -rw-r--r-- | chrome/browser/browser_browsertest.cc | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/chrome/browser/browser_browsertest.cc b/chrome/browser/browser_browsertest.cc index b306ec1..64fa35ca 100644 --- a/chrome/browser/browser_browsertest.cc +++ b/chrome/browser/browser_browsertest.cc @@ -197,14 +197,7 @@ IN_PROC_BROWSER_TEST_F(BrowserTest, Title) { EXPECT_EQ(WideToUTF16(test_title), tab_title); } -#if defined(OS_MACOSX) -// Test is crashing on Mac, see http://crbug.com/29424. -#define MAYBE_JavascriptAlertActivatesTab DISABLED_JavascriptAlertActivatesTab -#else -#define MAYBE_JavascriptAlertActivatesTab JavascriptAlertActivatesTab -#endif - -IN_PROC_BROWSER_TEST_F(BrowserTest, MAYBE_JavascriptAlertActivatesTab) { +IN_PROC_BROWSER_TEST_F(BrowserTest, JavascriptAlertActivatesTab) { GURL url(ui_test_utils::GetTestUrl(FilePath(FilePath::kCurrentDirectory), FilePath(kTitle1File))); ui_test_utils::NavigateToURL(browser(), url); |