Comment on attachment 101525 [details] [diff] [review] patch You'll probably have to remove the reference to the overlay from this file too: /netwerk/test/jarlist.dat sr=jag
Attachment #101525 - Flags: superreview+
Component: Browser-General → XP Apps: GUI Features
QA Contact: asa → sairuh
kaie, could you review ?
or bryner ?
Status: NEW → ASSIGNED
I'm reviewing now. Just curious, what was the motivation for merging? Less bloat?
Comment on attachment 101525 [details] [diff] [review] patch looks good r=kaie
Attachment #101525 - Flags: review+
performance, overlays are very expensive. the only question is what happens if an embedder decides not to build psm. i think the answer is that they get an always unlocked icon or make there own chrome. both of those seem like acceptable answers.
Yes, less bloat and it should Txul a little bit. Less overlays we have faster we are.
fix checked in, thanks for reviews
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
We could do something where we check for a well known security contract ID and hide the icon if the contract ID is not there.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.