diff options
| author | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-15 06:17:42 +0000 |
|---|---|---|
| committer | jar@chromium.org <jar@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-15 06:17:42 +0000 |
| commit | 8d34c475c53b27511a912880aebea627ad651625 (patch) | |
| tree | a581b4528f323ea84b5c8b5b6a92bc767096efa1 /chrome/common | |
| parent | 15294d803ac17bb5355076c90ddbecca983985a8 (diff) | |
| download | chromium_src-8d34c475c53b27511a912880aebea627ad651625.zip chromium_src-8d34c475c53b27511a912880aebea627ad651625.tar.gz chromium_src-8d34c475c53b27511a912880aebea627ad651625.tar.bz2 | |
Temporarilly disable SetURL to evaluate perf impact
This is a temporary checkin, meant to gather data from the bots that run
performance tests. I'll revert it asap.
Profiling suggests we call SetURL VERY often, so that we can get crash
info. The function turns out to cause a memory allocation, and also a
string conversion, and hence *might* have a significant perf impact.
TBR=mbelshe
Review URL: http://codereview.chromium.org/273051
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@29096 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/common')
| -rw-r--r-- | chrome/common/child_process_logging_win.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/chrome/common/child_process_logging_win.cc b/chrome/common/child_process_logging_win.cc index 2221713..ff64fe6 100644 --- a/chrome/common/child_process_logging_win.cc +++ b/chrome/common/child_process_logging_win.cc @@ -15,6 +15,7 @@ namespace child_process_logging { typedef void (__cdecl *MainSetActiveURL)(const wchar_t*); void SetActiveURL(const GURL& url) { + return; // TODO(jar): Revert this. This is a test to see the perf impact of not calling SetActiveURL. HMODULE exe_module = GetModuleHandle(chrome::kBrowserProcessExecutableName); if (!exe_module) return; |