<div class="gmail_quote">2011/6/21 Hironori Bono (坊野 博典) <span dir="ltr"><<a href="mailto:hbono@chromium.org">hbono@chromium.org</a>></span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">
On Wed, Jun 22, 2011 at 1:17 PM, Simon Fraser <<a href="mailto:simon.fraser@apple.com">simon.fraser@apple.com</a>> wrote:<br>
> This new API got turn on inadvertently on Mac just now, and a few of us spent<br>
> a wasted hour trying to get things to build. In this light, my comments on the API<br>
> may not be as favorable as they would otherwise be.<br>
><br>
> First, why is the API on HTMLDivElement?<br>
<br>
</div>Unfortunately, I have just copied and pasted my code in<br>
HTMLTextAreaElement to support contenteditable elements and designMode<br>
ones because I did not have better ideas at that time. I should have<br>
asked about it in this ML before landing this change.<br></blockquote><div><br></div><div>contenteditable attribute can appear in any element so I don't think we should restrict it to div.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div class="im">
> Second, since this exposes to the web API in a non-final spec, the new methods<br>
> should be prefixed with 'webkit' to avoid potential conflict if the API changes later.<br>
<br>
</div>This is totally my fault. (The sin of ignorance.) Sorry again for that.<br>
<br>
By the way, I wonder how I should treat my r88332 even though I<br>
understand this change was wrong. Even though there are several<br>
options: 1. re-design the API, 2. revert my r88332 and re-implement<br>
it, 3. send changes that apply these comments, etc. Would it be<br>
possible to give me your advice?<br></blockquote><div><br></div><div>Given that the original patch seems to have various issues and we're going to make a lot of changes (i.e. move it to HTMLElement and prefix it with webkit or chrome), option 2 seems most suitable approach here.</div>

<div><br></div><div>- Ryosuke</div><div><br></div></div>