[WebsiteSettings] Fix bug in permission bubble manager to alter showing state when bubble is closed.

chrome/browser/ui/website_settings/permission_bubble_manager_unittest.cc

Index: chrome/browser/ui/website_settings/permission_bubble_manager_unittest.cc
diff --git a/chrome/browser/ui/website_settings/permission_bubble_manager_unittest.cc b/chrome/browser/ui/website_settings/permission_bubble_manager_unittest.cc
index 0e12b56b83dfc05d8ed4fdc20534c2c19e4f9934..36336f02efe22f23731b7c615cdf6a20e72c48e7 100644
--- a/chrome/browser/ui/website_settings/permission_bubble_manager_unittest.cc
+++ b/chrome/browser/ui/website_settings/permission_bubble_manager_unittest.cc
@@ -188,3 +188,20 @@ TEST_F(PermissionBubbleManagerTest, TwoRequestsTabSwitch) {
   EXPECT_FALSE(request2_.granted_);
 }
 
+TEST_F(PermissionBubbleManagerTest, SequentialRequests) {
+  manager_->SetView(&view_);
+  manager_->AddRequest(&request1_);
+  EXPECT_TRUE(view_.shown_);
+
+  manager_->AddRequest(&request2_);

[leng, 2014/02/13 01:12:45]

It's not clear to me why you're adding request2 twice, because you're not
checking that it wasn't shown, or granted.  Perhaps add
EXPECT_FALSE(request2_.granted_); after EXPECT_TRUE(request1_.granted_)?

[Greg Billock, 2014/02/13 01:39:22]

Thanks. This was just exercising a log line to make sure the bug was there and
that this fixed it.

+  Accept();
+  EXPECT_TRUE(request1_.granted_);
+
+  EXPECT_FALSE(view_.shown_);
+
+  manager_->AddRequest(&request2_);
+  EXPECT_TRUE(view_.shown_);
+  Accept();
+  EXPECT_FALSE(view_.shown_);
+  EXPECT_TRUE(request2_.granted_);
+}