<span style>5. Moving forward, reviewers and committers take care not to re-add expectations when committing pending patches. While I expect svn will warn in most cases, I doubt that it will catch everything.</span><div style>
<br></div><div style>6. "Won't fix" any bugs related to chromium-cg results.</div><div style><br>Stephen.</div><br><div class="gmail_quote">On Wed, Jan 4, 2012 at 9:10 AM, Elliot Poger <span dir="ltr"><<a href="mailto:epoger@chromium.org">epoger@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I agree that it is reasonable (and a good idea) to remove the chromium-cg-mac expectations from WebKit now, and I am willing to take the lead on doing so (although I will need help from WebKit committers).<div>
<br></div><div>

Please let me know if anyone agrees/disagrees with the following steps to do so:</div><div><br></div><div>1. remove the following buildbots that rely on chromium-cg-mac expectations (otherwise, they will start failing once the CG expectations disappear):</div>


<div><ul><li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder%20%28CG%29%28deps%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder%20%28CG%29%28deps%29</a></li>


<li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20%28CG%29%28deps%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20%28CG%29%28deps%29</a></li>
<li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder%20%28CG%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder%20%28CG%29</a></li>

<li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.5%20%28CG%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.5%20%28CG%29</a></li><li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20%28CG%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20%28CG%29</a></li>


<li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder%20%28CG%29%28dbg%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac%20Builder%20%28CG%29%28dbg%29</a></li>
<li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.5%20%28CG%29%28dbg%29%281%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.5%20%28CG%29%28dbg%29%281%29</a></li>


<li><a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.5%20%28CG%29%28dbg%29%282%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.5%20%28CG%29%28dbg%29%282%29</a></li>
<li>
<a href="http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20%28CG%29%28dbg%29" target="_blank">http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Mac10.6%20%28CG%29%28dbg%29</a></li>
</ul></div><div>2. remove the LayoutTests/platform/*-cg-* directories from the WebKit repo</div><div><br></div><div>3. remove any CG-specific entries from LayoutTests/platform/chromium/test-expectations.txt</div><div><br>


</div><div>4. remove any CG-specific test code from the Tools/Scripts/webkitpy tree</div><div class="HOEnZb"><div class="h5"><div><br></div><div><br><div class="gmail_quote">On Wed, Jan 4, 2012 at 12:01 AM, David Levin <span dir="ltr"><<a href="mailto:levin@chromium.org" target="_blank">levin@chromium.org</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br><br><div class="gmail_quote"><div><div>On Tue, Jan 3, 2012 at 8:33 PM, Dirk Pranke <span dir="ltr"><<a href="mailto:dpranke@chromium.org" target="_blank">dpranke@chromium.org</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>On Tue, Jan 3, 2012 at 3:28 PM, Adam Barth <<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>> wrote:<br>
> On Tue, Jan 3, 2012 at 3:22 PM, Nico Weber <<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>> wrote:<br>
>> On Tue, Jan 3, 2012 at 3:00 PM, Adam Barth <<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>> wrote:<br>
>>> It looks like Chromium Mac has successfully moved to Skia.<br>
>><br>
>> I'd wait with this assessment until a version of Chrome with Skia has<br>
>> shipped to stable. Things are looking really good so that should be<br>
>> smooth sailing, but it's a bit early to say we're "successfully moved"<br>
>> :-)<br>
><br>
> Fair enough.  However, I believe the Skia transition plan called for<br>
> removing the chromium-cg-mac expectation much earlier than a Skia<br>
> build shipping to stable.  Originally, we were only supposed to have<br>
> to maintain both sets of expectations for about a month.  The<br>
> transition has taken longer than expected, but it seems like we have<br>
> sufficient confidence in Skia now that we can remove the<br>
> chromium-cg-mac expectations.<br>
><br>
<br>
</div>Has the skia transition hit beta yet? It seems like as soon as we get<br>
it onto a version that is pointing to a branched version of webkit, we<br>
should be completely safe to remove the directories on trunk (frankly,<br>
I'd agree with Adam that it's probably safe to remove it now, since we<br>
can always add them back in if we have to, but I can compromise as<br>
well).<br></blockquote><div><br></div></div></div><div>Remove it.</div><div><br></div><div>It is a cost on everyone who enlists in WebKit. Those of us who aren't creating new enlistments are not affected much but that doesn't mean it isn't costly.</div>




<div><br></div><div>dave</div><div><br></div></div>
</blockquote></div><br></div>
</div></div><br>_______________________________________________<br>
webkit-dev mailing list<br>
<a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br>
<a href="http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev" target="_blank">http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev</a><br>
<br></blockquote></div><br>