Don't parse data twice in case of not yet initialized remote frame

(#38)
Infocatcher · Aug 21, 2015 · d29c376 · d29c376
1 parent c2a06d7
commit d29c376
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/Session_Bookmarks/sessionBookmarks.js b/Session_Bookmarks/sessionBookmarks.js
@@ -1206,7 +1206,6 @@ this.bookmarks = {
 	setTabSession: function(tab, ssData, uri, mergeHistory, disableForceLoad, _isPrivate) {
 		var data;
 		if(ssData && "JSON" in window) try {
-			data = JSON.parse(ssData);
 			// For better compatibility with Private Tab extension
 			let privateAttr = "privateTab-isPrivate";
 			let isPrivate = _isPrivate || false;
@@ -1233,6 +1232,7 @@ this.bookmarks = {
 				mm.loadFrameScript("data:application/javascript," + encodeURIComponent(data), false);
 				return;
 			}
+			data = JSON.parse(ssData);
 			if(isPrivate) {
 				if(!data.attributes)
 					data.attributes = {};