1291280 - Declare nsStyleImageLayers::Layer::mSourceURI as FragmentOrURI

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[u459114]

A follow-up of bug 652991. Keep this url valid after window.history.pushState

Comment 1

[u459114]

Attachment: Bug 1291280 - Part 1. Move the definition of FragmentOrURL up.

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

Comment 2

[u459114]

Attachment: Bug 1291280 - Part 2. Declare nsStyleImageLayers::Layer::mSourceURI as FragmentOrURI

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

Comment 3

[u459114]

Attachment: Bug 1291280 - Part 3. Create mask local-ref test cases.

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

Comment 4

[u459114]

hi heycam,
This is a simple follow up of bug 652991 - replace type of mask uri from nsIURI to FragementOrURL.

Comment 5

[Cameron McCormack (:heycam)]

Comment on attachment 8778497 [details]
Bug 1291280 - Part 1. Move the definition of FragmentOrURL up.

https://reviewboard.mozilla.org/r/69796/#review67016

Comment 6

[Cameron McCormack (:heycam)]

https://reviewboard.mozilla.org/r/69798/#review67018

::: layout/style/nsRuleNode.cpp:6660
(Diff revision 1)
>                    aConditions);
>    }
>  };
>  
>  template <>
> -struct BackgroundItemComputer<nsCSSValueList, nsCOMPtr<nsIURI> >
> +struct BackgroundItemComputer<nsCSSValueList, FragmentOrURL >

Nit: while touching this line, you can remove the space before the ">", which isn't needed these days.

::: layout/svg/nsSVGEffects.cpp:393
(Diff revision 1)
>      frame->GetContent()->AsElement(), nsRestyleHint(0), changeHint);
>  }
>  
>  NS_IMPL_ISUPPORTS(nsSVGMaskProperty, nsISupports)
>  
> +static already_AddRefed<nsIURI> GetMaskURI(nsIFrame* aFrame, uint32_t aIndex);

All the other GetXXXURI functions that were added are static methods on nsSVGEffects, so we should do the same to be consistent.

Comment 7

[Cameron McCormack (:heycam)]

Comment on attachment 8778498 [details]
Bug 1291280 - Part 2. Declare nsStyleImageLayers::Layer::mSourceURI as FragmentOrURI

https://reviewboard.mozilla.org/r/69798/#review67020

Comment 8

[Cameron McCormack (:heycam)]

Comment on attachment 8778542 [details]
Bug 1291280 - Part 3. Create mask local-ref test cases.

https://reviewboard.mozilla.org/r/69814/#review67022

::: layout/reftests/bugs/reftest.list:1970
(Diff revision 1)
>  
>  HTTP == 652991-1a.html 652991-1-ref.html
>  HTTP == 652991-1b.html 652991-1-ref.html
>  HTTP == 652991-2.html 652991-2-ref.html
>  HTTP == 652991-3.html 652991-3-ref.html
> +HTTP == 652991-3.html 652991-3-ref.html

Remove this duplicated line.

Comment 9

[u459114]

Comment on attachment 8778497 [details]
Bug 1291280 - Part 1. Move the definition of FragmentOrURL up.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69796/diff/1-2/

Comment 10

[u459114]

Comment on attachment 8778498 [details]
Bug 1291280 - Part 2. Declare nsStyleImageLayers::Layer::mSourceURI as FragmentOrURI

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69798/diff/1-2/

Comment 11

[u459114]

Comment on attachment 8778542 [details]
Bug 1291280 - Part 3. Create mask local-ref test cases.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69814/diff/1-2/

Comment 12

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a854669c6103
Part 1. Move the definition of FragmentOrURL up. r=heycam
https://hg.mozilla.org/integration/autoland/rev/b2d3fed9052a
Part 2. Declare nsStyleImageLayers::Layer::mSourceURI as FragmentOrURI r=heycam
https://hg.mozilla.org/integration/autoland/rev/2f4de63ff19e
Part 3. Create mask local-ref test cases. r=heycam

Comment 13

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/a854669c6103
https://hg.mozilla.org/mozilla-central/rev/b2d3fed9052a
https://hg.mozilla.org/mozilla-central/rev/2f4de63ff19e

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9bd1de57490e
Part 4. Fix compile error after enable-mask-as-shorthand. r=me

Comment 15

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/9bd1de57490e