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

Unified Diff: ios/chrome/browser/ui/favicon_view.mm

Issue 2787503002: Move FaviconAttributes to UI (Closed)
Patch Set: Add blank line 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ios/chrome/browser/ui/favicon_view.h ('k') | ios/chrome/browser/ui/reading_list/BUILD.gn » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ios/chrome/browser/ui/favicon_view.mm
diff --git a/ios/chrome/browser/ui/favicon_view.mm b/ios/chrome/browser/ui/favicon_view.mm
deleted file mode 100644
index a8de21f21cceb5e63e3d2f3135c43c0f58479c98..0000000000000000000000000000000000000000
--- a/ios/chrome/browser/ui/favicon_view.mm
+++ /dev/null
@@ -1,88 +0,0 @@
-// Copyright 2016 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#import "ios/chrome/browser/ui/favicon_view.h"
-
-#import "ios/chrome/browser/ui/uikit_ui_util.h"
-
-#if !defined(__has_feature) || !__has_feature(objc_arc)
-#error "This file requires ARC support."
-#endif
-
-namespace {
-// Default corner radius for the favicon image view.
-const CGFloat kDefaultCornerRadius = 3;
-}
-
-@interface FaviconViewNew () {
- // Property releaser for FaviconViewNew.
-}
-
-// Image view for the favicon.
-@property(nonatomic, strong) UIImageView* faviconImageView;
-// Label for fallback favicon placeholder.
-@property(nonatomic, strong) UILabel* faviconFallbackLabel;
-
-@end
-
-@implementation FaviconViewNew
-@synthesize faviconImageView = _faviconImageView;
-@synthesize faviconFallbackLabel = _faviconFallbackLabel;
-
-- (instancetype)initWithFrame:(CGRect)frame {
- self = [super initWithFrame:frame];
- if (self) {
- _faviconImageView = [[UIImageView alloc] initWithFrame:self.bounds];
- _faviconImageView.clipsToBounds = YES;
- _faviconImageView.layer.cornerRadius = kDefaultCornerRadius;
- _faviconImageView.image = nil;
-
- _faviconFallbackLabel = [[UILabel alloc] initWithFrame:self.bounds];
- _faviconFallbackLabel.backgroundColor = [UIColor clearColor];
- _faviconFallbackLabel.textAlignment = NSTextAlignmentCenter;
- _faviconFallbackLabel.isAccessibilityElement = NO;
- _faviconFallbackLabel.clipsToBounds = YES;
- _faviconFallbackLabel.layer.cornerRadius = kDefaultCornerRadius;
- _faviconFallbackLabel.text = nil;
-
- [self addSubview:_faviconFallbackLabel];
- [self addSubview:_faviconImageView];
-
- [_faviconImageView setTranslatesAutoresizingMaskIntoConstraints:NO];
- [_faviconFallbackLabel setTranslatesAutoresizingMaskIntoConstraints:NO];
-
- // Both image and fallback label are centered and match the size of favicon.
- AddSameCenterConstraints(_faviconImageView, self);
- AddSameCenterConstraints(_faviconFallbackLabel, self);
- AddSameSizeConstraint(_faviconFallbackLabel, self);
- AddSameSizeConstraint(_faviconImageView, self);
- }
- return self;
-}
-
-- (void)configureWithAttributes:(FaviconAttributes*)attributes {
- if (attributes.faviconImage) {
- self.faviconImageView.image = attributes.faviconImage;
- self.faviconImageView.hidden = NO;
- self.faviconFallbackLabel.hidden = YES;
- } else {
- self.faviconFallbackLabel.backgroundColor = attributes.backgroundColor;
- self.faviconFallbackLabel.textColor = attributes.textColor;
- self.faviconFallbackLabel.text = attributes.monogramString;
- self.faviconFallbackLabel.hidden = NO;
- self.faviconImageView.hidden = YES;
- }
-}
-
-- (void)setFont:(UIFont*)font {
- self.faviconFallbackLabel.font = font;
-}
-
-#pragma mark - UIView
-
-- (CGSize)intrinsicContentSize {
- return CGSizeMake(kFaviconPreferredSize, kFaviconPreferredSize);
-}
-
-@end
« no previous file with comments | « ios/chrome/browser/ui/favicon_view.h ('k') | ios/chrome/browser/ui/reading_list/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698