Minor clean-up; remove old comment

git-svn-id: http://svg-edit.googlecode.com/svn/trunk@2647 eee81c28-f429-11dd-99c0-75d572ba1ddd
master
Brett Zamir 2014-01-31 07:13:17 +00:00
parent 5f01a845e6
commit 881d82859b
1 changed files with 18 additions and 17 deletions

View File

@ -41,12 +41,12 @@ var cbid = 0;
function getCallbackSetter (d) { function getCallbackSetter (d) {
return function(){ return function(){
var t = this, // new callback var t = this, // New callback
args = [].slice.call(arguments), args = [].slice.call(arguments),
cbid = t.send(d, args, function(){}); // the callback (currently it's nothing, but will be set later) cbid = t.send(d, args, function(){}); // The callback (currently it's nothing, but will be set later)
return function(newcallback){ return function(newcallback){
t.callbacks[cbid] = newcallback; // set callback t.callbacks[cbid] = newcallback; // Set callback
}; };
}; };
} }
@ -55,9 +55,9 @@ function EmbeddedSVGEdit(frame){
if (!(this instanceof EmbeddedSVGEdit)) { // Allow invocation without "new" keyword if (!(this instanceof EmbeddedSVGEdit)) { // Allow invocation without "new" keyword
return new EmbeddedSVGEdit(frame); return new EmbeddedSVGEdit(frame);
} }
// initialize communication // Initialize communication
this.frame = frame; this.frame = frame;
this.callbacks = {}; // successor to stack this.callbacks = {};
// List of functions extracted with this: // List of functions extracted with this:
// Run in firebug on http://svg-edit.googlecode.com/svn/trunk/docs/files/svgcanvas-js.html // Run in firebug on http://svg-edit.googlecode.com/svn/trunk/docs/files/svgcanvas-js.html
@ -67,8 +67,8 @@ function EmbeddedSVGEdit(frame){
//"moveSelectedToLayer", "clear"]; //"moveSelectedToLayer", "clear"];
// Newer, well, it extracts things that aren't documented as well. All functions accessible through the normal thingy can now be accessed though the API // Newer, well, it extracts things that aren't documented as well. All functions accessible through the normal thingy can now be accessed though the API
//var l=[];for(var i in svgCanvas){if(typeof svgCanvas[i] == "function"){l.push(i)}}; // var l = []; for (var i in svgCanvas){ if (typeof svgCanvas[i] == "function") { l.push(i);} };
//run in svgedit itself // Run in svgedit itself
var i, var i,
t = this, t = this,
functions = ["updateElementFromJson", "embedImage", "fixOperaXML", "clearSelection", "addToSelection", functions = ["updateElementFromJson", "embedImage", "fixOperaXML", "clearSelection", "addToSelection",
@ -94,7 +94,8 @@ function EmbeddedSVGEdit(frame){
// Older IE may need a polyfill for addEventListener, but so it would for SVG // Older IE may need a polyfill for addEventListener, but so it would for SVG
window.addEventListener('message', function(e) { window.addEventListener('message', function(e) {
// We accept and post strings for the sake of IE9 support // We accept and post strings as opposed to objets for the sake of IE9 support; this
// will most likely be changed in the future
if (typeof e.data !== 'string') { if (typeof e.data !== 'string') {
return; return;
} }
@ -120,7 +121,7 @@ EmbeddedSVGEdit.prototype.send = function(name, args, callback){
cbid++; cbid++;
this.callbacks[cbid] = callback; this.callbacks[cbid] = callback;
setTimeout(function(){ // delay for the callback to be set in case its synchronous setTimeout(function(){ // Delay for the callback to be set in case its synchronous
// Todo: Handle non-JSON arguments and return values (undefined, nonfinite numbers, functions, and built-in objects like Date, RegExp), etc.? // Todo: Handle non-JSON arguments and return values (undefined, nonfinite numbers, functions, and built-in objects like Date, RegExp), etc.?
// We accept and post strings for the sake of IE9 support // We accept and post strings for the sake of IE9 support
t.frame.contentWindow.postMessage(JSON.stringify({namespace: "svgCanvas", id: cbid, name: name, args: args}), '*'); t.frame.contentWindow.postMessage(JSON.stringify({namespace: "svgCanvas", id: cbid, name: name, args: args}), '*');