SoM: Added notice about chromium webkit failures + documentation links.

go/src/infra/appengine/sheriff-o-matic/elements/som-alert-item.html

 <link rel="import" href="/bower_components/polymer/polymer.html">
 
 <link rel="import" href="/bower_components/iron-icon/iron-icon.html">
 <link rel="import" href="/bower_components/iron-icons/iron-icons.html">
 <link rel="import" href="/bower_components/paper-button/paper-button.html">
 <link rel="import" href="/bower_components/paper-icon-button/paper-icon-button.html">
 <link rel="import" href="/elements/som-extension-build-failure.html">
 <link rel="import" href="/elements/som-uberify-behavior.html">
 
 <dom-module id="som-alert-item">
@@ skipping 29 matching lines @@
      .file-bug-icon,
      .snooze-icon {
        color: #aaa;
      }
      .snoozed {
        opacity: 0.5;
      }
      #root {
        transition: opacity .25s;
      }
-     #bugsList {
+     #bugsList,
+     #handleLink {
        margin-left: 15px;
      }
      hr {
        width: 100%;
        background-color: #000;
      }
      .divider {
        width: 20px;
      }
     </style>
     <div id="root" class$="[[_cssClass]]">
       <div class$="[[_classForAlert(alert, selected)]]">
         <div class="list-item">
           <div class="alert-title-bar horizontal layout justified">
             <div class="horizontal layout">
-              <div class="alert-title self-center">[[alert.title]]</div>
+              <div class="alert-title self-center">[[alert.title]] </div>
+              <div id="handleLink" class="self-center" hidden$="[[!_helpLinkForAlert(alert)]]">
+                [<a href="[[_helpLinkForAlert(alert)]]">Help</a>]
+              </div>
               <div id="bugsList" hidden$="[[!_hasBugs]]" class="alert-links">
                 <template is="dom-repeat" items="[[annotation.bugs]]" as="bug">
                   <a target="_blank" href$="[[bug]]">[[_bugLabel(bug)]]</a>
                   <paper-button id="remove[[index]]" on-tap="_removeBug" raised>Remove</paper-button>
                 </template>
               </div>
             </div>
             <div class="alert-times">
               [[_duration]]
             </div>
@@ skipping 98 matching lines @@
         let deltaSec = Math.round((alert.time - alert.start_time));
         let hours = Math.floor(deltaSec/60/60);
         let minutes = Math.floor((deltaSec - hours*60*60)/60);
         let seconds = deltaSec - hours * 60*60 - minutes*60;
         if (hours == 0 && minutes == 0 && seconds == 0) {
           return '';
         }
         return `Active for: ${hours}h ${minutes}m ${seconds}s`;
       },
 
+      _helpLinkForAlert: function(alert) {
+        // TODO: Add documentation links for other kinds of alerts
+        if (this._alertIsWebkit(alert)) {
+          return "http://www.chromium.org/blink/sheriffing";
+        }
+        return null;
+      },
+
+      _alertIsWebkit(alert) {
+        // TODO: Find a better way to categorize alerts

[martiniss, 2016/06/23 01:20:25]

Usually, with TODOs, you do something like TODO(zhangtiff) (or another
username), so that there's someone you can bother if you run into this TODO
later.

+        return alert.key.indexOf("chromium.webkit") > -1;

[martiniss, 2016/06/23 01:20:25]

I believe there's an array.include you can use.

[Tiff, 2016/06/23 17:24:57]

alert.key is a string so I don't think that'd work here, but I imagine there
might possibly be a better field for me to use than the key. 

[seanmccullough1, 2016/06/23 17:36:41]

There is an includes() on String too :)
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Obje...

[Tiff, 2016/06/23 18:46:58]

Ahh, you're right! Submitted another patch for this. Thank you. :)

+      },
+
       _classForAlert: function(alert, selected) {
         return 'alert' + (selected ? ' selected' : '');
       },
 
       _computeHasBugs: function(bugs) {
         return !!(bugs && bugs.length > 0);
       },
 
       _computeSnoozeText: function(snoozed) {
         return snoozed ? 'Unsnooze' : 'Snooze';
@@ skipping 38 matching lines @@
             change: {'snoozeTime': true},
           });
         } else {
           this.fire('snooze');
         }
       },
     });
   })();
   </script>
 </dom-module>