Move ANGLEWebKitBridge to platform/graphics/angle/ANGLEPlatformBridge.

Move ANGLEWebKitBridge to platform/graphics/angle/ANGLEPlatformBridge.

Part of the effort to migrate platform out of core.

R=abarth@chromium.org, abarth, eae, senorblanco
BUG=303332
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=158988

[Stephen Chennney, 2013-10-03 02:00:44]

Not sure if this is right. Local build is broken for some reason seemingly
unrelated.

[jamesr, 2013-10-03 02:12:20]

Why do we even have this?  All interactions with ANGLE for WebGL should be
outside blink (either in the gpu process or in the client-side validation code
in gpu/...).

[abarth-chromium, 2013-10-03 03:26:35]

Maybe this is for CSS shaders?  It would be good to understand.

[Stephen Chennney, 2013-10-03 14:23:44]

On 2013/10/03 03:26:35, abarth wrote:
> Maybe this is for CSS shaders?  It would be good to understand.

It seems to only be for the custom filters work. So once Adobe make a decision
maybe we delete it.

Still, I would like to get this done even if it's temporary. It's a prereq for
moving the graphics/filters code.

[abarth-chromium, 2013-10-03 16:45:01]

LGTM

https://codereview.chromium.org/25825003/diff/1/Source/platform/blink_platfor...
File Source/platform/blink_platform.gyp (right):

https://codereview.chromium.org/25825003/diff/1/Source/platform/blink_platfor...
Source/platform/blink_platform.gyp:67:
'<(DEPTH)/third_party/angle_dx11/src/build_angle.gyp:translator_glsl',
Can you add comments about what these dependencies are used for.

https://codereview.chromium.org/25825003/diff/1/Source/platform/blink_platfor...
Source/platform/blink_platform.gyp:76:
'<(DEPTH)/third_party/angle_dx11/include',
Here too.

[jamesr, 2013-10-03 17:11:30]

I think zmo just fixed some translator_glsl dependencies. Check these with
him before landing.

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

[Zhenyao Mo, 2013-10-03 17:44:01]

On 2013/10/03 17:11:30, jamesr wrote:
> I think zmo just fixed some translator_glsl dependencies. Check these with
> him before landing.
> 
> To unsubscribe from this group and stop receiving emails from it, send an
email
> to mailto:blink-reviews+unsubscribe@chromium.org.

Thanks James.

Here is my CL: https://codereview.chromium.org/25730004/

It should land soon.  Then you can merge with ToT.  translator_glsl is
depreciated and definitely should not appear anywhere anymore.

[Stephen Chennney, 2013-10-03 19:32:39]

On 2013/10/03 17:44:01, Zhenyao Mo wrote:
> On 2013/10/03 17:11:30, jamesr wrote:
> > I think zmo just fixed some translator_glsl dependencies. Check these with
> > him before landing.
> > 
> > To unsubscribe from this group and stop receiving emails from it, send an
> email
> > to mailto:blink-reviews+unsubscribe@chromium.org.
> 
> Thanks James.
> 
> Here is my CL: https://codereview.chromium.org/25730004/
> 
> It should land soon.  Then you can merge with ToT.  translator_glsl is
> depreciated and definitely should not appear anywhere anymore.

Thanks all. I've updated to reflect this. I'll see what the commit bot can do.

[commit-bot: I haz the power, 2013-10-03 19:32:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/schenney@chromium.org/25825003/11001

[commit-bot: I haz the power, 2013-10-03 22:13:12]

Retried try job too often on linux_blink_rel for step(s) webkit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...

[Stephen Chennney, 2013-10-05 13:48:23]

Committed patchset #4 manually as r158988 (presubmit successful).