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

Unified Diff: chrome/browser/extensions/api/system_indicator/system_indicator_apitest.cc

Issue 15841013: Make miscellaneous_bindings and event_bindings Required as needed. Previously (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/api/system_indicator/system_indicator_apitest.cc
diff --git a/chrome/browser/extensions/api/system_indicator/system_indicator_apitest.cc b/chrome/browser/extensions/api/system_indicator/system_indicator_apitest.cc
index 616fad4138f8bc90b3403aa76a3bf173ce55a677..8a57962e8c0b67936d1a6da1de3851ade3556be4 100644
--- a/chrome/browser/extensions/api/system_indicator/system_indicator_apitest.cc
+++ b/chrome/browser/extensions/api/system_indicator/system_indicator_apitest.cc
@@ -29,8 +29,11 @@ class SystemIndicatorApiTest : public ExtensionApiTest {
LazyBackgroundObserver page_complete;
base::FilePath extdir = test_data_dir_.AppendASCII(test_name);
const extensions::Extension* extension = LoadExtension(extdir);
- if (extension)
+ if (extension) {
+ LOG(INFO) << "waiting for page complete";
page_complete.Wait();
+ LOG(INFO) << "done";
+ }
return extension;
}
};
@@ -57,6 +60,8 @@ IN_PROC_BROWSER_TEST_F(SystemIndicatorApiTest, SystemIndicator) {
LoadExtensionAndWait("system_indicator/unloaded");
ASSERT_TRUE(extension) << message_;
+ LOG(INFO) << "ran test";
+
// Lazy Background Page has been shut down.
ExtensionProcessManager* pm =
extensions::ExtensionSystem::Get(profile())->process_manager();

Powered by Google App Engine
This is Rietveld 408576698