Commit 26c1ce49 authored by Natalie Chouinard's avatar Natalie Chouinard Committed by Commit Bot

Remove explicit setLayout from ChromeBasePreference

This layout is already set by the PreferenceTheme, and re-setting it
here was unintentionally overriding a custom layout used by Account
Management.

This fixes a ~1dp UI regression introduced by crrev.com/c/1668628.
See bug for screenshots and additional detail.

Bug: 1097375
Change-Id: If2b2d2fa9ba74e8ce5e5ced1344dbaadecd63bdf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2255221Reviewed-by: default avatarBoris Sazonov <bsazonov@chromium.org>
Commit-Queue: Natalie Chouinard <chouinard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#781439}
parent e640eaf3
...@@ -51,7 +51,6 @@ public class ChromeBasePreference extends Preference { ...@@ -51,7 +51,6 @@ public class ChromeBasePreference extends Preference {
public ChromeBasePreference(Context context, AttributeSet attrs) { public ChromeBasePreference(Context context, AttributeSet attrs) {
super(context, attrs); super(context, attrs);
setLayoutResource(R.layout.preference_compat);
setSingleLineTitle(false); setSingleLineTitle(false);
TypedArray a = context.obtainStyledAttributes(attrs, R.styleable.ChromeBasePreference); TypedArray a = context.obtainStyledAttributes(attrs, R.styleable.ChromeBasePreference);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment