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

Unified Diff: ios/public/provider/chrome/browser/signin/fake_chrome_identity_service.h

Issue 2673553003: [ObjC ARC] Converts ios/chrome/browser/ui/history:eg_tests to ARC. (Closed)
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: ios/public/provider/chrome/browser/signin/fake_chrome_identity_service.h
diff --git a/ios/public/provider/chrome/browser/signin/fake_chrome_identity_service.h b/ios/public/provider/chrome/browser/signin/fake_chrome_identity_service.h
index c3e52c47f30cdc8989e9edcd12b8ae4452470745..b378fd985a652458456a5408cb6431dad79cc203 100644
--- a/ios/public/provider/chrome/browser/signin/fake_chrome_identity_service.h
+++ b/ios/public/provider/chrome/browser/signin/fake_chrome_identity_service.h
@@ -52,7 +52,8 @@ class FakeChromeIdentityService : public ChromeIdentityService {
MOCK_METHOD2(GetAvatarForIdentity,
void(ChromeIdentity* identity, GetAvatarCallback callback));
- MOCK_METHOD1(GetCachedAvatarForIdentity, UIImage*(ChromeIdentity* identity));
+ virtual UIImage* GetCachedAvatarForIdentity(
marq (ping after 24h) 2017/02/03 14:26:14 Add a comment explaining why this can't be mocked
stkhapugin 2017/02/03 15:49:07 I actually made a separate CL where I've addressed
+ ChromeIdentity* identity) override;
MOCK_METHOD2(GetHostedDomainForIdentity,
void(ChromeIdentity* identity,

Powered by Google App Engine
This is Rietveld 408576698