Revert "Don't call SetPriorityClass if CBSText.dll is loaded in the browser

process. The CBSText.dll loads as a global GetMessage hook into the browser
process and intercepts/unintercepts the kernel32 API SetPriorityClass in a
background thread. If the browser calls this API just"

This file is compiled on all platforms and this is windows specific code.

This reverts commit r10648.

Review URL: http://codereview.chromium.org/29008


git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10653 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 11 deletions

diff --git a/chrome/browser/renderer_host/browser_render_process_host.cc b/chrome/browser/renderer_host/browser_render_process_host.cc
index d22dbac..88b50c9 100644
--- a/chrome/browser/renderer_host/browser_render_process_host.cc
+++ b/chrome/browser/renderer_host/browser_render_process_host.cc
@@ -833,17 +833,9 @@ void BrowserRenderProcessHost::OnUpdatedCacheStats(
 void BrowserRenderProcessHost::SetBackgrounded(bool backgrounded) {
   // If the process_ is NULL, the process hasn't been created yet.
   if (process_.handle()) {
-    // The cbstext.dll loads as a global GetMessage hook in the browser process
-    // and intercepts/unintercepts the kernel32 API SetPriorityClass in a
-    // background thread. If the UI thread invokes this API just when it is
-    // intercepted the stack is messed up on return from the interceptor
-    // which causes random crashes in the browser process. Our hack for now
-    // is to not invoke the SetPriorityClass API if the dll is loaded.
-    if (!GetModuleHandle(L"cbstext.dll")) {
-      bool rv = process_.SetProcessBackgrounded(backgrounded);
-      if (!rv) {
-        return;
-      }
+    bool rv = process_.SetProcessBackgrounded(backgrounded);
+    if (!rv) {
+      return;
     }
 
     // Now tune the memory footprint of the renderer.