Update two chromeos system dialogs to reuse more DialogDelegateView

Update two chromeos system dialogs to reuse more DialogDelegateView
functionality.

Screenshots on the bug - this is so vastly better looking IMO as to not
need UI review.

BUG=658410, 633986
Committed: https://crrev.com/a02af5e6818e4e5e89a5c594557d5389b9d6c4a8
Cr-Commit-Position: refs/heads/master@{#426934}

[Evan Stade, 2016-10-21 21:43:47]

Description was changed from

==========
Update two chromeos system dialogs to reuse more DialogDelegateView
functionality.

Screenshots on the bug - this is so vastly better looking IMO as to not
need UI review.

BUG=658410
==========

to

==========
Update two chromeos system dialogs to reuse more DialogDelegateView
functionality.

Screenshots on the bug - this is so vastly better looking IMO as to not
need UI review.

BUG=658410,633986
==========

[Evan Stade, 2016-10-21 21:44:56]

estade@chromium.org changed reviewers:
+ derat@chromium.org

[Daniel Erat, 2016-10-21 22:00:39]

lgtm

agreed that this looks much 


better than the whitespace-heavy




old dialogs

.

https://codereview.chromium.org/2439863004/diff/1/chrome/app/chromeos_strings...
File chrome/app/chromeos_strings.grdp (right):

https://codereview.chromium.org/2439863004/diff/1/chrome/app/chromeos_strings...
chrome/app/chromeos_strings.grdp:5595: This feature allows you to access quickly
any signed-in user without requiring a password.
"to access quickly any signed-in user" <-- yay awkward phrasing to avoid split
infinitive :-P

https://codereview.chromium.org/2439863004/diff/1/chrome/app/chromeos_strings...
chrome/app/chromeos_strings.grdp:5599: <message
name="IDS_MULTIPROFILES_INTRO_EXPLANATION" desc="Content of the multi-profiles
intro dialog shown when the user adds another user in multi-profiles mode.">
i think that this is unused now and can be deleted.

[Evan Stade, 2016-10-21 22:17:42]

https://codereview.chromium.org/2439863004/diff/1/chrome/app/chromeos_strings...
File chrome/app/chromeos_strings.grdp (right):

https://codereview.chromium.org/2439863004/diff/1/chrome/app/chromeos_strings...
chrome/app/chromeos_strings.grdp:5595: This feature allows you to access quickly
any signed-in user without requiring a password.
On 2016/10/21 22:00:39, Daniel Erat wrote:
> "to access quickly any signed-in user" <-- yay awkward phrasing to avoid split
> infinitive :-P

I'm going to fix this because "to quickly access..." sounds 10x more fluent than
"to access quickly...", and because Star Trek is a better model to follow than
Strunk and White.

https://codereview.chromium.org/2439863004/diff/1/chrome/app/chromeos_strings...
chrome/app/chromeos_strings.grdp:5599: <message
name="IDS_MULTIPROFILES_INTRO_EXPLANATION" desc="Content of the multi-profiles
intro dialog shown when the user adds another user in multi-profiles mode.">
On 2016/10/21 22:00:39, Daniel Erat wrote:
> i think that this is unused now and can be deleted.

done

[Evan Stade, 2016-10-21 22:17:45]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2016-10-21 22:17:45]

The patchset sent to the CQ was uploaded after l-g-t-m from derat@chromium.org
Link to the patchset: https://codereview.chromium.org/2439863004/#ps20001
(title: "to boldly make changes")

[commit-bot: I haz the power, 2016-10-21 22:18:36]

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

[Evan Stade, 2016-10-21 22:36:26]

side note, I confirmed the string change with ainslie@

[commit-bot: I haz the power, 2016-10-22 00:25:45]

Description was changed from

==========
Update two chromeos system dialogs to reuse more DialogDelegateView
functionality.

Screenshots on the bug - this is so vastly better looking IMO as to not
need UI review.

BUG=658410,633986
==========

to

==========
Update two chromeos system dialogs to reuse more DialogDelegateView
functionality.

Screenshots on the bug - this is so vastly better looking IMO as to not
need UI review.

BUG=658410,633986
==========

[commit-bot: I haz the power, 2016-10-22 00:25:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-22 00:28:32]

Description was changed from

==========
Update two chromeos system dialogs to reuse more DialogDelegateView
functionality.

Screenshots on the bug - this is so vastly better looking IMO as to not
need UI review.

BUG=658410,633986
==========

to

==========
Update two chromeos system dialogs to reuse more DialogDelegateView
functionality.

Screenshots on the bug - this is so vastly better looking IMO as to not
need UI review.

BUG=658410,633986

Committed: https://crrev.com/a02af5e6818e4e5e89a5c594557d5389b9d6c4a8
Cr-Commit-Position: refs/heads/master@{#426934}
==========

[commit-bot: I haz the power, 2016-10-22 00:28:34]

Patchset 2 (id:??) landed as
https://crrev.com/a02af5e6818e4e5e89a5c594557d5389b9d6c4a8
Cr-Commit-Position: refs/heads/master@{#426934}