Revert 36399 - linux: enable pepper_test_plugin

This is breaking official x64 builds, notified team of break earlier in the 
week, but haven't seen a fix.  I'm reverting now.  

Please fix before re-enabling.

Original message:
This includes a way to build it with full support on x64 (and arm ?), by specifying linux_fpic=1 in GYP_DEFINES and rerunning gyp.

Review URL: http://codereview.chromium.org/551041

TBR=piman@chromium.org
Review URL: http://codereview.chromium.org/548133

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@36969 0039d316-1c4b-4281-b951-d872f2087c98

5 files changed, 11 insertions, 23 deletions

diff --git a/build/all.gyp b/build/all.gyp
index 8a0910b..03d7fee 100644
--- a/build/all.gyp
+++ b/build/all.gyp
@@ -36,7 +36,6 @@
         '../third_party/sqlite/sqlite.gyp:*',
         '../third_party/WebKit/WebKit/chromium/WebKit.gyp:*',
         '../third_party/zlib/zlib.gyp:*',
-        '../webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp:*',
         '../webkit/tools/test_shell/test_shell.gyp:*',
         '../webkit/webkit.gyp:*',
         'util/build_util.gyp:*',
@@ -56,6 +55,7 @@
         ['OS=="mac"', {
           'dependencies': [
             '../third_party/ocmock/ocmock.gyp:*',
+            '../webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp:*',
           ],
         }],
         ['OS=="linux"', {
@@ -101,6 +101,9 @@
             '../third_party/bspatch/bspatch.gyp:*',
             '../third_party/gles2_book/gles2_book.gyp:*',
             '../tools/memory_watcher/memory_watcher.gyp:*',
+            # As of now, build machines do not have GL headers to compile this
+            # TODO(sehr,brettw, neb): Make this unconditional.
+            '../webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp:*',
           ],
         }, {
           'dependencies': [
diff --git a/build/common.gypi b/build/common.gypi
index 76af9cc..819515c 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -187,8 +187,6 @@
     # Enable TCMalloc.
     'linux_use_tcmalloc%': 0,
 
-    'linux_fpic%': 0,
-
     # Set to select the Title Case versions of strings in GRD files.
     'use_titlecase_in_grd_files%': 0,
 
@@ -808,11 +806,6 @@
               }],
             ],
           }],
-          ['linux_fpic==1', {
-            'cflags': [
-              '-fPIC',
-            ],
-          }],
           ['sysroot!=""', {
             'target_conditions': [
               ['_toolset=="target"', {
diff --git a/gpu/pgl/pgl.cc b/gpu/pgl/pgl.cc
index 2fd3d8b..6f24a31 100644
--- a/gpu/pgl/pgl.cc
+++ b/gpu/pgl/pgl.cc
@@ -152,7 +152,7 @@ PGLBoolean pglDestroyContext(PGLContext pgl_context) {
   if (!pgl_context)
     return false;
 
-  delete static_cast<PGLContextImpl*>(pgl_context);
+  delete pgl_context;
   return true;
 }
 
diff --git a/webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp b/webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp
index 35ba1b0..1cdc8d3 100644
--- a/webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp
+++ b/webkit/tools/pepper_test_plugin/pepper_test_plugin.gyp
@@ -30,6 +30,10 @@
           'product_name': 'pepper_test_plugin',
           'type': 'shared_library',
           'msvs_guid': 'EE00E36E-9E8C-4DFB-925E-FBE32CEDB91A',
+          'dependencies': [
+            '../../../gpu/gpu.gyp:gles2_demo_lib',
+            '../../../gpu/gpu.gyp:pgl',
+          ],
           'sources': [
             'pepper_test_plugin.def',
             'pepper_test_plugin.rc',
@@ -45,11 +49,7 @@
             ],
           },
         }],
-        ['OS=="linux"', {
-          'type': 'shared_library',
-          'cflags': ['-fvisibility=hidden'],
-        }],
-        ['OS=="linux" and (target_arch=="x64" or target_arch=="arm") and linux_fpic!=1', {
+        ['OS=="linux" and (target_arch=="x64" or target_arch=="arm")', {
           'product_name': 'pepper_test_plugin',
           # Shared libraries need -fPIC on x86-64
           'cflags': ['-fPIC'],
@@ -59,14 +59,6 @@
             '../../../base/base.gyp:base',
             '../../../skia/skia.gyp:skia',
           ],
-          'conditions': [
-            ['OS!="mac"', {
-              'dependencies': [
-                '../../../gpu/gpu.gyp:gles2_demo_lib',
-                '../../../gpu/gpu.gyp:pgl',
-              ],
-            }],
-          ]
         }],
         ['OS=="mac"', {
           'type': 'loadable_module',
diff --git a/webkit/tools/pepper_test_plugin/plugin_object.cc b/webkit/tools/pepper_test_plugin/plugin_object.cc
index 7738a8cc..aa6422d 100644
--- a/webkit/tools/pepper_test_plugin/plugin_object.cc
+++ b/webkit/tools/pepper_test_plugin/plugin_object.cc
@@ -35,7 +35,7 @@
 #else
 #include "base/logging.h"
 #include "gpu/command_buffer/client/gles2_demo_cc.h"
-#include "gpu/command_buffer/common/GLES2/gl2.h"
+#include "gpu/command_buffer/common/gles2/gl2.h"
 #include "third_party/skia/include/core/SkBitmap.h"
 #include "third_party/skia/include/core/SkCanvas.h"
 #include "third_party/skia/include/effects/SkGradientShader.h"