Timeline: promisify paint profiler and Layers panel, part I

third_party/WebKit/Source/devtools/front_end/timeline/TimelinePaintProfilerView.js

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @param {!WebInspector.TimelineFrameModel} frameModel
  * @extends {WebInspector.SplitWidget}
  */
 WebInspector.TimelinePaintProfilerView = function(frameModel)
@@ skipping 48 matching lines @@
         if (this._event.name === WebInspector.TimelineModel.RecordType.RasterTask)
             return this._frameModel.hasRasterTile(this._event);
         return false;
     },
 
     _update: function()
     {
         this._logTreeView.setCommandLog(null, []);
         this._paintProfilerView.setSnapshotAndLog(null, [], null);
 
-        if (this._event.name === WebInspector.TimelineModel.RecordType.Paint)
-            this._event.picture.requestObject(onDataAvailable.bind(this));
-        else if (this._event.name === WebInspector.TimelineModel.RecordType.RasterTask)
-            this._frameModel.requestRasterTile(this._event, onSnapshotLoaded.bind(this))
-        else
+        var snapshotPromise;
+        if (this._event.name === WebInspector.TimelineModel.RecordType.Paint) {
+            snapshotPromise = this._event.picture.objectPromise()
+                .then(data => WebInspector.PaintProfilerSnapshot.load(this._target, data["skp64"]))
+                .then(snapshot => snapshot && {rect: null, snapshot: snapshot});
+        } else if (this._event.name === WebInspector.TimelineModel.RecordType.RasterTask) {
+            snapshotPromise = this._frameModel.rasterTilePromise(this._event);
+        } else {
             console.assert(false, "Unexpected event type: " + this._event.name);
-
-        /**
-         * @param {!Object} data
-         * @this WebInspector.TimelinePaintProfilerView
-         */
-        function onDataAvailable(data)
-        {
-            if (data)
-                WebInspector.PaintProfilerSnapshot.load(this._target, data["skp64"], onSnapshotLoaded.bind(this, null));
+            return;
         }
-        /**
-         * @param {?DOMAgent.Rect} tileRect
-         * @param {?WebInspector.PaintProfilerSnapshot} snapshot
-         * @this WebInspector.TimelinePaintProfilerView
-         */
-        function onSnapshotLoaded(tileRect, snapshot)
-        {
+        snapshotPromise.then(snapshotWithRect => {
             this._disposeSnapshot();
-            this._lastLoadedSnapshot = snapshot;
-            this._imageView.setMask(tileRect);
-            if (!snapshot) {
+            if (!snapshotWithRect) {
                 this._imageView.showImage();
                 return;
             }
-            snapshot.commandLog(onCommandLogDone.bind(this, snapshot, tileRect));
-        }
+            var snapshot = snapshotWithRect.snapshot;
+            this._lastLoadedSnapshot = snapshot;
+            this._imageView.setMask(snapshotWithRect.rect);
+            snapshot.commandLog(onCommandLogDone.bind(this, snapshot, snapshotWithRect.rect));
+        });
 
         /**
          * @param {!WebInspector.PaintProfilerSnapshot} snapshot
          * @param {?DOMAgent.Rect} clipRect
          * @param {!Array.<!WebInspector.PaintProfilerLogItem>=} log
          * @this {WebInspector.TimelinePaintProfilerView}
          */
         function onCommandLogDone(snapshot, clipRect, log)
         {
             this._logTreeView.setCommandLog(snapshot.target(), log || []);
@@ skipping 92 matching lines @@
      * @param {?DOMAgent.Rect} maskRectangle
      */
     setMask: function(maskRectangle)
     {
         this._maskRectangle = maskRectangle;
         this._maskElement.classList.toggle("hidden", !maskRectangle);
     },
 
     __proto__: WebInspector.Widget.prototype
 };