Declare dependency from ppapi_proxy targets on blink API

Declare dependencie from ppapi_proxy targets on blink API

websocket_resource.cc in ppapi_proxy uses blink's WebSocket.h, but does
not declare a dependency. We'd like to have all such dependencies declared
explicitly so we can export settings like include_dirs from the blink public
API to make the includes within the API more rational.

This adds a dependency on blink_headers (since this does not depend on any
symbols exported from blink) so that it can be configured correctly.

BUG=350097

[jamesr, 2014-03-17 22:05:19]

bbudge@ - please take a look, missed this target in the last round.

[bbudge, 2014-03-17 22:17:40]

LGTM please add a TODO for me to fix this.

https://codereview.chromium.org/202523003/diff/1/ppapi/ppapi_internal.gyp
File ppapi/ppapi_internal.gyp (right):

https://codereview.chromium.org/202523003/diff/1/ppapi/ppapi_internal.gyp#new...
ppapi/ppapi_internal.gyp:131:
'../third_party/WebKit/public/blink_headers.gyp:blink_headers',
Maybe a TODO(bbudge) to add a conversion fn in the appropriate place?

[jamesr, 2014-03-17 22:29:09]

On 2014/03/17 22:17:40, bbudge wrote:
> LGTM please add a TODO for me to fix this.
> 
> https://codereview.chromium.org/202523003/diff/1/ppapi/ppapi_internal.gyp
> File ppapi/ppapi_internal.gyp (right):
> 
>
https://codereview.chromium.org/202523003/diff/1/ppapi/ppapi_internal.gyp#new...
> ppapi/ppapi_internal.gyp:131:
> '../third_party/WebKit/public/blink_headers.gyp:blink_headers',
> Maybe a TODO(bbudge) to add a conversion fn in the appropriate place?

Sure - do you know where such a thing should go, or should I just have a TODO on
you to figure that out too?

[jamesr, 2014-03-17 22:40:03]

The CQ bit was checked by jamesr@chromium.org

[commit-bot: I haz the power, 2014-03-17 22:48:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jamesr@chromium.org/202523003/30001

[commit-bot: I haz the power, 2014-03-18 00:37:29]

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

[commit-bot: I haz the power, 2014-03-18 00:37:30]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_rel

[bbudge, 2014-03-18 17:24:57]

On 2014/03/17 22:29:09, jamesr wrote:
> On 2014/03/17 22:17:40, bbudge wrote:
> > LGTM please add a TODO for me to fix this.
> > 
> > https://codereview.chromium.org/202523003/diff/1/ppapi/ppapi_internal.gyp
> > File ppapi/ppapi_internal.gyp (right):
> > 
> >
>
https://codereview.chromium.org/202523003/diff/1/ppapi/ppapi_internal.gyp#new...
> > ppapi/ppapi_internal.gyp:131:
> > '../third_party/WebKit/public/blink_headers.gyp:blink_headers',
> > Maybe a TODO(bbudge) to add a conversion fn in the appropriate place?
> 
> Sure - do you know where such a thing should go, or should I just have a TODO
on
> you to figure that out too?

I have a CL to fix it (by moving the reference to the host side of the proxy,
where we already depend on Blink.)
https://codereview.chromium.org/202143007/

[jamesr, 2014-03-18 18:07:27]

That's even better. Closing.

[jamesr, 2014-03-18 18:07:28]

That's even better. Closing.