Remove non-standard API entry points in CanvasRenderingContext2D

Remove non-standard API entry points in CanvasRenderingContext2D

Rmoving the API entry points for the following 2d canvas methods:
setAlpha
setCompositingOperation
setLineWidth
setLineCap
setLineJoin
setMiterLimit
clearShadow
setStrokeColor
setFillColor
drawImageFromRect
setShadow

According to usage statistics, these entry points are not used anywhere
on the web. They are undocumented and redundant with existing standard
APIs.

BUG=363198, 363165, 363166, 363180, 363181, 363184, 363185, 363186, 363188, 363189, 363191
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=185835

[Justin Novosad, 2014-07-21 20:39:49]

PTAL

[Stephen White, 2014-07-22 18:00:31]

This looks ok (once the test failures are fixed), but I'd 
like Eric to take a look.

My only concern is that some enterprise sites might be 
using these APIs and aren't showing up in the UMA stats.
But if someone tells me I'm worrying for nothing, I'll go 
with that.

[Justin Novosad, 2014-07-22 18:11:12]

On 2014/07/22 18:00:31, Stephen White wrote:
> This looks ok (once the test failures are fixed), but I'd 
> like Eric to take a look.
> 
> My only concern is that some enterprise sites might be 
> using these APIs and aren't showing up in the UMA stats.
> But if someone tells me I'm worrying for nothing, I'll go 
> with that.

So the usage counters we added in M36.  All of the targeted APIs have generated
0 counts so far, which I take to mean that they can be removed without going
through deprecation.
But wait... I need to send out an intent to remove thread to blink-dev

[Justin Novosad, 2014-11-21 18:45:22]

This got all the requires LGTMs on blink-dev. Several months of usage metrics
show no usage of these APIs
I refreshed the patch.

PTAL.

[Stephen White, 2014-11-21 20:02:32]

https://codereview.chromium.org/408813007/diff/80001/Source/bindings/core/v8/...
File Source/bindings/core/v8/custom/V8CanvasRenderingContext2DCustom.cpp
(right):

https://codereview.chromium.org/408813007/diff/80001/Source/bindings/core/v8/...
Source/bindings/core/v8/custom/V8CanvasRenderingContext2DCustom.cpp:81:
impl->setStrokeStyle(CanvasStyle::createFromString(colorString));
Intentional change?

[Justin Novosad, 2014-11-21 20:17:02]

https://codereview.chromium.org/408813007/diff/80001/Source/bindings/core/v8/...
File Source/bindings/core/v8/custom/V8CanvasRenderingContext2DCustom.cpp
(right):

https://codereview.chromium.org/408813007/diff/80001/Source/bindings/core/v8/...
Source/bindings/core/v8/custom/V8CanvasRenderingContext2DCustom.cpp:81:
impl->setStrokeStyle(CanvasStyle::createFromString(colorString));
On 2014/11/21 20:02:32, Stephen White wrote:
> Intentional change?

I double checked, and yes this is intentional. The setStrokeColor method was an
API binding entry point that was removed.

[Justin Novosad, 2014-11-21 21:06:31]

New patch.

Fixed failure on mac caused by unnecessary platform-specific baseline overloads.

[Stephen White, 2014-11-21 21:21:00]

LGTM

[Justin Novosad, 2014-11-21 22:30:48]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2014-11-21 22:31:44]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/408813007/100001

[commit-bot: I haz the power, 2014-11-21 22:38:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-11-21 22:38:13]

Try jobs failed on following builders:
  blink_presubmit on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/blink_presubmit/builds/2...)

[Justin Novosad, 2014-11-21 23:55:14]

junov@chromium.org changed reviewers:
+ haraken@chromium.org
- eseidel@chromium.org

[Justin Novosad, 2014-11-21 23:55:15]

Looks like I was missing a Source/bindings OWNER

+haraken

[haraken, 2014-11-22 00:57:30]

bindings LGTM

[Justin Novosad, 2014-11-22 02:52:56]

The CQ bit was checked by junov@chromium.org

[commit-bot: I haz the power, 2014-11-22 02:53:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/408813007/100001

[commit-bot: I haz the power, 2014-11-22 02:56:15]

Committed patchset #6 (id:100001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=185835