Makes MenuButton compile and fixes bug in GtkWidget I happened upon....

views/controls/button/menu_button.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 "views/controls/button/menu_button.h"
 
 #include "app/drag_drop_types.h"
 #include "app/gfx/canvas.h"
 #include "app/l10n_util.h"
 #include "app/resource_bundle.h"
-#include "app/win_util.h"
 #include "grit/app_strings.h"
 #include "grit/app_resources.h"
 #include "views/controls/button/button.h"
 #include "views/controls/menu/view_menu_delegate.h"
 #include "views/event.h"
 #include "views/widget/root_view.h"
 #include "views/widget/widget.h"
 
+#if defined(OS_WIN)
+#include "app/win_util.h"
+#endif
+
 using base::Time;
 using base::TimeDelta;
 
 namespace views {
 
 // The amount of time, in milliseconds, we wait before allowing another mouse
 // pressed event to show the menu.
 static const int64 kMinimumTimeBetweenButtonClicks = 100;
 
 // The down arrow used to differentiate the menu button from normal
@@ skipping 72 matching lines @@
 ////////////////////////////////////////////////////////////////////////////////
 
 int MenuButton::GetMaximumScreenXCoordinate() {
   Widget* widget = GetWidget();
 
   if (!widget) {
     NOTREACHED();
     return 0;
   }
 
+#if defined(OS_WIN)
   HWND hwnd = widget->GetNativeView();
   RECT t;
   ::GetWindowRect(hwnd, &t);
 
   gfx::Rect r(t);
   gfx::Rect monitor_rect = win_util::GetMonitorBoundsForRect(r);
   return monitor_rect.x() + monitor_rect.width() - 1;
+#else
+  NOTIMPLEMENTED();
+  return 1000000;
+#endif
 }
 
 bool MenuButton::Activate() {
   SetState(BS_PUSHED);
   // We need to synchronously paint here because subsequently we enter a
   // menu modal loop which will stop this window from updating and
   // receiving the paint message that should be spawned by SetState until
   // after the menu closes.
   PaintNow();
   if (menu_delegate_) {
@@ skipping 69 matching lines @@
       state() != BS_DISABLED && !canceled && !InDrag() &&
       e.IsOnlyLeftMouseButton() && HitTest(e.location())) {
     Activate();
   } else {
     TextButton::OnMouseReleased(e, canceled);
   }
 }
 
 // When the space bar or the enter key is pressed we need to show the menu.
 bool MenuButton::OnKeyReleased(const KeyEvent& e) {
+#if defined(OS_WIN)
   if ((e.GetCharacter() == VK_SPACE) || (e.GetCharacter() == VK_RETURN)) {
     return Activate();
   }
+#else
+  NOTIMPLEMENTED();
+#endif
   return true;
 }
 
 // The reason we override View::OnMouseExited is because we get this event when
 // we display the menu. If we don't override this method then
 // BaseButton::OnMouseExited will get the event and will set the button's state
 // to BS_NORMAL instead of keeping the state BM_PUSHED. This, in turn, will
 // cause the button to appear depressed while the menu is displayed.
 void MenuButton::OnMouseExited(const MouseEvent& event) {
   if ((state_ != BS_DISABLED) && (!menu_visible_) && (!InDrag())) {
@@ skipping 22 matching lines @@
 }
 
 bool MenuButton::GetAccessibleState(AccessibilityTypes::State* state) {
   DCHECK(state);
 
   *state = AccessibilityTypes::STATE_HASPOPUP;
   return true;
 }
 
 }  // namespace views