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

Unified Diff: chrome/test/perf/perf_ui_test_suite.cc

Issue 15037005: Add debugging statements for problem that only happens on bots. (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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/perf/perf_ui_test_suite.cc
diff --git a/chrome/test/perf/perf_ui_test_suite.cc b/chrome/test/perf/perf_ui_test_suite.cc
index 05ebde45d5aea8d54431f64831aa9d6ae1f9f838..f6ab0cc9d06892fba2beb506a99d75ce99425b8a 100644
--- a/chrome/test/perf/perf_ui_test_suite.cc
+++ b/chrome/test/perf/perf_ui_test_suite.cc
@@ -4,6 +4,8 @@
#include "chrome/test/perf/perf_ui_test_suite.h"
+#include <stdio.h>
+
#include "base/file_util.h"
#include "base/json/json_file_value_serializer.h"
#include "base/lazy_instance.h"
@@ -43,6 +45,7 @@ base::LazyInstance<base::FilePath> g_complex_profile_dir =
PerfUITestSuite::PerfUITestSuite(int argc, char** argv)
: UITestSuite(argc, argv) {
+ fprintf(stderr, "In the constructor of the PerfUITestSuite.\n");
base::PlatformThread::SetName("Tests_Main");
}
@@ -72,23 +75,30 @@ void PerfUITestSuite::Initialize() {
//
// TODO(darin): Kill this once http://crbug.com/52609 is fixed.
base::FilePath dll;
+ fprintf(stderr, "About to get dir module.\n");
PathService::Get(base::DIR_MODULE, &dll);
dll = dll.Append(chrome::kBrowserResourcesDll);
+ fprintf(stderr, "About to LoadLibraryExW.\n");
HMODULE res_mod = ::LoadLibraryExW(dll.value().c_str(),
NULL, LOAD_LIBRARY_AS_DATAFILE | LOAD_LIBRARY_AS_IMAGE_RESOURCE);
DCHECK(res_mod);
+ fprintf(stderr, "About to _AtlBaseModule.SetResourceInstance().\n");
_AtlBaseModule.SetResourceInstance(res_mod);
+ fprintf(stderr, "About to ui::SetResourcesDataDLL.\n");
ui::SetResourcesDataDLL(_AtlBaseModule.GetResourceInstance());
#endif
+ fprintf(stderr, "About to go through UITestSuite::Initialize().\n");
UITestSuite::Initialize();
+ fprintf(stderr, "About to default_profile_dir_.CreateUniqueTempDir().\n");
if (!default_profile_dir_.CreateUniqueTempDir()) {
LOG(FATAL) << "Failed to create default profile directory...";
}
// Build a profile in default profile dir.
+ fprintf(stderr, "About to call GenerateProfile().\n");
base::FilePath default_path =
default_profile_dir_.path().AppendASCII("Default");
if (!GenerateProfile(TOP_SITES, kNumURLs, default_path)) {
@@ -97,10 +107,12 @@ void PerfUITestSuite::Initialize() {
g_default_profile_dir.Get() = default_profile_dir_.path();
+ fprintf(stderr, "About to build second temp dir().\n");
if (!complex_profile_dir_.CreateUniqueTempDir()) {
LOG(FATAL) << "Failed to create complex profile directory...";
}
+ fprintf(stderr, "About to copy profile directory.\n");
if (!file_util::CopyDirectory(default_path,
complex_profile_dir_.path(),
true)) {
@@ -117,12 +129,14 @@ void PerfUITestSuite::Initialize() {
base_data_dir = base_data_dir.AppendASCII("profile_with_complex_theme");
base_data_dir = base_data_dir.AppendASCII("Default");
+ fprintf(stderr, "About to copy extensions directory.\n");
if (!file_util::CopyDirectory(base_data_dir,
complex_profile_dir_.path(),
true)) {
LOG(FATAL) << "Failed to copy default to complex profile";
}
+ fprintf(stderr, "About to build theme pack.\n");
// Parse the manifest and make a temporary extension object because the
// theme system takes extensions as input.
base::FilePath extension_base =
@@ -134,6 +148,7 @@ void PerfUITestSuite::Initialize() {
BuildCachedThemePakIn(extension_base);
g_complex_profile_dir.Get() = complex_profile_dir_.path();
+ fprintf(stderr, "All done with init.\n");
}
void PerfUITestSuite::BuildCachedThemePakIn(
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698