biancadanforth / tracking-protection-shield-study

A Shield study to determine the optimal messaging, if any, for Tracking Protection in Firefox.
0 stars 3 forks source link

Cleanup code a bit #38

Closed biancadanforth closed 6 years ago

biancadanforth commented 6 years ago

Tidying up the code a bit before various reviewers start looking at it.

Unfortunately, when loading resource:// modules like Config.jsm, they can't be accessed before addon startup(), so I had to remove calls to log in addon install().

Feature.jsm still needs some work in the code executed in applyExperimentalTreatment() and beyond. But the first half of it is a lot easier to follow now. Also I didn't even touch new-tab-variation.js, the frame script.