Fix a crash on 64-bit systems when an external message hook is installed. The...

chrome/views/combo_box.cc

 // Copyright (c) 2006-2008 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/views/combo_box.h"
 
 #include "base/gfx/native_theme.h"
 #include "base/gfx/rect.h"
 #include "chrome/common/gfx/chrome_canvas.h"
 #include "chrome/common/gfx/chrome_font.h"
@@ skipping 21 matching lines @@
 }
 
 gfx::Size ComboBox::GetPreferredSize() {
   HWND hwnd = GetNativeControlHWND();
   if (!hwnd)
     return gfx::Size();
 
   COMBOBOXINFO cbi;
   memset(reinterpret_cast<unsigned char*>(&cbi), 0, sizeof(cbi));
   cbi.cbSize = sizeof(cbi);
-  ::SendMessage(hwnd, CB_GETCOMBOBOXINFO, 0, reinterpret_cast<LPARAM>(&cbi));
+  // Note: Don't use CB_GETCOMBOBOXINFO since that crashes on WOW64 systems
+  // when you have a global message hook installed.
+  GetComboBoxInfo(hwnd, &cbi);
   gfx::Rect rect_item(cbi.rcItem);
   gfx::Rect rect_button(cbi.rcButton);
   gfx::Size border = gfx::NativeTheme::instance()->GetThemeBorderSize(
       gfx::NativeTheme::MENULIST);
 
   // The padding value of '3' is the xy offset from the corner of the control
   // to the corner of rcItem.  It does not seem to be queryable from the theme.
   // It is consistent on all versions of Windows from 2K to Vista, and is
   // invariant with respect to the combobox border size.  We could conceivably
   // get this number from rect_item.x, but it seems fragile to depend on position
@@ skipping 103 matching lines @@
   // CB_SELECTSTRING does).
   ::SendMessage(hwnd, CB_SETCURSEL, selected_item_, 0);
 }
 
 int ComboBox::GetSelectedItem() {
   return selected_item_;
 }
 
 }  // namespace views