Refactor: Move app/surface to ui/gfx/surface.

content/content_renderer.gypi

 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'targets': [
     {
       'target_name': 'content_renderer',
       'msvs_guid': '9AAA8CF2-9B3D-4895-8CB9-D70BBD125EAD',
       'type': '<(library)',
       'dependencies': [
         'content_common',
         '../ppapi/ppapi.gyp:ppapi_proxy',
         '../skia/skia.gyp:skia',
         '../third_party/ffmpeg/ffmpeg.gyp:ffmpeg',
         '../third_party/icu/icu.gyp:icuuc',
         '../third_party/libjingle/libjingle.gyp:libjingle',
         '../third_party/npapi/npapi.gyp:npapi',
         '../third_party/WebKit/Source/WebKit/chromium/WebKit.gyp:webkit',
+        '../ui/gfx/surface/surface.gyp:surface',

[Ben Goodger (Google), 2011/03/23 18:41:10]

Hrm. I don't think content depended on app before. How did this work?

[jonathan.backer, 2011/03/24 13:02:07]

render_widget.cc (part of content_renderer) needs TransportDIB (which is part of
surface).  Similar things are true of chrome_browser and chrome_renderer. I did
a try without the explicit dependency on surface in these three targets, and
everything was OK (I'm confused about this).  However, I think that we should
make the dependency explicit because we are direct consumers.

Let me know if you want me to change it.

       ],
       'include_dirs': [
         '..',
       ],
       'sources': [
         'renderer/active_notification_tracker.cc',
         'renderer/active_notification_tracker.h',
         'renderer/audio_device.cc',
         'renderer/audio_device.h',
         'renderer/audio_message_filter.cc',
@@ skipping 123 matching lines @@
           'link_settings': {
             'mac_bundle_resources': [
               'renderer/renderer.sb',
             ],
           },
         }],
       ],
     },
   ],
 }