ah ok.<div><br><br><div class="gmail_quote">On Sat, Aug 18, 2012 at 10:06 PM, Tim Horton <span dir="ltr">&lt;<a href="mailto:timothy_horton@apple.com" target="_blank">timothy_horton@apple.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><br><div><div class="im"><div>On Aug 18, 2012, at 8:35 PM, Jake &lt;<a href="mailto:jake@jakeonthenet.com" target="_blank">jake@jakeonthenet.com</a>&gt; wrote:</div><br><blockquote type="cite">
Just a guess but shouldn&#39;t the #if line be:<div><br></div><div><span style="color:rgb(80,0,80);font-size:13px;font-family:arial,sans-serif">#if defined(ENABLE_CSS3_TEXT_</span><span style="color:rgb(80,0,80);font-size:13px;font-family:arial,sans-serif">DECORATION) &amp;&amp; defined(ENABLE_CSS3_TEXT_DECORATION)</span><br>
</div></blockquote><div><br></div></div><div>No, this is supposed to be &quot;if ENABLE_CSS3_TEXT_DECORATION is defined and it is defined to true&quot;. The line is correct as it stands.</div><div><div class="h5"><br><blockquote type="cite">
<div>-Jake</div><div><br></div><div><div class="gmail_quote">On Sat, Aug 18, 2012 at 8:20 PM, Tim Horton <span dir="ltr">&lt;<a href="mailto:timothy_horton@apple.com" target="_blank">timothy_horton@apple.com</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><br>
<br>
On Aug 17, 2012, at 11:57 AM, Bruno Abinader &lt;<a href="mailto:brunoabinader@gmail.com" target="_blank">brunoabinader@gmail.com</a>&gt; wrote:<br>
<br>
&gt; On Fri, Aug 17, 2012 at 2:24 PM, Joe Mason &lt;<a href="mailto:jmason@rim.com" target="_blank">jmason@rim.com</a>&gt; wrote:<br>
&gt;&gt; What do you mean by &quot;precompiler&quot;?  The preprocessor?  Or is this a precompiled headers thing?<br>
&gt;<br>
&gt; Indeed, s/precompiler/preprocessor :)<br>
&gt;<br>
&gt;&gt;<br>
&gt;&gt; Are you sure there wasn&#39;t a typo in the &quot;case CSSPropertyWebkitTextDecorationLine:&quot; line which caused a perfectly normal syntax error when CSS3_TEXT_DECORATION was defined?  Or maybe CSSPropertyWebkitTextDecorationLine was not defined?<br>


&gt;<br>
&gt; There is no typo as far as I&#39;ve checked. The following was added to<br>
&gt; Source/WebCore/css/CSSPropertyNames.in:<br>
&gt;<br>
&gt; ...<br>
&gt; #if defined(ENABLE_CSS3_TEXT_DECORATION) &amp;&amp; ENABLE_CSS3_TEXT_DECORATION<br>
&gt; -webkit-text-decoration-line<br>
&gt; #endif<br>
&gt; ...<br>
&gt;<br>
&gt;&gt; I dislike this.  Repeated code should be avoided. Fallthrough is a widely used and accepted technique to do that, ever since the days of C. I can&#39;t believe there&#39;s a compiler that doesn&#39;t handle this correctly.<br>


&gt;<br>
&gt; +1 on that. I&#39;ve re-checked the build bot output and it seems the<br>
&gt; issue is actually caused because CSSPropertyWebkitTextDecorationLine<br>
&gt; gets undefined. So I&#39;m starting to believe the fail reason is<br>
&gt; something else skipping from my eyes.<br>
<br>
</div>Without looking at the code, I vaguely think that CSSPropertyWebkitTextDecorationLine comes from a generated file that might not rebuild when it&#39;s supposed to. You should try a clean build and see -- there&#39;ve been a few problems with this recently (I ran into it when switching flexbox and regions/exclusions on and off a lot a few months ago, and a colleague ran into it again recently in a similar situation).<br>


<div><div><br>
&gt; The actual bug related to this<br>
&gt; issue is <a href="https://bugs.webkit.org/show_bug.cgi?id=94108" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=94108</a> .<br>
&gt;<br>
&gt;&gt;<br>
&gt;&gt;&gt; Where each switch case is handled individually (if you&#39;re curious<br>
&gt;&gt;&gt; about it, it is fixed in bug 90493). Said this, I would like to<br>
&gt;&gt;<br>
&gt;&gt; <a href="https://bugs.webkit.org/show_bug.cgi?id=90493" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=90493</a> is &quot;[chromium] Don&#39;t archive build files generated by VS2010&quot;, and doesn&#39;t seem to be related.<br>


&gt;<br>
&gt; My bad :/ The right bug is 94093:<br>
&gt; <a href="https://bugs.webkit.org/show_bug.cgi?id=94093" target="_blank">https://bugs.webkit.org/show_bug.cgi?id=94093</a> .<br>
&gt;<br>
&gt; --<br>
&gt; Bruno de Oliveira Abinader<br>
&gt; _______________________________________________<br>
&gt; webkit-dev mailing list<br>
&gt; <a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
&gt; <a href="http://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
<br>
_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org" target="_blank">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo/webkit-dev</a><br>
</div></div></blockquote></div><br></div>
</blockquote></div></div></div><br></div></blockquote></div><br></div>