Closed Bug 1374731 Opened 4 years ago Closed 4 years ago
Remove built-in Flash test tables
59 bytes, text/x-review-board-request
On bug 1307604, the test entries from itisatrap.org to test the flash blocking lists were added to the hardcoded list in SafeBrowsing.addMozEntries(), following the pattern of the other existing ones (from the malware/tracking-protection features). However, all these Flash entries are present in the actual lists served by shavar, and it's been agreed that it's not necessary to have them hardcoded by the browser. It's likely that this will represent a perf win, or at least open the way for one. When bug 1307604 landed, it caused bug 1337043. That regression was recovered by some other means, but removing this is strictly less work to be done on startup. Also, I wish to remove more stuff from this in the future, but that involves more conversations, and this is the easy first step to get those conversations started.
Priority: -- → P1
Whiteboard: [photon-performance] → [reserve-photon-performance]
I wasn't sure if it was necessary to add the test tables to disallow completions during tests, so I added them just to be sure.
Comment on attachment 8879719 [details] Bug 1374731 - Don't use hardcoded test entries for the Flash classification tables. https://reviewboard.mozilla.org/r/151060/#review155908 Looks good to me.
Attachment #8879719 - Flags: review?(ksteuber) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/100a1fdb6daa Don't use hardcoded test entries for the Flash classification tables. r=bytesized
BTW, next time we should ensure that one of the URL Classifier peers (Dimi, Henry, GCP or me) takes a look a the patch too before landing. (I just had a look through it and it looks all good to me.)
You need to log in before you can comment on or make changes to this bug.