<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Mar 9, 2013 at 8:55 PM, Maciej Stachowiak <span dir="ltr">&lt;<a href="mailto:mjs@apple.com" target="_blank">mjs@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 Mar 9, 2013, at 8:00 PM, Adam Barth &lt;<a href="mailto:abarth@webkit.org" target="_blank">abarth@webkit.org</a>&gt; wrote:</div>

<br></div><blockquote type="cite"><div style="letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div dir="ltr"><div class="im">On Sat, Mar 9, 2013 at 5:02 PM, Maciej Stachowiak<span> </span><span dir="ltr">&lt;<a href="mailto:mjs@apple.com" target="_blank">mjs@apple.com</a>&gt;</span><span> </span>wrote:<br>

</div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word">

<br><div class="im"><div><div>My recommendation would be:</div><div><br></div><div>* Do one of the following two options:</div><div>   <span> </span>-  Plan A: Rename -webkit-dashboard-region to -apple-dashboard-region in CSSPropertyNames.in (it is only compiled in with ENABLE_DASHBOARD_SUPPORT anyway).</div>

<div>   <span> </span>-  Plan B: Support -apple- as a prefix alias only for -webkit-dashboard-region (it is only compiled in with ENABLE_DASHBOARD_SUPPORT anyway).</div><div>* Remove all other support for the -apple- prefix as a synonym for -webkit-</div>

<div>* Make -khtml- prefix alias support only available if ENABLE_DASHBOARD_SUPPORT is on, and also runtime-conditional based on usesDashboardBackwardCompatibilityMode having ever been set for any Settings object.</div><div>

<br></div><div><br></div><div>Depending on whether Plan A or Plan B is preferred, I can file a bug to replace -apple-dashboard-region with -webkit-dashboard-region in the preinstalled Dashboard widgets.</div></div></div>
</div>
</blockquote><div class="im"><div><br></div><div>That sounds like a good plan.  Given that Plans A and B differ only builds shipped by Apple, I&#39;ll leave that choice up to you.  I&#39;m happy to write a patch that executes the parts of this plan that involve the WebKit codebase.</div>

</div></div></div></div></div></blockquote><br></div><div>I very slightly prefer Plan A. The main downside is having a non-webkit prefix in the code with no migration path off of it. But it&#39;s a property that is not applicable to anything but Dashboard as designed[*]. So maybe lack of migration path is OK.</div>

<div><br></div><div>I wasn&#39;t sure if anyone else would object to semi-permanently having code for a -apple- property, which is why I listed the alternative, which does have a migration path.</div></div></blockquote><div>

<br></div><div style>That doesn&#39;t seem like a big burden for the project, especially if it&#39;s just one property and not exposed to the web.  The real risk here is web sites becoming dependent on these prefixes to handle quirks between CoreAnimation and the Chromium Compositor.</div>

<div style><br></div><div style>Adam</div><div style><br></div></div></div></div>