Add pinned property to tabs.update and create.

Fire tabs.onUpdated event if a tab's pinned state changes.

BUG=24781
TEST=ExtensionApiTest

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

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

1 files changed, 9 insertions, 3 deletions

diff --git a/chrome/browser/tabs/tab_strip_model_unittest.cc b/chrome/browser/tabs/tab_strip_model_unittest.cc
index a08a5bc..81847e9 100644
--- a/chrome/browser/tabs/tab_strip_model_unittest.cc
+++ b/chrome/browser/tabs/tab_strip_model_unittest.cc
@@ -1685,12 +1685,15 @@ TEST_F(TabStripModelTest, Pinning) {
   {
     tabstrip.SetTabPinned(2, true);
 
-    // The pinning should have resulted in a move.
-    ASSERT_EQ(1, observer.GetStateCount());
+    // The pinning should have resulted in a move and a pinned notification.
+    ASSERT_EQ(2, observer.GetStateCount());
     State state(contents3, 0, MockTabStripModelObserver::MOVE);
     state.src_index = 2;
     EXPECT_TRUE(observer.StateEquals(0, state));
 
+    state = State(contents3, 0, MockTabStripModelObserver::PINNED);
+    EXPECT_TRUE(observer.StateEquals(1, state));
+
     // And verify the state.
     EXPECT_EQ("3p 1 2", GetPinnedState(tabstrip));
 
@@ -1729,11 +1732,14 @@ TEST_F(TabStripModelTest, Pinning) {
   {
     tabstrip.SetTabPinned(0, false);
 
-    ASSERT_EQ(1, observer.GetStateCount());
+    ASSERT_EQ(2, observer.GetStateCount());
     State state(contents3, 1, MockTabStripModelObserver::MOVE);
     state.src_index = 0;
     EXPECT_TRUE(observer.StateEquals(0, state));
 
+    state = State(contents3, 1, MockTabStripModelObserver::PINNED);
+    EXPECT_TRUE(observer.StateEquals(1, state));
+
     // And verify the state.
     EXPECT_EQ("1p 3 2", GetPinnedState(tabstrip));