Avoid double-setting of the tool band ID in the CEEE executors manager....

ceee/ie/plugin/toolband/tool_band_unittest.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.
 //
 // IE toolband unit tests.
 #include "ceee/ie/plugin/toolband/tool_band.h"
 
 #include <exdisp.h>
 #include <shlguid.h>
 
@@ skipping 493 matching lines @@
 
 TEST_F(ToolBandTest, SendSessionIdToBho) {
   MockCeeeBho* mock_bho;
   CComPtr<IObjectWithSite> mock_bho_with_site;
   ASSERT_HRESULT_SUCCEEDED(
       MockCeeeBho::CreateInitialized(&mock_bho, &mock_bho_with_site));
   EXPECT_CALL(*tool_band_, GetSessionId(_)).WillOnce(DoAll(
       SetArgumentPointee<0>(5), Return(S_OK)));
   EXPECT_CALL(*mock_bho, SetToolBandSessionId(5)).WillOnce(Return(S_OK));
   EXPECT_EQ(S_OK, tool_band_->CallSendSessionIdToBho(mock_bho_with_site));
+  // Second call should not do anything.
+  EXPECT_EQ(S_FALSE, tool_band_->CallSendSessionIdToBho(mock_bho_with_site));
+}
 
-  // Test error handling.
+TEST_F(ToolBandTest, SendSessionIdToBhoErrorHandling) {
+  MockCeeeBho* mock_bho;
+  CComPtr<IObjectWithSite> mock_bho_with_site;
+  ASSERT_HRESULT_SUCCEEDED(
+      MockCeeeBho::CreateInitialized(&mock_bho, &mock_bho_with_site));
   EXPECT_CALL(*tool_band_, GetSessionId(_)).WillOnce(Return(E_FAIL));
   EXPECT_EQ(E_FAIL, tool_band_->CallSendSessionIdToBho(mock_bho_with_site));
 
   EXPECT_CALL(*tool_band_, GetSessionId(_)).WillOnce(DoAll(
       SetArgumentPointee<0>(6), Return(S_OK)));
   EXPECT_CALL(*mock_bho, SetToolBandSessionId(6)).WillOnce(Return(E_FAIL));
   EXPECT_EQ(E_FAIL, tool_band_->CallSendSessionIdToBho(mock_bho_with_site));
 }
 
 }  // namespace