Closed Bug 1159742 Opened 5 years ago Closed 5 years ago
Eliminate the pref annotation from test
This annotation is broken by design. The point of these tests is to test that we're exposing what we mean to be exposing. Right now any time the pref annotation is used they're just testing that the pref does in fact control the exposure... but not testing that the pref value is set correctly. Seems to me like we should be using platform+channel annotations to describe the thing we actually care about (i.e. where the API is exposed) and not worry about the mechanism (prefs, test functions, whatever).
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
You need to log in before you can comment on or make changes to this bug.