The addon is in admin review queue due to the minified code. They can take longer to review.
I am not an admin reviewer, however, your addon will be rejected. I am going to email you a few issues that would be better to fix, and then upload a new version.
Here are the above mentioned issues:
Your custom CSP allows remote-script injection which is not allowed.
Your add-on includes a JavaScript library file that doesn’t match any versions known to us. We require all add-ons to use unmodified release versions.
We accept JQuery/JQuery-UI libraries downloaded from ‘ajax.googleapis.com’, ‘jquery.com’ or ‘jqueryui.com’; and used without any modification. Minified versions are better. (file-name change does not matter)
I’m sorry, but we cannot accept modified, re-configured or customized libraries.
e.g.:
jquery.min.js
Add-on contains duplicate/hidden/unused/unnecessary files or folders. These complicate the review process and may contain sensitive information about the system you created the XPI on. Such files may also create cause for rejection. Please correct your packaging process so that these files aren’t included.
e.g.:
__MACOSX … and its content
Your add-on’s code includes instances of printing debugging information to the Console, which is generally not allowed in production add-ons. Please remove or disable such logging.
We don’t allow add-ons to use remote scripts because they can create serious security vulnerabilities. We also need to review all add-on code, and this makes it much more difficult. Please insert those scripts locally from your add-on code.
eg:
tracks.js
We do not allow the Google Analytics script to be included in the extension, you need to use a content iframe to include GA.
Further information: https://blog.mozilla.org/addons/2016/05/31/using-google-analytics-in-extensions/
Using Google Analytics in an add-on, requires disclosure in both the add-on description and the Privacy Policy.
The full source code is also provided as instructed.
It would have been great is the issues were raised faster, I hope I won’t wait one more month for the next reviewers comments.
I’m pretty sure I know what the issues will be. The problem is that now that Firefox supports webextension I just ported the code I use already in Chrome so to reduce coding time of course. The old code was thus abandonned.
But Firefox is far more restrictive compared to Chrome. So in the end I feel like I will still have to rewrite lots of things.
I’ve fixed most issues in the last submitted version. I didn’t minify this time and I removed the content-policy field in the manifest which was not really used so to reduce review times.
I’m not sure about 1), I’ll need more details from a reviewer.
if it has extra line breaks etc it will fail the hash checksum. It is best to use unmodified. The included one, fails the checksum. The new JQuery is fine in Version 1.7.9.1
That is not a cause for rejection but it should be removed.
Console logging does not normally cause a rejection but it should be controlled. I haven’t checked it all but for example:
tracks.js is there. If it is not used by the addon, it should not be included. Please note the number 3:
The unminifed code will help get the addon reviewed faster … BUT … you have to ask admin (on IRC for example) to move your addon to normal queue. Once addon is in admin queue, it will stay there unless admin manually move.
I have not checked background.js
The rest of the issues are still there and will cause a rejection.