Chromecast: adds CastNetworkDelegate.

chromecast/chromecast.gyp

Index: chromecast/chromecast.gyp
diff --git a/chromecast/chromecast.gyp b/chromecast/chromecast.gyp
index 07e9d7df174b21947dd7f0d7fb8a3e20539e4dbd..c023a3bf0ac9de64d10faadeac8fe57eb79abee8 100644
--- a/chromecast/chromecast.gyp
+++ b/chromecast/chromecast.gyp
@@ -102,9 +102,12 @@
       'target_name': 'cast_service',
       'type': '<(component)',
       'dependencies': [
+        'cast_common',
         '../skia/skia.gyp:skia',
       ],
       'sources': [
+        'service/cast_network_delegate.cc',
+        'service/cast_network_delegate.h',

[lcwu1, 2014/09/25 18:53:18]

I think it makes more sense for the network delegate files to be in either
chromecast/shell/browser (like what chrome or content shell dose) or maybe in
chromecast/net (as the base class in in net/base). And they should be in
cast_shell_common target.

[gunsch, 2014/09/25 19:24:59]

I disagree. See my internal CL. ChromecastService depends on
CastNetworkDelegate, which it's a layering violation for us. This was already a
layering violation before I merged it internally and moved it from the
cast_shell and cast_shell_internal targets to
cast_service/cast_service_internal.

[lcwu1, 2014/09/25 20:07:04]

ChromecastService should not depend on CastNetworkDelegate. The network delegate
should be used (called) inside content's network requests. If Cast service has
to depend on network delegate, then there is something wrong that we need to
fix.

Basically I don't see network delegate as part of the cast service. It should be
part of the content embedder.

[gunsch, 2014/09/25 20:46:34]

Done.

         'service/cast_service.cc',
         'service/cast_service.h',
       ],
@@ -118,6 +121,9 @@
             '../base/base.gyp:base',
             '../content/content.gyp:content',
           ],
+          'sources': [
+            'service/cast_network_delegate_simple.cc',
+          ],
           'conditions': [
             ['OS=="android"', {
               'sources': [