Updated ownership of NavigationItems within CRWSessionController.

ios/web/navigation/crw_session_entry_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 "ios/web/navigation/crw_session_entry.h"
 
 #import <Foundation/Foundation.h>
 #include <stdint.h>
 
 #include <utility>
@@ skipping 15 matching lines @@
  public:
   static void expectEqualSessionEntries(CRWSessionEntry* entry1,
                                         CRWSessionEntry* entry2,
                                         ui::PageTransition transition);
 
  protected:
   void SetUp() override {
     GURL url("http://init.test");
     ui::PageTransition transition =
         ui::PAGE_TRANSITION_AUTO_BOOKMARK;
-    std::unique_ptr<web::NavigationItemImpl> item(
-        new web::NavigationItemImpl());
-    item->SetOriginalRequestURL(url);
-    item->SetURL(url);
-    item->SetTransitionType(transition);
-    item->SetTimestamp(base::Time::Now());
-    item->SetPostData([@"Test data" dataUsingEncoding:NSUTF8StringEncoding]);
+    item_.reset(new web::NavigationItemImpl());
+    item_->SetOriginalRequestURL(url);
+    item_->SetURL(url);
+    item_->SetTransitionType(transition);
+    item_->SetTimestamp(base::Time::Now());
+    item_->SetPostData([@"Test data" dataUsingEncoding:NSUTF8StringEncoding]);
     session_entry_.reset(
-        [[CRWSessionEntry alloc] initWithNavigationItem:std::move(item)]);
+        [[CRWSessionEntry alloc] initWithNavigationItem:item_.get()]);
   }
 
  protected:
+  std::unique_ptr<web::NavigationItemImpl> item_;
   base::scoped_nsobject<CRWSessionEntry> session_entry_;
 };
 
 void CRWSessionEntryTest::expectEqualSessionEntries(
     CRWSessionEntry* entry1,
     CRWSessionEntry* entry2,
     ui::PageTransition transition) {
   web::NavigationItemImpl* navItem1 = entry1.navigationItemImpl;
   web::NavigationItemImpl* navItem2 = entry2.navigationItemImpl;
   // url is not compared because it could differ after copy or archive.
@@ skipping 30 matching lines @@
 TEST_F(CRWSessionEntryTest, NonEmptyVirtualUrl) {
   web::NavigationItem* item = [session_entry_ navigationItem];
   item->SetVirtualURL(GURL("http://user.friendly"));
   EXPECT_EQ(GURL("http://user.friendly/"), item->GetVirtualURL());
   EXPECT_EQ(GURL("http://init.test/"), item->GetURL());
 }
 
 TEST_F(CRWSessionEntryTest, EmptyDescription) {
   EXPECT_GT([[session_entry_ description] length], 0U);
 }