Revert r42156, r42157, r42160. Allow dynamic switching in and out of sidetabs...

chrome/browser/views/tabs/tab.cc

 // 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/views/tabs/tab.h"
 
 #include "app/gfx/canvas.h"
 #include "app/gfx/font.h"
 #include "app/l10n_util.h"
 #include "app/menus/simple_menu_model.h"
@@ skipping 33 matching lines @@
     // nested message loop that may not return until after we are deleted.
     Tab::TabDelegate* delegate = tab_->delegate();
     menu_->RunMenuAt(point, views::Menu2::ALIGN_TOPLEFT);
     // We could be gone now. Assume |this| is junk!
     if (delegate)
       delegate->StopAllHighlighting();
   }
 
   // Overridden from menus::SimpleMenuModel::Delegate:
   virtual bool IsCommandIdChecked(int command_id) const {
-    return tab_ && tab_->delegate()->IsCommandCheckedForTab(
-        static_cast<TabStripModel::ContextMenuCommand>(command_id),
-        tab_);
+    if (!tab_ || command_id != TabStripModel::CommandTogglePinned)
+      return false;
+    return tab_->delegate()->IsTabPinned(tab_);
   }
   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) {
     return tab_->GetWidget()->GetAccelerator(command_id, accelerator);
@@ skipping 173 matching lines @@
 
   // Right end cap.
   path->lineTo(w - kTabCapWidth + kTabTopCurveWidth, kTabTopCurveWidth);
   path->lineTo(w - kTabBottomCurveWidth, h - kTabBottomCurveWidth);
   path->lineTo(w, h);
 
   // Close out the path.
   path->lineTo(0, h);
   path->close();
 }