[ios] WebStateList owns all WebState it manages.

ios/chrome/browser/tabs/tab_model_unittest.mm

 // Copyright 2012 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.
 
 #import <objc/runtime.h>
 
 #include "base/files/file_path.h"
 #include "base/mac/scoped_nsautorelease_pool.h"
 #include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
@@ skipping 21 matching lines @@
 #include "ios/web/public/test/test_web_thread_bundle.h"
 #include "ios/web/public/web_thread.h"
 #import "ios/web/web_state/ui/crw_web_controller.h"
 #import "ios/web/web_state/web_state_impl.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "testing/gtest_mac.h"
 #include "testing/platform_test.h"
 #import "third_party/ocmock/OCMock/OCMock.h"
 #include "third_party/ocmock/gtest_support.h"
 
-using web::WebStateImpl;
-
-
 @interface TabModel (VisibleForTesting)
 - (SessionWindowIOS*)windowForSavingSession;
 @end
 
 // Defines a TabModelObserver for use in unittests.  This class can be used to
 // test if an observer method was called or not.
 @interface TabModelObserverPong : NSObject<TabModelObserver> {
   // TODO(crbug.com/661989): Add tests for the other observer methods.
   BOOL tabMovedWasCalled_;
 }
@@ skipping 1002 matching lines @@
   EXPECT_NSEQ(tab1, [tab_model_ tabAtIndex:0]);
   EXPECT_NSEQ(tab2, [tab_model_ tabAtIndex:1]);
   EXPECT_NSEQ(tab0, [tab_model_ tabAtIndex:2]);
   EXPECT_FALSE([tab_model_observer tabMovedWasCalled]);
 
   // TabModel asserts that there are no observer when it is deallocated,
   // so remove the observer before the end of the method.
   [tab_model_ removeObserver:tab_model_observer.get()];
 }
 
-TEST_F(TabModelTest, SetParentModel) {

[rohitrao (ping after 24h), 2017/04/03 17:49:35]

Should we rewrite/fix this test instead of deleting it?

[sdefresne, 2017/04/04 13:47:56]

Done. Also added a test that check a Tab is created when a WebState is added to
TabModel's WebStateList.

-  // Create a tab without a parent model and make sure it doesn't crash.  Then
-  // set its parent TabModel and make sure that works as well.
-  base::scoped_nsobject<Tab> tab([[Tab alloc]
-      initWithBrowserState:chrome_browser_state_.get()
-                    opener:nil
-               openedByDOM:NO
-                     model:nil]);
-  EXPECT_TRUE([tab parentTabModel] == nil);
-  [tab_model_ insertTab:tab atIndex:0];
-  [tab setParentTabModel:tab_model_.get()];
-  EXPECT_FALSE([tab parentTabModel] == nil);
-  [tab_model_ closeTabAtIndex:0];
-}
-
 TEST_F(TabModelTest, PersistSelectionChange) {
   NSString* stashPath =
       base::SysUTF8ToNSString(chrome_browser_state_->GetStatePath().value());
 
   [tab_model_ insertTabWithURL:GURL(kURL1)
                       referrer:web::Referrer()
                     transition:ui::PAGE_TRANSITION_TYPED
                         opener:nil
                    openedByDOM:NO
                        atIndex:[tab_model_ count]
@@ skipping 53 matching lines @@
             [tab_model_ openerOfTab:[tab_model_ tabAtIndex:2]]);
   EXPECT_EQ([tab_model_ tabAtIndex:2],
             [tab_model_ openerOfTab:[tab_model_ tabAtIndex:0]]);
 
   // Clean up.
   EXPECT_TRUE([[NSFileManager defaultManager] removeItemAtPath:stashPath
                                                          error:nullptr]);
 }
 
 }  // anonymous namespace