Fix build breakage on chrome-for-tv

chrome/chrome_common.gypi

 # Copyright (c) 2013 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': 'common',
       'type': 'static_library',
       'variables': {
@@ skipping 569 matching lines @@
             'common/extensions/api/extension_action/page_action_handler.cc',
             'common/extensions/api/spellcheck/spellcheck_handler.cc',
             'common/extensions/manifest_handlers/minimum_chrome_version_checker.cc',
             'common/extensions/manifest_handlers/nacl_modules_handler.cc',
             'common/icon_with_badge_image_source.cc',
             'common/importer/imported_bookmark_entry.cc',
             'common/importer/importer_bridge.cc',
             'common/importer/importer_data_types.cc',
             'common/importer/importer_url_row.cc',
             'common/net/url_util.cc',
-            'common/pepper_permission_util.cc',
             'common/spellcheck_common.cc',
           ],
+          'conditions': [

[jam, 2013/09/06 16:22:51]

nit: no need to have a nested conditions section here, just pull this out to be
at the same level as "os==android" above. this would also make it work correctly
if a platform other than android is disabling plugins

[kjyoun, 2013/09/09 00:13:45]

chrome-for-tv is based on the Android (meaning that OS=="android"), not a
separate platform. Rather, it's basically chrome-for-android built with a little
bit different options. (such as enable_plugins).

On 2013/09/06 16:22:51, jam wrote:

[jam, 2013/09/09 14:57:33]

Yes I understand that part, but what you're fixing is the config of
enable_plugins. since so far it's just been disabled on android, incorrectly
some files that shouldn't be listed for enable_plugins==0 get put in an
android==1 instead. 

On 2013/09/09 00:13:45, kjyoun wrote:

[kjyoun, 2013/09/10 02:21:25]

Got it and done!
On 2013/09/09 14:57:33, jam wrote:

+            ['enable_plugins!=1', {
+              'source!' : [
+                'common/pepper_permission_util.cc',
+              ],
+            }],
+          ],
           'dependencies!': [
             '<(DEPTH)/chrome/app/policy/cloud_policy_codegen.gyp:policy',
           ],
         }],
         ['OS=="win"', {
           'include_dirs': [
             '<(DEPTH)/breakpad/src',
             '<(DEPTH)/third_party/wtl/include',
           ],
         }],
@@ skipping 248 matching lines @@
         'common/metrics/proto/user_action_event.proto',
       ],
       'variables': {
         'proto_in_dir': 'common/metrics/proto',
         'proto_out_dir': 'chrome/common/metrics/proto',
       },
       'includes': [ '../build/protoc.gypi' ],
     },
   ],
 }