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:

  1. We no longer use style sheets for element hiding emulation (#6504)
  2. 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

spam

Last edited on 10/08/2019 at 06:06:25 PM by kzar

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.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from (none).
 
Note: See TracTickets for help on using tickets.