Не экономьте на переменных, это не ассемблер. Вообще самое стремное — это вот эти куски типа
v1 = v2.something();
v2 = v1.somethingElse();
Отлаживать это кошмар, модифицировать — как ходить по минному полю.
document.write не очень хорошо, т.к. блокирует страницу, да и деструктивно очень уж. Вообще эти места не очень опрятное, так и хочется создать функцию, которая принимает URL, создает iframe и вставляет его в положенное место.
Вместо кучи split и replace можно регулярками, особенно в случае с ютубом.
Не используйте $ для переменных, это не PHP (в крайнем случае можно, только если в такой переменной лежит jQuery-объект)