Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(416)

Side by Side Diff: ios/shared/chrome/browser/ui/browser_list/browser.mm

Issue 2775623002: [ios] WebStateList owns all WebState it manages. (Closed)
Patch Set: Fix ios_chrome_unittests. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "ios/shared/chrome/browser/ui/browser_list/browser.h" 5 #import "ios/shared/chrome/browser/ui/browser_list/browser.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/memory/ptr_util.h" 8 #include "base/memory/ptr_util.h"
9 #import "ios/shared/chrome/browser/tabs/web_state_list.h" 9 #import "ios/shared/chrome/browser/tabs/web_state_list.h"
10 #import "ios/shared/chrome/browser/ui/browser_list/browser_web_state_list_delega te.h" 10 #import "ios/shared/chrome/browser/ui/browser_list/browser_web_state_list_delega te.h"
11 11
12 #if !defined(__has_feature) || !__has_feature(objc_arc) 12 #if !defined(__has_feature) || !__has_feature(objc_arc)
13 #error "This file requires ARC support." 13 #error "This file requires ARC support."
14 #endif 14 #endif
15 15
16 Browser::Browser(ios::ChromeBrowserState* browser_state) 16 Browser::Browser(ios::ChromeBrowserState* browser_state)
17 : browser_state_(browser_state) { 17 : browser_state_(browser_state) {
18 DCHECK(browser_state_); 18 DCHECK(browser_state_);
19 web_state_list_delegate_ = 19 web_state_list_delegate_ =
20 base::MakeUnique<BrowserWebStateListDelegate>(this); 20 base::MakeUnique<BrowserWebStateListDelegate>(this);
21 web_state_list_ = base::MakeUnique<WebStateList>( 21 web_state_list_ =
22 web_state_list_delegate_.get(), WebStateList::WebStateOwned); 22 base::MakeUnique<WebStateList>(web_state_list_delegate_.get());
23 } 23 }
24 24
25 Browser::~Browser() = default; 25 Browser::~Browser() = default;
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698