DevTools: [CSS] promisify CSSStyleModel fetching methods

Source/devtools/front_end/emulation/MediaQueryInspector.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
  * @extends {WebInspector.Widget}
  * @implements {WebInspector.TargetManager.Observer}
  */
 WebInspector.MediaQueryInspector = function()
@@ skipping 167 matching lines @@
     /**
      * @param {!WebInspector.Throttler.FinishCallback} finishCallback
      */
     _refetchMediaQueries: function(finishCallback)
     {
         if (!this._enabled || !this._cssModel) {
             finishCallback();
             return;
         }
 
-        /**
-         * @param {!Array.<!WebInspector.CSSMedia>} cssMedias
-         * @this {!WebInspector.MediaQueryInspector}
-         */
-        function callback(cssMedias)
-        {
-            this._rebuildMediaQueries(cssMedias);
-            finishCallback();
-        }
-        this._cssModel.getMediaQueries(callback.bind(this));
+        this._cssModel.mediaQueriesPromise()
+            .then(this._rebuildMediaQueries.bind(this))
+            .then(finishCallback)

[pfeldman, 2015/06/26 09:59:17]

then(finishCallback, finishCallback);

[lushnikov, 2015/06/26 10:16:26]

The .catch is way more easy to read and find visually.
I would rather leave this as-is.

+            .catch(finishCallback);
     },
 
     /**
      * @param {!Array.<!WebInspector.MediaQueryInspector.MediaQueryUIModel>} models
      * @return {!Array.<!WebInspector.MediaQueryInspector.MediaQueryUIModel>}
      */
     _squashAdjacentEqual: function(models)
     {
         var filtered = [];
         for (var i = 0; i < models.length; ++i) {
@@ skipping 299 matching lines @@
     },
 
     /**
      * @return {boolean}
      */
     active: function()
     {
         return this._active;
     }
 }