DevTools: Allow setting -0 value to object property

Source/devtools/front_end/RemoteObject.js

 /*
  * Copyright (C) 2009 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 233 matching lines @@
      * @param {string} value
      * @param {function(string=)} callback
      */
     setPropertyValue: function(name, value, callback)
     {
         if (!this._objectId) {
             callback("Can't set a property of non-object.");
             return;
         }
 
-        RuntimeAgent.evaluate.invoke({expression:value, doNotPauseOnExceptionsAndMuteConsole:true}, evaluatedCallback.bind(this));
+        RuntimeAgent.evaluate.invoke({expression: value, doNotPauseOnExceptionsAndMuteConsole: true, forceObjectId: true}, evaluatedCallback.bind(this));
 
         /**
          * @param {?Protocol.Error} error
          * @param {RuntimeAgent.RemoteObject} result
          * @param {boolean=} wasThrown
          */
         function evaluatedCallback(error, result, wasThrown)
         {
             if (error || wasThrown) {
                 callback(error || result.description);
@@ skipping 13 matching lines @@
      * @param {function(string=)} callback
      */
     doSetObjectPropertyValue: function(result, name, callback)
     {
         // This assignment may be for a regular (data) property, and for an acccessor property (with getter/setter).
         // Note the sensitive matter about accessor property: the property may be physically defined in some proto object,
         // but logically it is bound to the object in question. JavaScript passes this object to getters/setters, not the object
         // where property was defined; so do we.
         var setPropertyValueFunction = "function(a, b) { this[a] = b; }";
 
-        // Special case for NaN, Infinity and -Infinity
-        if (result.type === "number" && typeof result.value !== "number")
-            setPropertyValueFunction = "function(a) { this[a] = " + result.description + "; }";
-
         delete result.description; // Optimize on traffic.
-        RuntimeAgent.callFunctionOn(this._objectId, setPropertyValueFunction, [{ value:name }, result], true, undefined, undefined, propertySetCallback.bind(this));
+        RuntimeAgent.callFunctionOn(this._objectId, setPropertyValueFunction, [{ value: name }, result], true, undefined, undefined, undefined, propertySetCallback.bind(this));
 
         /**
          * @param {?Protocol.Error} error
          * @param {RuntimeAgent.RemoteObject} result
          * @param {boolean=} wasThrown
          */
         function propertySetCallback(error, result, wasThrown)
         {
             if (error || wasThrown) {
                 callback(error || result.description);
@@ skipping 39 matching lines @@
         function mycallback(error, result, wasThrown)
         {
             if (!callback)
                 return;
             if (error)
                 callback(null, false);
             else
                 callback(WebInspector.RemoteObject.fromPayload(result), wasThrown);
         }
 
-        RuntimeAgent.callFunctionOn(this._objectId, functionDeclaration.toString(), args, true, undefined, undefined, mycallback);
+        RuntimeAgent.callFunctionOn(this._objectId, functionDeclaration.toString(), args, true, undefined, undefined, undefined, mycallback);
     },
 
     /**
      * @param {function(this:Object)} functionDeclaration
      * @param {Array.<RuntimeAgent.CallArgument>|undefined} args
      * @param {function(*)} callback
      */
     callFunctionJSON: function(functionDeclaration, args, callback)
     {
         /**
          * @param {?Protocol.Error} error
          * @param {RuntimeAgent.RemoteObject} result
          * @param {boolean=} wasThrown
          */
         function mycallback(error, result, wasThrown)
         {
             callback((error || wasThrown) ? null : result.value);
         }
 
-        RuntimeAgent.callFunctionOn(this._objectId, functionDeclaration.toString(), args, true, true, false, mycallback);
+        RuntimeAgent.callFunctionOn(this._objectId, functionDeclaration.toString(), args, true, true, false, false, mycallback);
     },
 
     release: function()
     {
         if (!this._objectId)
             return;
         RuntimeAgent.releaseObject(this._objectId);
     },
 
     /**
@@ skipping 147 matching lines @@
     },
 
     /**
      * @override
      * @param {RuntimeAgent.RemoteObject} result
      * @param {string} name
      * @param {function(string=)} callback
      */
     doSetObjectPropertyValue: function(result, name, callback)
     {
-        var newValue;
-
-        switch (result.type) {
-            case "undefined":
-                newValue = {};
-                break;
-            case "object":
-            case "function":
-                newValue = { objectId: result.objectId };
-                break;
-            default:
-                newValue = { value: result.value };
-        }
-
+        var newValue = { objectId: result.objectId, value: result.value };
         DebuggerAgent.setVariableValue(this._scopeRef.number, name, newValue, this._scopeRef.callFrameId, this._scopeRef.functionId, setVariableValueCallback.bind(this));
 
         /**
          * @param {?Protocol.Error} error
          */
         function setVariableValueCallback(error)
         {
             if (error) {
                 callback(error);
                 return;
@@ skipping 243 matching lines @@
     },
 
     /**
      * @return {number}
      */
     arrayLength: function()
     {
         return this._value instanceof Array ? this._value.length : 0;
     }
 }