[DevTools] Added RemoteArray and RemoteFunction wrappers

[DevTools] Added RemoteArray and RemoteFunction wrappers

RemoteArray wrapper incapsulates at and length methods. RemoteFunction incapsulates targetFunction method. Wrappers have createFromObject methods which check type of object.
In utilities.js method storeResultTo is added to Promise.prototype. This method allow to save promise result value to object[propertyName] and return it for following processing.

BUG=61643
R=pfeldman@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201653

[kozy, 2015-09-01 02:42:44]

Pavel, please take a look.

[kozy, 2015-09-01 21:26:24]

Pavel, ptal, ready for review!

[pfeldman, 2015-09-02 03:05:52]

lgtm

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
File Source/devtools/front_end/platform/utilities.js (right):

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
Source/devtools/front_end/platform/utilities.js:1510: function
storeResultTo(object, propertyName, result) {
{ goes next line. not a fan of this since it is not compiler-friendly. You can't
control that propertyName makes sense on the object.

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
File Source/devtools/front_end/sdk/RemoteObject.js (right):

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
Source/devtools/front_end/sdk/RemoteObject.js:1368:
WebInspector.RemoteArray.createFromArray = function(objects)
Their names are so close, but their semantics is so different.

Use objectAsArray & createFromRemoteObjects instead?

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
Source/devtools/front_end/sdk/RemoteObject.js:1476:
WebInspector.RemoteFunction.createFromObject = function(object)
objectAsFunction

[kozy, 2015-09-02 16:33:43]

Patchset #4 (id:60001) has been deleted

[kozy, 2015-09-02 16:34:04]

All done.

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
File Source/devtools/front_end/platform/utilities.js (right):

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
Source/devtools/front_end/platform/utilities.js:1510: function
storeResultTo(object, propertyName, result) {
On 2015/09/02 03:05:52, pfeldman wrote:
> { goes next line. not a fan of this since it is not compiler-friendly. You
can't
> control that propertyName makes sense on the object.

Done.

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
File Source/devtools/front_end/sdk/RemoteObject.js (right):

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
Source/devtools/front_end/sdk/RemoteObject.js:1368:
WebInspector.RemoteArray.createFromArray = function(objects)
On 2015/09/02 03:05:52, pfeldman wrote:
> Their names are so close, but their semantics is so different.
> 
> Use objectAsArray & createFromRemoteObjects instead?

Done.

https://codereview.chromium.org/1325783003/diff/40001/Source/devtools/front_e...
Source/devtools/front_end/sdk/RemoteObject.js:1476:
WebInspector.RemoteFunction.createFromObject = function(object)
On 2015/09/02 03:05:52, pfeldman wrote:
> objectAsFunction

Done.

[kozy, 2015-09-02 16:34:08]

The CQ bit was checked by kozyatinskiy@chromium.org

[kozy, 2015-09-02 16:34:08]

The patchset sent to the CQ was uploaded after l-g-t-m from
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1325783003/#ps80001
(title: " ")

[commit-bot: I haz the power, 2015-09-02 16:34:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1325783003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1325783003/80001

[commit-bot: I haz the power, 2015-09-02 17:32:37]

Committed patchset #4 (id:80001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201653