Adjust view for the password exceptions in the new UI

Adjust view for the password exceptions in the new UI

BUG=619868
Review-Url: https://codereview.chromium.org/2966383004
Cr-Commit-Position: refs/heads/master@{#485289}
Committed: https://chromium.googlesource.com/chromium/src/+/6383dae82efae518c8bea41fae9a0fcec6ff461e

[melandory, 2017-07-07 14:54:15]

The CQ bit was checked by melandory@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-07 14:54:22]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-07 15:38:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-07-07 15:38:54]

Dry run: This issue passed the CQ dry run.

[melandory, 2017-07-10 08:12:23]

The CQ bit was checked by melandory@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-10 08:12:29]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-10 10:04:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-07-10 10:04:36]

Dry run: This issue passed the CQ dry run.

[melandory, 2017-07-10 10:52:18]

melandory@chromium.org changed reviewers:
+ twellington@chromium.org

[melandory, 2017-07-10 10:52:19]

PTAL, thanks.

(Screenshots are attached to the bug)

[Theresa, 2017-07-10 15:12:51]

lgtm % style nits

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
File chrome/android/java/res/layout/password_entry_editor_site_row.xml (right):

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
chrome/android/java/res/layout/password_entry_editor_site_row.xml:21:
android:textAppearance="?android:attr/textAppearanceMedium" />
nit: add a blank line after this one.

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
File chrome/android/java/res/layout/password_entry_exception.xml (right):

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
chrome/android/java/res/layout/password_entry_exception.xml:18: 
nit: remove one of these extra blank lines and add a blank line after the
<include

[melandory, 2017-07-10 15:19:06]

The CQ bit was checked by melandory@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-10 15:19:19]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-10 16:10:38]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-07-10 16:10:39]

Dry run: This issue passed the CQ dry run.

[melandory, 2017-07-10 16:19:37]

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
File chrome/android/java/res/layout/password_entry_editor_site_row.xml (right):

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
chrome/android/java/res/layout/password_entry_editor_site_row.xml:21:
android:textAppearance="?android:attr/textAppearanceMedium" />
On 2017/07/10 15:12:50, Theresa wrote:
> nit: add a blank line after this one.

Done.

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
File chrome/android/java/res/layout/password_entry_exception.xml (right):

https://codereview.chromium.org/2966383004/diff/20001/chrome/android/java/res...
chrome/android/java/res/layout/password_entry_exception.xml:18: 
On 2017/07/10 15:12:50, Theresa wrote:
> nit: remove one of these extra blank lines and add a blank line after the
> <include

Done.

[melandory, 2017-07-10 16:19:41]

The CQ bit was checked by melandory@chromium.org

[melandory, 2017-07-10 16:19:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
twellington@chromium.org
Link to the patchset: https://codereview.chromium.org/2966383004/#ps40001
(title: "adressed comments")

[commit-bot: I haz the power, 2017-07-10 16:19:55]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-10 16:23:30]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1499703581882050,
"parent_rev": "a4440342df5feaf88c50eecb6110541609c7fbd9", "commit_rev":
"6383dae82efae518c8bea41fae9a0fcec6ff461e"}

[commit-bot: I haz the power, 2017-07-10 16:23:47]

Description was changed from

==========
Adjust view for the password exceptions in the new UI

BUG=619868
==========

to

==========
Adjust view for the password exceptions in the new UI

BUG=619868

Review-Url: https://codereview.chromium.org/2966383004
Cr-Commit-Position: refs/heads/master@{#485289}
Committed:
https://chromium.googlesource.com/chromium/src/+/6383dae82efae518c8bea41fae9a...
==========

[commit-bot: I haz the power, 2017-07-10 16:23:49]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/6383dae82efae518c8bea41fae9a...