>
> @@ +194,5 @@
> > +
> > +PRUint32 ToLinkMask(const nsAString& aLink, PRUint32 aLinkMask)
> > +{
> > + if (aLink.Equals(NS_LITERAL_STRING("prefetch")))
> > + return aLinkMask |= PREFETCH;
>
> Don't take aLinkMask as a parameter. Just return PREFETCH and require the
> caller to | it in if necessary. Same for the other values.
>
> This will make ToLinkMask simpler.
Since we're returning PRUint32, what will be the default return if the string doesn't match any of the specified links?

Created attachment 575325[details][diff][review]
implementation of nsStyleLinkElement::ParseLinkTypes returning a bitmask
Oops silly me. Thanks. I made the requisite changes and here's hoping that three is actually the charm.