hogehoge

gistfile1.md

overlay.xul

var urlkey = new Array();

var urlKey = []; の方がイマふうです。

var domain = content.location.host;

var host = な気がします。

urlkey[domain] = key;

urlkey は object {} の方が適していると思います。

if ( href.match(/^http:\/\/twitter/) ) {

(/hogehoge/).test(href); のがいいです。

delete urlkey;
delete checkurl;

var で定義した変数は delete で消せません。

overlay.js

var regex = new RegExp(pattern);

不要です。pattern のまま使えます。

return key;

if 文の中で return で抜けてしまったほうが無駄なループが減ると思います。

contentArea.addEventListener('load', mixicheckChangeState, true);

false でもいいと思います。

全体的に lowercamelcase になっている所は lowerCamelCase にしたほうが読みやすいです。