From 606211afa77c84d30aef62db7c9d4307f77b031e Mon Sep 17 00:00:00 2001 From: Simon Kornblith Date: Thu, 22 Sep 2011 06:33:27 +0000 Subject: [PATCH] Add "Dear friendly AMO reviewer" message to processNextEvent() call that is only used in Z standalone --- chrome/content/zotero/xpcom/zotero.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/chrome/content/zotero/xpcom/zotero.js b/chrome/content/zotero/xpcom/zotero.js index c89e47ce9..4955f494b 100644 --- a/chrome/content/zotero/xpcom/zotero.js +++ b/chrome/content/zotero/xpcom/zotero.js @@ -685,6 +685,11 @@ if(appInfo.platformVersion[0] >= 2) { var timeout = Date.now() + 5000; // 5 second timeout while(_waitingForDBLock && !Zotero.closing && Date.now() < timeout) { + // Dear friendly AMO reviewer: + // + // The following processNextEvent() call is only used within Zotero + // Standalone. It is never used when Zotero is running as a Firefox + // extension. Zotero.mainThread.processNextEvent(true); } if(Zotero.closing) return false; @@ -735,6 +740,7 @@ if(appInfo.platformVersion[0] >= 2) { */ function stateCheck() { if(!Zotero.isConnector && Zotero.DB.transactionInProgress()) { + Zotero.logError("State check failed due to transaction in progress"); this.initialized = false; this.skipLoading = true; return false; @@ -1501,7 +1507,7 @@ if(appInfo.platformVersion[0] >= 2) { }; /** - * Pumps a generator until it yields false. See schema.js for an example. + * Pumps a generator until it yields false. See itemTreeView.js for an example. */ this.pumpGenerator = function(generator, ms) { _waiting++;