MD Settings: Fix issue of long names breaking layout bounds in People section

MD Settings: Fix issue of long names breaking layout bounds in People section

BUG=666666
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2629483002
Cr-Commit-Position: refs/heads/master@{#443280}
Committed: https://chromium.googlesource.com/chromium/src/+/5e234b7e88e8a8f0c48de5713460fe600d510e72

[scottchen, 2017-01-11 21:55:18]

Description was changed from

==========
MD Settings: Fix issue of long names breaking layout bounds in People section

BUG=666666
==========

to

==========
MD Settings: Fix issue of long names breaking layout bounds in People section

BUG=666666
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[scottchen, 2017-01-11 21:56:41]

scottchen@chromium.org changed reviewers:
+ dpapad@chromium.org, tommycli@chromium.org

[scottchen, 2017-01-11 21:56:48]

On 2017/01/11 21:56:41, scottchen wrote:

http://imgur.com/a/kmFic

[Dan Beam, 2017-01-11 21:59:01]

dbeam@chromium.org changed reviewers:
+ dbeam@chromium.org

[Dan Beam, 2017-01-11 21:59:01]

lgtm

https://codereview.chromium.org/2629483002/diff/1/chrome/browser/resources/se...
File chrome/browser/resources/settings/people_page/people_page.html (right):

https://codereview.chromium.org/2629483002/diff/1/chrome/browser/resources/se...
chrome/browser/resources/settings/people_page/people_page.html:114: <div
class="middle two-line text-elide" on-tap="onPictureTap_" actionable>
80 col wrap

[dpapad, 2017-01-11 22:12:38]

dpapad@chromium.org changed reviewers:
- dpapad@chromium.org

[tommycli, 2017-01-11 22:33:47]

lgtm

[scottchen, 2017-01-11 22:47:07]

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

[commit-bot: I haz the power, 2017-01-11 22:48:33]

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-01-12 00:17:29]

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

[commit-bot: I haz the power, 2017-01-12 00:17:30]

Dry run: This issue passed the CQ dry run.

[scottchen, 2017-01-12 17:34:13]

The CQ bit was checked by scottchen@chromium.org

[scottchen, 2017-01-12 17:34:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
tommycli@chromium.org, dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/2629483002/#ps20001
(title: "80 col")

[commit-bot: I haz the power, 2017-01-12 17:34:44]

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-01-12 17:41:11]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484242453121200,
"parent_rev": "f38ee194e671002a245a2a2c0c52cc7b0a762da1", "commit_rev":
"5e234b7e88e8a8f0c48de5713460fe600d510e72"}

[commit-bot: I haz the power, 2017-01-12 17:42:03]

Description was changed from

==========
MD Settings: Fix issue of long names breaking layout bounds in People section

BUG=666666
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD Settings: Fix issue of long names breaking layout bounds in People section

BUG=666666
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2629483002
Cr-Commit-Position: refs/heads/master@{#443280}
Committed:
https://chromium.googlesource.com/chromium/src/+/5e234b7e88e8a8f0c48de5713460...
==========

[commit-bot: I haz the power, 2017-01-12 17:42:04]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/5e234b7e88e8a8f0c48de5713460...