DevTools: bring violations and timeline warnings in sync.

third_party/WebKit/Source/devtools/front_end/timeline_model/TimelineFrameModel.js

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 426 matching lines @@
     /** @type {!Array.<!TimelineModel.LayerPaintEvent>} */
     this._paints = [];
     /** @type {number|undefined} */
     this._mainFrameId = undefined;
   }
 
   /**
    * @return {boolean}
    */
   hasWarnings() {
-    var /** @const */ longFrameDurationThresholdMs = 22;
-    return !this.idle && this.duration > longFrameDurationThresholdMs;
+    return false;

[caseq, 2016/11/19 05:51:08]

This function becomes meaningless in its present form, it's not an override.
Please nuke it altogether along with all usages.

   }
 
   /**
    * @param {number} endTime
    */
   _setEndTime(endTime) {
     this.endTime = endTime;
     this.duration = this.endTime - this.startTime;
   }
 
@@ skipping 87 matching lines @@
   constructor(triggerTime, timeByCategory) {
     /** @type {!Object.<string, number>} */
     this.timeByCategory = timeByCategory;
     /** @type {!Array.<!TimelineModel.LayerPaintEvent>} */
     this.paints = [];
     /** @type {number|undefined} */
     this.mainFrameId = undefined;
     this.triggerTime = triggerTime;
   }
 };