Refactor RenderThemeChromiumDefault

Source/core/core.gyp

 #
 # Copyright (C) 2009 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
@@ skipping 544 matching lines @@
           'msvs_shard': 5,
         }],
         ['use_default_render_theme==0', {
           'sources/': [
             ['exclude', 'rendering/RenderThemeChromiumDefault.*'],
           ],
         }],
         ['OS=="win"', {
           'sources/': [
             ['exclude', 'Posix\\.cpp$'],
+            ['include', 'rendering/RenderThemeChromiumDefault\\.cpp$'],

[tkent, 2014/11/21 01:22:37]

I think this is unnecessary.  core.gypi contains RenderThemeChromiumDefault.cpp
by default.

           ],
         },{ # OS!="win"
           'sources/': [
             ['exclude', 'Win\\.cpp$'],
           ],
         }],
         ['OS=="win" and chromium_win_pch==1', {
           'sources/': [
             ['include', '<(DEPTH)/third_party/WebKit/Source/build/win/Precompile.cpp'],
           ],
         }],
         ['OS=="mac"', {
           'sources/': [
             # RenderThemeChromiumSkia is not used on mac since RenderThemeChromiumMac
             # does not reference the Skia code that is used by Windows, Linux and Android.
             ['exclude', 'rendering/RenderThemeChromiumSkia\\.cpp$'],
             # RenderThemeChromiumFontProvider is used by RenderThemeChromiumSkia.
             ['exclude', 'rendering/RenderThemeChromiumFontProvider\\.cpp'],
             ['exclude', 'rendering/RenderThemeChromiumFontProvider\\.h'],
           ],
         },{ # OS!="mac"
           'sources/': [['exclude', 'Mac\\.(cpp|mm?)$']]
         }],
         ['OS == "android" and target_arch == "ia32" and gcc_version == 46', {
           # Due to a bug in gcc 4.6 in android NDK, we get warnings about uninitialized variable.
           'cflags': ['-Wno-uninitialized'],
         }],
-        ['OS != "linux"', {
+        ['OS == "linux"', {
+          'sources/': [
+            ['include', 'rendering/RenderThemeChromiumDefault\\.cpp$'],

[tkent, 2014/11/21 01:22:37]

Ditto.

+          ],
+        },{ # OS!="linux"
           'sources/': [
             ['exclude', 'Linux\\.cpp$'],
           ],
         }],
         ['OS=="android"', {
           'sources/': [
             ['include', 'rendering/RenderThemeChromiumFontProviderLinux\\.cpp$'],
             ['include', 'rendering/RenderThemeChromiumDefault\\.cpp$'],
           ],
         },{ # OS!="android"
@@ skipping 187 matching lines @@
         '<(bindings_core_v8_output_dir)/V8InternalDictionaryDerived.h',
         '<(bindings_core_v8_output_dir)/V8PluginPlaceholderOptions.cpp',
         '<(bindings_core_v8_output_dir)/V8PluginPlaceholderOptions.h',
       ],
       'sources/': [
         ['exclude', 'testing/js'],
       ],
     },
   ],  # targets
 }