1386986 - Move nsDisplaySVGEffect::mEffectsBounds to nsDisplayFilter

Status: RESOLVED FIXED

(Core :: Web Painting, enhancement)

Description

[u459114]

This is nsDisplayFilter only value(nsDisplayMask does not need it)

Comment 1

[u459114]

Attachment: Bug 1386986 - Move nsDisplaySVGEffect::mEffectsBounds to nsDisplayFilter.

Put mEffectsBounds in nsDisplyaSVGEffect does not really make sense, since only
nsDisplayFilter need it.

Review commit: https://reviewboard.mozilla.org/r/164326/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/164326/

Comment 2

[Markus Stange [:mstange]]

Comment on attachment 8893275 [details]
Bug 1386986 - Move nsDisplaySVGEffect::mEffectsBounds to nsDisplayFilter.

https://reviewboard.mozilla.org/r/164326/#review170116

Comment 3

[Mozilla Autoland]

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s ed950f0a0b11 -d edbd09a070b0: rebasing 411536:ed950f0a0b11 "Bug 1386986 - Move nsDisplaySVGEffect::mEffectsBounds to nsDisplayFilter. r=mstange" (tip)
merging layout/painting/nsDisplayList.cpp
merging layout/painting/nsDisplayList.h
warning: conflicts while merging layout/painting/nsDisplayList.h! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/19acac7630a7
Move nsDisplaySVGEffect::mEffectsBounds to nsDisplayFilter. r=mstange

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/19acac7630a7