SOM: Removed alert-body div.

SOM: Removed alert-body div.

BUG=619187
Committed: https://chromium.googlesource.com/infra/infra/+/7780e52fc82d28e95901002c2438f6e24bd0e1af

[Tiff, 2016-06-21 21:33:37]

zhangtiff@google.com changed reviewers:
+ seanmccullough@chromium.org

[Tiff, 2016-06-21 21:33:38]

PTAL

[seanmccullough1, 2016-06-21 21:39:43]

seanmccullough@google.com changed reviewers:
+ seanmccullough@google.com

[seanmccullough1, 2016-06-21 21:39:44]

https://codereview.chromium.org/2085553006/diff/1/go/src/infra/appengine/sher...
File go/src/infra/appengine/sheriff-o-matic/elements/som-alert-item.html
(right):

https://codereview.chromium.org/2085553006/diff/1/go/src/infra/appengine/sher...
go/src/infra/appengine/sheriff-o-matic/elements/som-alert-item.html:78:
[[alert.body]]
I think the intent was to remove alert.body *completely*, since it usually
doesn't usually say anything that isn't already in the title or otherwise
constructed in the UI from other fields.

[Tiff, 2016-06-21 21:45:24]

On 2016/06/21 at 21:39:44, seanmccullough wrote:
>
https://codereview.chromium.org/2085553006/diff/1/go/src/infra/appengine/sher...
> File go/src/infra/appengine/sheriff-o-matic/elements/som-alert-item.html
(right):
> 
>
https://codereview.chromium.org/2085553006/diff/1/go/src/infra/appengine/sher...
> go/src/infra/appengine/sheriff-o-matic/elements/som-alert-item.html:78:
[[alert.body]]
> I think the intent was to remove alert.body *completely*, since it usually
doesn't usually say anything that isn't already in the title or otherwise
constructed in the UI from other fields.

Ahh, looking at it more carefully, I think you are absolutely right. Thanks! I
went ahead and uploaded a new patch.

[seanmccullough1, 2016-06-21 21:48:26]

lgtm

[Tiff, 2016-06-21 21:48:43]

The CQ bit was checked by zhangtiff@google.com

[seanmccullough, 2016-06-21 21:48:44]

lgtm

[Tiff, 2016-06-21 21:52:59]

The CQ bit was checked by zhangtiff@google.com

[commit-bot: I haz the power, 2016-06-21 21:53:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2085553006/20001

[commit-bot: I haz the power, 2016-06-21 22:26:14]

Description was changed from

==========
SOM: Removed alert-body div.

BUG=619187
==========

to

==========
SOM: Removed alert-body div.

BUG=619187

Committed:
https://chromium.googlesource.com/infra/infra/+/7780e52fc82d28e95901002c2438f...
==========

[commit-bot: I haz the power, 2016-06-21 22:26:15]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/7780e52fc82d28e95901002c2438f...