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

Side by Side Diff: ios/chrome/browser/ui/authentication/signin_promo_view_mediator.h

Issue 2749703003: Adding mediator for Sign-in promo (Closed)
Patch Set: Visual constraints 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
(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 #ifndef IOS_CHROME_BROWSER_UI_AUTHENTICATION_SIGNIN_PROMO_VIEW_MEDIATOR_H_
6 #define IOS_CHROME_BROWSER_UI_AUTHENTICATION_SIGNIN_PROMO_VIEW_MEDIATOR_H_
7
8 #import <Foundation/Foundation.h>
9
10 #import "ios/chrome/browser/ui/authentication/signin_promo_view.h"
11
12 // Class that configures a SigninPromoView instance. This class is a simple
13 // implementation that only stores data.
14 // TODO(jlebel): Create a subclass that monitor profile notification.
lpromero 2017/03/24 10:36:20 monitors presubmit will also ask you to change the
jlebel 2017/03/24 20:59:09 Done.
15 @interface SigninPromoViewMediator : NSObject<SigninPromoViewConfigurator>
16
17 @property(nonatomic, copy) NSString* userFullName;
msarda 2017/03/22 12:18:39 My understanding is that the mediator mediates bet
lpromero 2017/03/22 16:18:49 Right, the mediator is not part of the UI layer, a
jlebel 2017/03/24 20:59:09 The ProfileSigninPromoViewMediator will be able to
msarda 2017/03/27 09:34:16 I do not understand the interest of having a Signi
lpromero 2017/03/27 13:14:46 Agreed. The cell catalog should not add complexity
18 @property(nonatomic, copy) NSString* userEmail;
19 @property(nonatomic, copy) UIImage* userImage;
20
21 @end
22
23 #endif // IOS_CHROME_BROWSER_UI_AUTHENTICATION_SIGNIN_PROMO_VIEW_MEDIATOR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698