Changing the pre-reading of chrome.dll to read it as an image section instead...

chrome_frame/test/perf/chrome_frame_perftest.cc

 // Copyright (c) 2010 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 #include "chrome_frame/test/perf/chrome_frame_perftest.h"
 
 #include <atlwin.h>
 #include <atlhost.h>
 #include <map>
 #include <vector>
 #include <string>
 
 #include "chrome_tab.h"  // Generated from chrome_tab.idl.
 
 #include "base/file_path.h"
+#include "base/file_util.h"
 #include "base/path_service.h"
 #include "base/process_util.h"
 #include "base/registry.h"
 #include "base/scoped_ptr.h"
 #include "base/scoped_bstr_win.h"
 #include "base/scoped_comptr_win.h"
 #include "base/scoped_variant_win.h"
 #include "base/string_util.h"
 #include "base/time.h"
 #include "chrome/common/chrome_constants.h"
@@ skipping 372 matching lines @@
     CComObjectStackEx<ChromeFrameActiveXContainer> wnd;
     wnd.CreateChromeFrameWindow(startup_url_);
     wnd.CreateControl(true);
     module.RunMessageLoop();
     *end_time = TimeTicks::Now();
   }
 };
 
 // This class measures the load time of chrome and chrome frame binaries
 class ChromeFrameBinariesLoadTest : public ChromeFrameStartupTestActiveX {
+  static const size_t kStepSize = 4 * 1024;
+ public:
+  ChromeFrameBinariesLoadTest()
+      : pre_read_(false),
+        step_size_(kStepSize),
+        bytes_to_read_(0) {}
+
  protected:
   virtual void RunStartupTestImpl(TimeTicks* start_time,
                                   TimeTicks* end_time) {
     *start_time = TimeTicks::Now();
 
+    if (pre_read_) {
+      EXPECT_TRUE(file_util::PreReadImage(chrome_exe_.value().c_str(),
+                                          bytes_to_read_,
+                                          step_size_));
+      EXPECT_TRUE(file_util::PreReadImage(chrome_dll_.value().c_str(),
+                                          bytes_to_read_,
+                                          step_size_));
+    }
+
     HMODULE chrome_exe = LoadLibrary(chrome_exe_.value().c_str());
-    ASSERT_TRUE(chrome_exe != NULL);
+    EXPECT_TRUE(chrome_exe != NULL);
 
     HMODULE chrome_dll = LoadLibrary(chrome_dll_.value().c_str());
-    ASSERT_TRUE(chrome_dll != NULL);
-
-    HMODULE chrome_tab_dll = LoadLibrary(chrome_frame_dll_.value().c_str());
-    ASSERT_TRUE(chrome_tab_dll != NULL);
+    EXPECT_TRUE(chrome_dll != NULL);
 
     *end_time = TimeTicks::Now();
 
     FreeLibrary(chrome_exe);
     FreeLibrary(chrome_dll);
-    FreeLibrary(chrome_tab_dll);
   }
+
+  bool pre_read_;
+  size_t bytes_to_read_;
+  size_t step_size_;
 };
 
 // This class provides functionality to run the startup performance test for
 // the ChromeFrame ActiveX against a reference build. At this point we only run
 // this test in warm mode.
 class ChromeFrameStartupTestActiveXReference
     : public ChromeFrameStartupTestActiveX {
  public:
   // override the browser directory to use the reference build instead.
   virtual void SetUp() {
@@ skipping 485 matching lines @@
   RunStartupTest("warm", "t", "about:blank", false /* cold */, 0, NULL,
                  true /* important */, false);
 }
 
 TEST_F(ChromeFrameBinariesLoadTest, PerfWarm) {
   RunStartupTest("binary_load_warm", "t", "", false /* cold */, 0, NULL,
                  true /* important */, false);
 }
 
 TEST_F(ChromeFrameStartupTestActiveX, PerfCold) {
+  SetConfigInt(L"PreRead", 0);
   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);
+  DeleteConfigValue(L"PreRead");
+}
+
+TEST_F(ChromeFrameStartupTestActiveX, PerfColdPreRead) {
+  SetConfigInt(L"PreRead", 1);
+  FilePath binaries_to_evict[] = { gears_dll_, avcodec52_dll_,
+      avformat52_dll_, avutil50_dll_, chrome_exe_, chrome_dll_,
+      chrome_frame_dll_};
+  RunStartupTest("cold_preread", "t", "about:blank", true /* cold */,
+                 arraysize(binaries_to_evict), binaries_to_evict,
+                 false /* not important */, false);
+  DeleteConfigValue(L"PreRead");
 }
 
 TEST_F(ChromeFrameBinariesLoadTest, PerfCold) {
-  FilePath binaries_to_evict[] = {chrome_exe_, chrome_dll_, chrome_frame_dll_};
+  FilePath binaries_to_evict[] = {chrome_exe_, chrome_dll_};
   RunStartupTest("binary_load_cold", "t", "", true /* cold */,
                  arraysize(binaries_to_evict), binaries_to_evict,
                  false /* not important */, false);
 }
 
+TEST_F(ChromeFrameBinariesLoadTest, PerfColdPreRead) {
+  FilePath binaries_to_evict[] = {chrome_exe_, chrome_dll_};
+  pre_read_ = true;
+  RunStartupTest("binary_load_cold_preread", "t", "", true /* cold */,
+                 arraysize(binaries_to_evict), binaries_to_evict,
+                 false /* not important */, false);
+}
+
 TEST_F(ChromeFrameStartupTestActiveXReference, PerfWarm) {
   RunStartupTest("warm", "t_ref", "about:blank", false /* cold */, 0, NULL,
                  true /* important */, false);
 }
 
 TEST_F(ChromeFrameStartupTestActiveX, PerfChromeFrameInitializationWarm) {
   RunStartupTest("ChromeFrame_init_warm", "t", "", false /* cold */, 0,
                  NULL, true /* important */, false);
 }
 
@@ skipping 118 matching lines @@
   ASSERT_TRUE(silverlight_key.ReadValue(L"", &plugin_path));
   ASSERT_FALSE(plugin_path.empty());
 
   FilePath silverlight_path = FilePath::FromWStringHack(plugin_path);
   FilePath binaries_to_evict[] = {silverlight_path};
 
   RunStartupTest("creation_cold", "t_silverlight", "", true /* cold */,
                  arraysize(binaries_to_evict), binaries_to_evict,
                  false /* important */, true);
 }