diff options
author | slightlyoff@chromium.org <slightlyoff@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-03 01:00:19 +0000 |
---|---|---|
committer | slightlyoff@chromium.org <slightlyoff@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-03 01:00:19 +0000 |
commit | de5834cf1962af5607b459731f8960cfd3baa4c2 (patch) | |
tree | bae4be82b0b5b16994abf829992aaeba621ca595 /chrome_frame | |
parent | 2085baaafba8448c060ce566f6433a9bef4058bb (diff) | |
download | chromium_src-de5834cf1962af5607b459731f8960cfd3baa4c2.zip chromium_src-de5834cf1962af5607b459731f8960cfd3baa4c2.tar.gz chromium_src-de5834cf1962af5607b459731f8960cfd3baa4c2.tar.bz2 |
Ensures that cold startup perf tests clear more DLLs. Helps bring results into line with worst-case expectations on Vista.
BUG=None
TeST=run GCF perftests
Review URL: http://codereview.chromium.org/2544003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@48799 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame')
-rw-r--r-- | chrome_frame/test/perf/chrome_frame_perftest.cc | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/chrome_frame/test/perf/chrome_frame_perftest.cc b/chrome_frame/test/perf/chrome_frame_perftest.cc index 4806865..c0fef40 100644 --- a/chrome_frame/test/perf/chrome_frame_perftest.cc +++ b/chrome_frame/test/perf/chrome_frame_perftest.cc @@ -291,7 +291,11 @@ class ChromeFrameStartupTest : public ChromeFramePerfTestBase { chrome_frame_dll_ = dir_app_.Append(FILE_PATH_LITERAL("servers")); chrome_frame_dll_ = chrome_frame_dll_.Append( FilePath::FromWStringHack(kChromeFrameDllName)); - DLOG(INFO) << __FUNCTION__ << ": " << chrome_frame_dll_.value(); + icu_dll_ = dir_app_.Append(FILE_PATH_LITERAL("icudt42.dll")); + gears_dll_ = dir_app_.Append(FILE_PATH_LITERAL("gears.dll")); + avcodec52_dll_ = dir_app_.Append(FILE_PATH_LITERAL("avcodec-52.dll")); + avformat52_dll_ = dir_app_.Append(FILE_PATH_LITERAL("avformat-52.dll")); + avutil50_dll_ = dir_app_.Append(FILE_PATH_LITERAL("avutil-50.dll")); } virtual void TearDown() { DeleteConfigValue(kChromeFrameUnpinnedMode); @@ -319,6 +323,7 @@ class ChromeFrameStartupTest : public ChromeFramePerfTestBase { if (!ignore_cache_error) { ASSERT_TRUE(result); } else if (!result) { + LOG(ERROR) << GetLastError(); printf("\nFailed to evict file %ls from cache. Not running test\n", binaries_to_evict[binary_index].value().c_str()); return; @@ -351,6 +356,11 @@ class ChromeFrameStartupTest : public ChromeFramePerfTestBase { FilePath chrome_dll_; FilePath chrome_exe_; FilePath chrome_frame_dll_; + FilePath icu_dll_; + FilePath gears_dll_; + FilePath avcodec52_dll_; + FilePath avformat52_dll_; + FilePath avutil50_dll_; protected: // Individual startup tests should implement this function. @@ -924,7 +934,9 @@ TEST_F(ChromeFrameBinariesLoadTest, PerfWarm) { } TEST_F(ChromeFrameStartupTestActiveX, PerfCold) { - FilePath binaries_to_evict[] = {chrome_exe_, chrome_dll_, chrome_frame_dll_}; + FilePath binaries_to_evict[] = { gears_dll_, avcodec52_dll_, + avformat52_dll_, avutil50_dll_, chrome_exe_, chrome_dll_, + chrome_frame_dll_}; RunStartupTest("cold", "t", "about:blank", true /* cold */, arraysize(binaries_to_evict), binaries_to_evict, false /* not important */, false); |