DevTools: fixed test ExtensionApiTest.Debugger

BUG=380082,234166

TBR=finnur

Review URL: https://codereview.chromium.org/333223003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@277740 0039d316-1c4b-4281-b951-d872f2087c98
parent 14eee11a
...@@ -6,14 +6,6 @@ ...@@ -6,14 +6,6 @@
#include "chrome/browser/extensions/extension_apitest.h" #include "chrome/browser/extensions/extension_apitest.h"
#include "chrome/common/chrome_switches.h" #include "chrome/common/chrome_switches.h"
// Debugger is flaky on browser_tests on Windows, Linux, and Chrome OS: IN_PROC_BROWSER_TEST_F(ExtensionApiTest, Debugger) {
// crbug.com/234166.
#if defined(OS_WIN) || defined(OS_LINUX) || defined(OS_CHROMEOS)
#define MAYBE(x) DISABLED_##x
#else
#define MAYBE(x) x
#endif
IN_PROC_BROWSER_TEST_F(ExtensionApiTest, MAYBE(Debugger)) {
ASSERT_TRUE(RunExtensionTest("debugger")) << message_; ASSERT_TRUE(RunExtensionTest("debugger")) << message_;
} }
...@@ -140,7 +140,9 @@ chrome.test.runTests([ ...@@ -140,7 +140,9 @@ chrome.test.runTests([
}, },
function createAndDiscoverTab() { function createAndDiscoverTab() {
function onUpdated(tabId) { function onUpdated(tabId, changeInfo) {
if (changeInfo.status == 'loading')
return;
chrome.tabs.onUpdated.removeListener(onUpdated); chrome.tabs.onUpdated.removeListener(onUpdated);
chrome.debugger.getTargets(function(targets) { chrome.debugger.getTargets(function(targets) {
var page = targets.filter( var page = targets.filter(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment