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

Side by Side Diff: ios/chrome/browser/tabs/tab_model_list.mm

Issue 2621083003: Implement 1:N mapping from ios::ChromeBrowserState to TabModel. (Closed)
Patch Set: Use NSMutableSet<TabModel*>* to hold the TabModels. Created 3 years, 11 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #import "ios/chrome/browser/tabs/tab_model_list.h"
6
7 #include "base/logging.h"
8 #include "base/macros.h"
9 #include "base/supports_user_data.h"
10 #include "ios/chrome/browser/browser_state/chrome_browser_state.h"
11 #import "ios/chrome/browser/tabs/tab_model.h"
12
13 #if !defined(__has_feature) || !__has_feature(objc_arc)
14 #error "This file requires ARC support."
15 #endif
16
17 namespace {
18 // Wrapper around a NSMutableArray<TabModel> allowing to bind it to a
19 // base::SupportsUserData. Any base::SupportsUserData that owns this
20 // wrapper will owns the reference to the TabModels.
21 class TabModelList : public base::SupportsUserData::Data {
22 public:
23 TabModelList();
24 ~TabModelList() override;
25
26 static TabModelList* GetForBrowserState(
27 ios::ChromeBrowserState* browser_state,
28 bool create);
29
30 NSMutableSet<TabModel*>* tab_models() const { return tab_models_; }
31
32 private:
33 NSMutableSet<TabModel*>* tab_models_;
34
35 DISALLOW_COPY_AND_ASSIGN(TabModelList);
36 };
37
38 const char kTabModelListKey = 0;
39
40 TabModelList::TabModelList() : tab_models_([[NSMutableSet alloc] init]) {}
41
42 TabModelList::~TabModelList() {
43 // TabModelList is destroyed during base::SupportsUserData destruction. At
44 // that point, all TabModels must have been unregistered already.
45 DCHECK_EQ([tab_models_ count], 0u);
46 }
47
48 // static
49 TabModelList* TabModelList::GetForBrowserState(
50 ios::ChromeBrowserState* browser_state,
51 bool create) {
52 TabModelList* tab_model_list =
53 static_cast<TabModelList*>(browser_state->GetUserData(&kTabModelListKey));
54 if (!tab_model_list && create) {
55 // The ownership of TabModelList is transfered to base::SupportsUserData
56 // via the SetUserData call (should take a std::unique_ptr<>).
57 tab_model_list = new TabModelList;
58 browser_state->SetUserData(&kTabModelListKey, tab_model_list);
59 }
60 return tab_model_list;
61 }
62 } // namespace
63
64 void RegisterTabModelWithChromeBrowserState(
65 ios::ChromeBrowserState* browser_state,
66 TabModel* tab_model) {
67 NSMutableSet<TabModel*>* tab_models =
68 TabModelList::GetForBrowserState(browser_state, true)->tab_models();
69 DCHECK(![tab_models containsObject:tab_model]);
70 [tab_models addObject:tab_model];
71 }
72
73 void UnregisterTabModelFromChromeBrowserState(
74 ios::ChromeBrowserState* browser_state,
75 TabModel* tab_model) {
76 NSMutableSet<TabModel*>* tab_models =
77 TabModelList::GetForBrowserState(browser_state, false)->tab_models();
78 DCHECK([tab_models containsObject:tab_model]);
79 [tab_models removeObject:tab_model];
80 }
81
82 NSArray<TabModel*>* GetTabModelsForChromeBrowserState(
83 ios::ChromeBrowserState* browser_state) {
84 TabModelList* tab_model_list =
85 TabModelList::GetForBrowserState(browser_state, false);
86 return tab_model_list ? [tab_model_list->tab_models() allObjects] : nil;
87 }
OLDNEW
« no previous file with comments | « ios/chrome/browser/tabs/tab_model_list.h ('k') | ios/chrome/browser/tabs/tab_model_list_unittest.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698