There's a couple of cases where we're mixing && and || in the same
expression without using parentheses.
In particular ChromeClient::contentsSizeChanged() has this:
&& (requisition.height != size.height())
|| (requisition.width != size.width()))
Due to the operator precedence rules, this means
if ((A && B) || C)
if (A && (B || C))
which is what we want.
AcceleratedCompositingContext::renderLayersToWindow() has a similar
case. If I'm not wrong here the parentheses are not really necessary
but I'd suggest using them as well to make the precedence explicit.
Created attachment 203739 [details]
Comment on attachment 203739 [details]
Clearing flags on attachment: 203739
Committed r151198: <http://trac.webkit.org/changeset/151198>
All reviewed patches have been landed. Closing bug.