benmmurphy / jsdeminifier_xpi

MIT License
52 stars 16 forks source link

When JSD plugin is "on" it's destroying page source variables #9

Closed KevinFaurholt closed 1 year ago

KevinFaurholt commented 12 years ago

Hello Ben,

Thanks for the js deminifier. it is truly a great contribution to the open source community and firefox! :)

However, I just discovered that when having your extension installed and "activated" it breaks the page.

Quick example. If you do the following

  1. install chris pedricks javascript developer plugin (if you have not already)
  2. enter: about:config in address bar
  3. find javascript.options.strict and set to true + restart
  4. goto http://www.volvocars.com/uk/all-cars/volvo-c70/tools/Pages/offers.aspx
  5. Open Firebug
  6. Look at the Net Tab and find outgoing image request to tribalfusion. if you keep reloading it keeps firing
  7. then activate jsdeminifier
  8. reload the page
  9. Tribalfusion image requests stops working.
  10. Error is thrown in error console on the page.
  11. turn off jsdeminifier and everything works again.

Thanks for looking into.

Cheers, Kevin

benmmurphy commented 11 years ago

thanks for the bug report

unfortunately, the algorithm jsdeminifier uses is not 100% robust. it is very fast but not 100% accurate. previously I had looked at using uglifyjs to do the beautifying but it would run out of stack space. i'll have a look at it again because uglifyjs may have been improved.

KevinFaurholt commented 1 year ago

Closing

KevinFaurholt commented 1 year ago

close #9