Closed Bug 1271328 Opened 9 years ago Closed 7 years ago

Add a test to ensure NotifyLayersUpdated copies all the right fields

Categories

(Core :: Panning and Zooming, defect, P5)

49 Branch
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: kats, Unassigned)

Details

(Whiteboard: gfx-noted)

In AsyncPanZoomController::NotifyLayersUpdated we do this thing where we copy a bunch of fields from one scroll metadata object to another. We should add some gtests to make sure that all the right things are getting copied in the different branches, so that if a new field is added we don't accidentally miss copying it.
Whiteboard: gfx-noted
Priority: -- → P5
Version: Trunk → 49 Branch
I don't really understand what the intention is to do in this test. After triggering the code that's supposed to do the copying, the test would need to either check that each field was copied individually (in which case, the test doesn't serve much purpose since after adding a new field you could forget to add the check to the *test*), or it uses operator==, in which case you're relying on the user remembering to add the field to operator==.
Yeah.. I'd be ok closing this bug as invalid or wontfix.
Mentor: botond
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.