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

Side by Side Diff: components/signin/ios/browser/account_consistency_service.h

Issue 2928383002: [ObjC ARC] Converts components/signin/ios/browser:browser to ARC. (Closed)
Patch Set: Made other delegate explicitly weak. Created 3 years, 6 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #ifndef COMPONENTS_SIGNIN_IOS_BROWSER_ACCOUNT_CONSISTENCY_SERVICE_H_ 5 #ifndef COMPONENTS_SIGNIN_IOS_BROWSER_ACCOUNT_CONSISTENCY_SERVICE_H_
6 #define COMPONENTS_SIGNIN_IOS_BROWSER_ACCOUNT_CONSISTENCY_SERVICE_H_ 6 #define COMPONENTS_SIGNIN_IOS_BROWSER_ACCOUNT_CONSISTENCY_SERVICE_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <map> 9 #include <map>
10 #include <memory> 10 #include <memory>
11 #include <set> 11 #include <set>
12 #include <string> 12 #include <string>
13 13
14 #include "base/mac/scoped_nsobject.h" 14 #include "base/mac/scoped_nsobject.h"
msarda 2017/06/13 13:15:30 Should this include be removed?
marq (ping after 24h) 2017/06/15 09:27:33 Done.
15 #include "base/macros.h" 15 #include "base/macros.h"
16 #include "base/memory/ref_counted.h" 16 #include "base/memory/ref_counted.h"
17 #include "base/time/time.h" 17 #include "base/time/time.h"
18 #include "components/content_settings/core/browser/cookie_settings.h" 18 #include "components/content_settings/core/browser/cookie_settings.h"
19 #include "components/keyed_service/core/keyed_service.h" 19 #include "components/keyed_service/core/keyed_service.h"
20 #include "components/signin/core/browser/gaia_cookie_manager_service.h" 20 #include "components/signin/core/browser/gaia_cookie_manager_service.h"
21 #include "components/signin/core/browser/signin_manager.h" 21 #include "components/signin/core/browser/signin_manager.h"
22 #import "components/signin/ios/browser/manage_accounts_delegate.h" 22 #import "components/signin/ios/browser/manage_accounts_delegate.h"
23 #include "ios/web/public/active_state_manager.h" 23 #include "ios/web/public/active_state_manager.h"
24 24
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
171 171
172 // Whether a CHROME_CONNECTED cookie request is currently being applied. 172 // Whether a CHROME_CONNECTED cookie request is currently being applied.
173 bool applying_cookie_requests_; 173 bool applying_cookie_requests_;
174 // The queue of CHROME_CONNECTED cookie requests to be applied. 174 // The queue of CHROME_CONNECTED cookie requests to be applied.
175 std::deque<CookieRequest> cookie_requests_; 175 std::deque<CookieRequest> cookie_requests_;
176 // The map between domains where a CHROME_CONNECTED cookie is present and 176 // The map between domains where a CHROME_CONNECTED cookie is present and
177 // the time when the cookie was last updated. 177 // the time when the cookie was last updated.
178 std::map<std::string, base::Time> last_cookie_update_map_; 178 std::map<std::string, base::Time> last_cookie_update_map_;
179 179
180 // Web view used to apply the CHROME_CONNECTED cookie requests. 180 // Web view used to apply the CHROME_CONNECTED cookie requests.
181 base::scoped_nsobject<WKWebView> web_view_; 181 base::scoped_nsobject<WKWebView> web_view_;
msarda 2017/06/13 13:15:30 Should this change and be now a WKWebView*?
marq (ping after 24h) 2017/06/15 09:27:33 We can only do that in header files if the header
182 // Navigation delegate of |web_view_| that informs the service when a cookie 182 // Navigation delegate of |web_view_| that informs the service when a cookie
183 // request has been applied. 183 // request has been applied.
184 base::scoped_nsobject<AccountConsistencyNavigationDelegate> 184 AccountConsistencyNavigationDelegate* navigation_delegate_;
185 navigation_delegate_;
186 185
187 // Handlers reacting on GAIA responses with the X-Chrome-Manage-Accounts 186 // Handlers reacting on GAIA responses with the X-Chrome-Manage-Accounts
188 // header set. 187 // header set.
189 std::map<web::WebState*, std::unique_ptr<web::WebStatePolicyDecider>> 188 std::map<web::WebState*, std::unique_ptr<web::WebStatePolicyDecider>>
190 web_state_handlers_; 189 web_state_handlers_;
191 190
192 DISALLOW_COPY_AND_ASSIGN(AccountConsistencyService); 191 DISALLOW_COPY_AND_ASSIGN(AccountConsistencyService);
193 }; 192 };
194 193
195 #endif // COMPONENTS_SIGNIN_IOS_BROWSER_ACCOUNT_CONSISTENCY_SERVICE_H_ 194 #endif // COMPONENTS_SIGNIN_IOS_BROWSER_ACCOUNT_CONSISTENCY_SERVICE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698