Remove canvas.drawCustomFocusRing()

Remove canvas.drawCustomFocusRing()

The method was removed from the HTML Canvas2D Context Level 1 Specification:
  http://www.w3.org/html/wg/drafts/2dcontext/html5_canvas_CR#removals

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=175036

[Justin Novosad, 2014-05-27 21:29:28]

Is drawCustomFocusRing slated to be removed from the spec?

[Rik, 2014-05-27 23:13:47]

On 2014/05/27 21:29:28, junov wrote:
> Is drawCustomFocusRing slated to be removed from the spec?

It's unclear what will happen with this API as its use is not clearly defined.
If we will keep it, it will certainly be under a different name.

[dmazzoni, 2014-05-28 15:27:18]

On Tue, May 27, 2014 at 4:13 PM, <cabanier@adobe.com> wrote:

> On 2014/05/27 21:29:28, junov wrote:
>
>> Is drawCustomFocusRing slated to be removed from the spec?
>>
>
> It's unclear what will happen with this API as its use is not clearly
> defined.
> If we will keep it, it will certainly be under a different name.
>

Agreed, the name no longer makes sense since drawSystemFocusRing was
changed to drawFocusIfNeeded.

Personally I don't think drawCustomFocusRing is needed, under any name.
It's confusing and it's unlikely any developers will implement it
correctly. Once we implement hit testing, we'll be able to support
essentially all of the accessibility features that drawCustomFocusRing was
supposed to provide.

Let's delete it.

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[eseidel, 2014-05-28 22:10:55]

rslgtm.

[zino, 2014-05-29 03:56:41]

On 2014/05/28 22:10:55, eseidel wrote:
> rslgtm.

Dear all,

So, do we all agree to remove the method?
If so, I can just click the commit chk box.

Thank you :)

[dmazzoni, 2014-05-29 04:12:17]

lgtm

I think we're all in agreement, let's get rid of it. Thanks!

[commit-bot: I haz the power, 2014-05-29 04:12:33]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/jinho.bang@samsung.com/289283019/20001

[commit-bot: I haz the power, 2014-05-29 05:31:35]

Change committed as 175036