From: Cheng Zhao Date: Fri, 20 Sep 2013 13:37:47 +0000 (+0800) Subject: Use event.returnValue instead of event.result in atom-shell's code. X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a9c824eba1ea2cc028b2b813f8f8ad9d8edf685b;p=platform%2Fframework%2Fweb%2Fcrosswalk-tizen.git Use event.returnValue instead of event.result in atom-shell's code. event.result is still kept for backward compatible. --- diff --git a/browser/atom/rpc-server.coffee b/browser/atom/rpc-server.coffee index f62cd09..30e9dbd 100644 --- a/browser/atom/rpc-server.coffee +++ b/browser/atom/rpc-server.coffee @@ -63,15 +63,15 @@ unwrapArgs = (processId, routingId, args) -> ipc.on 'ATOM_BROWSER_REQUIRE', (event, processId, routingId, module) -> try - event.result = valueToMeta processId, routingId, require(module) + event.returnValue = valueToMeta processId, routingId, require(module) catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_GLOBAL', (event, processId, routingId, name) -> try - event.result = valueToMeta processId, routingId, global[name] + event.returnValue = valueToMeta processId, routingId, global[name] catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_RELEASE_RENDER_VIEW', (event, processId, routingId) -> objectsRegistry.clear processId, routingId @@ -80,9 +80,9 @@ ipc.on 'ATOM_BROWSER_CURRENT_WINDOW', (event, processId, routingId) -> try BrowserWindow = require 'browser-window' window = BrowserWindow.fromProcessIdAndRoutingId processId, routingId - event.result = valueToMeta processId, routingId, window + event.returnValue = valueToMeta processId, routingId, window catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_CONSTRUCTOR', (event, processId, routingId, id, args) -> try @@ -91,18 +91,18 @@ ipc.on 'ATOM_BROWSER_CONSTRUCTOR', (event, processId, routingId, id, args) -> # Call new with array of arguments. # http://stackoverflow.com/questions/1606797/use-of-apply-with-new-operator-is-this-possible obj = new (Function::bind.apply(constructor, [null].concat(args))) - event.result = valueToMeta processId, routingId, obj + event.returnValue = valueToMeta processId, routingId, obj catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_FUNCTION_CALL', (event, processId, routingId, id, args) -> try args = unwrapArgs processId, routingId, args func = objectsRegistry.get id ret = func.apply global, args - event.result = valueToMeta processId, routingId, ret + event.returnValue = valueToMeta processId, routingId, ret catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_MEMBER_CONSTRUCTOR', (event, processId, routingId, id, method, args) -> try @@ -110,32 +110,32 @@ ipc.on 'ATOM_BROWSER_MEMBER_CONSTRUCTOR', (event, processId, routingId, id, meth constructor = objectsRegistry.get(id)[method] # Call new with array of arguments. obj = new (Function::bind.apply(constructor, [null].concat(args))) - event.result = valueToMeta processId, routingId, obj + event.returnValue = valueToMeta processId, routingId, obj catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_MEMBER_CALL', (event, processId, routingId, id, method, args) -> try args = unwrapArgs processId, routingId, args obj = objectsRegistry.get id ret = obj[method].apply(obj, args) - event.result = valueToMeta processId, routingId, ret + event.returnValue = valueToMeta processId, routingId, ret catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_MEMBER_SET', (event, processId, routingId, id, name, value) -> try obj = objectsRegistry.get id obj[name] = value catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_MEMBER_GET', (event, processId, routingId, id, name) -> try obj = objectsRegistry.get id - event.result = valueToMeta processId, routingId, obj[name] + event.returnValue = valueToMeta processId, routingId, obj[name] catch e - event.result = errorToMeta e + event.returnValue = errorToMeta e ipc.on 'ATOM_BROWSER_DEREFERENCE', (processId, routingId, storeId) -> objectsRegistry.remove processId, routingId, storeId diff --git a/spec/main.js b/spec/main.js index f2e5262..ac211ff 100644 --- a/spec/main.js +++ b/spec/main.js @@ -23,7 +23,7 @@ ipc.on('process.exit', function(pid, rid, code) { }); ipc.on('eval', function(ev, pid, rid, script) { - ev.result = eval(script); + ev.returnValue = eval(script); }); process.on('uncaughtException', function() {