Filed <a href="https://bugs.webkit.org/show_bug.cgi?id=75548">https://bugs.webkit.org/show_bug.cgi?id=75548</a> ('[rollup] remove chromium-cg-mac baselines')<br><br><div class="gmail_quote">On Wed, Jan 4, 2012 at 10:37 AM, Adam Barth <span dir="ltr"><<a href="mailto:abarth@webkit.org">abarth@webkit.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Wed, Jan 4, 2012 at 6:10 AM, Elliot Poger <<a href="mailto:epoger@chromium.org">epoger@chromium.org</a>> wrote:<br>


</div><div class="im">> I agree that it is reasonable (and a good idea) to remove the<br>
> chromium-cg-mac expectations from WebKit now, and I am willing to take the<br>
> lead on doing so (although I will need help from WebKit committers).<br>
<br>
</div>Sounds like a good plan.  I'm happy to help.  Let's coordinate off-list.<br>
<br>
Thanks,<br>
Adam<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> Please let me know if anyone agrees/disagrees with the following steps to do<br>
> so:<br>
><br>
> 1. remove the following buildbots that rely on chromium-cg-mac expectations<br>
> (otherwise, they will start failing once the CG expectations disappear):<br>
><br>
> <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><br>


> <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><br>


> <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><br>
> <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><br>
> <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><br>
> <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><br>


> <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><br>


> <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><br>


> <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><br>
><br>
> 2. remove the LayoutTests/platform/*-cg-* directories from the WebKit repo<br>
><br>
> 3. remove any CG-specific entries from<br>
> LayoutTests/platform/chromium/test-expectations.txt<br>
><br>
> 4. remove any CG-specific test code from the Tools/Scripts/webkitpy tree<br>
><br>
><br>
> On Wed, Jan 4, 2012 at 12:01 AM, David Levin <<a href="mailto:levin@chromium.org">levin@chromium.org</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On Tue, Jan 3, 2012 at 8:33 PM, Dirk Pranke <<a href="mailto:dpranke@chromium.org">dpranke@chromium.org</a>> wrote:<br>
>>><br>
>>> On Tue, Jan 3, 2012 at 3:28 PM, Adam Barth <<a href="mailto:abarth@webkit.org">abarth@webkit.org</a>> wrote:<br>
>>> > On Tue, Jan 3, 2012 at 3:22 PM, Nico Weber <<a href="mailto:thakis@chromium.org">thakis@chromium.org</a>> wrote:<br>
>>> >> On Tue, Jan 3, 2012 at 3:00 PM, Adam Barth <<a href="mailto:abarth@webkit.org">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>
>>> 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>
>><br>
>><br>
>> Remove it.<br>
>><br>
>> It is a cost on everyone who enlists in WebKit. Those of us who aren't<br>
>> creating new enlistments are not affected much but that doesn't mean it<br>
>> isn't costly.<br>
>><br>
>> dave<br>
>><br>
><br>
</div></div></blockquote></div><br>