Skip to content

Commit

Permalink
Fix password settings crash by modifying property attributes of
Browse files Browse the repository at this point in the history
TableViewTextItem.textColor.

The Canary crashes when a TableViewTextItem configures its cell and
checks its "textColor" property, which is "assign" and might get released
after assigned. This CL fixes it by changing the property's "assign"
attribute to "strong" so that the UIColor object will be retained.

Bug: 910542
Change-Id: I44b6a60c52c5a470e69a44ca6670720446ef084b
Reviewed-on: https://chromium-review.googlesource.com/c/1356585
Reviewed-by: Sergio Collazos <sczs@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Commit-Queue: Yi Su <mrsuyi@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#612716}(cherry picked from commit 17c608f)
Reviewed-on: https://chromium-review.googlesource.com/c/1356809
Reviewed-by: Yi Su <mrsuyi@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#7}
Cr-Branched-From: d897fb1-refs/heads/master@{#612437}
  • Loading branch information
Yi Su committed Dec 3, 2018
1 parent 3272cbc commit e8d7195
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
// UIColor for the cell's textLabel. Default is
// kTableViewTextLabelColorLightGrey. ChromeTableViewStyler's |cellTitleColor|
// takes precedence over the default color, but not over |textColor|.
@property(nonatomic, assign) UIColor* textColor;
@property(nonatomic, strong) UIColor* textColor;

@property(nonatomic, strong) NSString* text;

Expand Down

0 comments on commit e8d7195

Please sign in to comment.