Changes the tab menu to use pin and unpin instead of a check. The mac

chrome/browser/gtk/tabs/tab_gtk.cc

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// 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/browser/gtk/tabs/tab_gtk.h"
 
 #include <gdk/gdkkeysyms.h>
 
 #include "app/gtk_dnd_util.h"
 #include "app/l10n_util.h"
 #include "app/menus/accelerator_gtk.h"
@@ skipping 20 matching lines @@
 
   return font->GetStringWidth(title);
 }
 
 }  // namespace
 
 class TabGtk::ContextMenuController : public menus::SimpleMenuModel::Delegate {
  public:
   explicit ContextMenuController(TabGtk* tab)
       : tab_(tab),
-        model_(this) {
+        model_(this, tab->delegate()->IsTabPinned(tab)) {
     menu_.reset(new MenuGtk(NULL, &model_));
   }
 
   virtual ~ContextMenuController() {}
 
   void RunMenu() {
     menu_->PopupAsContext(gtk_get_current_event_time());
   }
 
   void Cancel() {
     tab_ = NULL;
     menu_->Cancel();
   }
 
  private:
   // Overridden from menus::SimpleMenuModel::Delegate:
   virtual bool IsCommandIdChecked(int command_id) const {
-    if (!tab_ || command_id != TabStripModel::CommandTogglePinned)
-      return false;
-    return tab_->delegate()->IsTabPinned(tab_);
+    return false;
   }
   virtual bool IsCommandIdEnabled(int command_id) const {
     return tab_ && tab_->delegate()->IsCommandEnabledForTab(
         static_cast<TabStripModel::ContextMenuCommand>(command_id),
         tab_);
   }
   virtual bool GetAcceleratorForCommandId(
       int command_id,
       menus::Accelerator* accelerator) {
     int browser_command = 0;
@@ skipping 268 matching lines @@
 
 void TabGtk::SetBounds(const gfx::Rect& bounds) {
   TabRendererGtk::SetBounds(bounds);
   UpdateTooltipState();
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // TabGtk, private:
 
 void TabGtk::ShowContextMenu() {
-  if (!menu_controller_.get())
-    menu_controller_.reset(new ContextMenuController(this));
+  menu_controller_.reset(new ContextMenuController(this));
 
   menu_controller_->RunMenu();
 }
 
 void TabGtk::ContextMenuClosed() {
   delegate()->StopAllHighlighting();
   menu_controller_.reset();
 }
 
 void TabGtk::UpdateTooltipState() {
@@ skipping 50 matching lines @@
     gdk_event_free(last_mouse_down_);
     last_mouse_down_ = NULL;
   }
 
   // Notify the drag helper that we're done with any potential drag operations.
   // Clean up the drag helper, which is re-created on the next mouse press.
   delegate_->EndDrag(canceled);
 
   observer_.reset();
 }