Opened on 12/19/2018 at 01:42:59 PM
Closed on 08/29/2019 at 05:43:18 PM
Last modified on 10/08/2019 at 06:06:25 PM
#7180 closed change (rejected)
Add new style sheet only if it differs from old style sheet
Reported by: | mjethani | Assignee: | |
---|---|---|---|
Priority: | Unknown | Milestone: | |
Module: | Platform | Keywords: | closed-in-favor-of-gitlab |
Cc: | kzar | Blocked By: | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | no | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Background
In #5864 we started maintaining only one style sheet for element hiding emulation. The changes for this add the new style sheet even if it is identical to the old style sheet (and then remove the old style sheet if it differs), as explained in this comment:
// Ideally we would compare the old and new style sheets and skip this code // if they're the same, but the old style sheet can be a leftover from a // previous instance of the frame. We must add the new style sheet // regardless.
Two things have changed since:
- We no longer use style sheets for element hiding emulation (#6504)
- The changes for #7104 should already ensure that the old style sheet is deleted from the frame object when the frame's contents have changed
It should be possible now to simplify the code so that the style sheet is updated only if it has really changed.
What to change
[To be determined]
Attachments (0)
Change History (2)
comment:1 Changed on 07/29/2019 at 07:42:20 AM by jha2125
comment:2 Changed on 08/29/2019 at 05:43:18 PM by sebastian
- Keywords closed-in-favor-of-gitlab added
- Resolution set to rejected
- Status changed from new to closed
Sorry, but we switched to GitLab. If this issue is still relevant, please file it again in the new issue tracker.
spam