1585858 - Remove nsChangeHint_SyncFrameViews

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Comment 1

[Emilio Cobos Álvarez [:emilio]]

Attachment: Bug 1585858 - Remove nsChangeHint_SyncFrameView.

It's always used along with nsChangeHint_RepaintFrame, which does most of the
work.

It's only useful to skip calling SyncFrameViewProperties(). That call is really
cheap if nothing actually changed, furthermore since only a handful of frames
actually have views.

So it doesn't seem like it serves any useful purpose.

Comment 2

[Emilio Cobos Álvarez [:emilio]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=92e76c7aebcbf53274a59bd50bbe3aace15a212e (ignore gtest failures from another patch)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7c80af009dee
Remove nsChangeHint_SyncFrameView. r=tnikkel

Comment 4

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/7c80af009dee