Chromium Code Reviews
Help | Chromium Project | Sign in
(44)

Issue 11367081: Revert 165808 - Enable PlatformAppBrowserTest.DevToolsOpenedWithReload for debug builds. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 8 months ago by vandebo (ex-Chrome)
Modified:
2 years, 8 months ago
Reviewers:
tapted
CC:
chromium-reviews, Aaron Boodman, chromium-apps-reviews_chromium.org
Visibility:
Public.

Description

Revert 165808 - Enable PlatformAppBrowserTest.DevToolsOpenedWithReload for debug builds. [28577:28577:1102/204920:1539270891:FATAL:ref_counted.cc(81)] Check failed: !in_dtor_. http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/22007 Webkit Roll in r165534 and fix in https://bugs.webkit.org/show_bug.cgi?id=100930 (WebKit revision 133188) means that this test will no longer cause a segfault on Debug builds due to http://crbug.com/157097. BUG=158283 TEST=browser_tests --gtest_filter=PlatformAppBrowserTest.DevToolsOpenedWithReload Review URL: https://chromiumcodereview.appspot.com/11366052 TBR=tapted@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=165846

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -1 line) Patch
M chrome/browser/extensions/platform_app_browsertest.cc View 1 chunk +11 lines, -1 line 0 comments Download
Trybot results:
Commit: CQ not working?

Messages

Total messages: 2 (0 generated)
vandebo (ex-Chrome)
2 years, 8 months ago (2012-11-03 05:17:53 UTC) #1
tapted
2 years, 8 months ago (2012-11-03 11:54:30 UTC) #2
lgtm - thanks. I'll look into it.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1f9106d