Add new gpu driver bug workaround DISABLE_TRANSPARENT_VISUALS

extensions/browser/api/app_window/app_window_apitest.cc

 // Copyright (c) 2012 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.
 
 #include "base/macros.h"
 #include "base/run_loop.h"
 #include "base/strings/string_number_conversions.h"
 #include "build/build_config.h"
 #include "chrome/browser/apps/app_browsertest_util.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/extensions/features/feature_channel.h"
 #include "chrome/test/base/testing_profile.h"
 #include "components/version_info/version_info.h"
 #include "extensions/browser/app_window/app_window.h"
 #include "extensions/browser/app_window/app_window_registry.h"
 #include "extensions/browser/app_window/native_app_window.h"
 #include "extensions/test/extension_test_message_listener.h"
 #include "ui/base/base_window.h"
 #include "ui/gfx/geometry/rect.h"
 
 #if defined(OS_WIN)
 #include "ui/base/win/shell.h"
 #endif
 
+#if defined(USE_X11) && !defined(OS_CHROMEOS)
+#include "ui/gfx/x/x11_switches.h"
+#endif
+
 namespace extensions {
 
 namespace {
 
 class TestAppWindowRegistryObserver : public AppWindowRegistry::Observer {
  public:
   explicit TestAppWindowRegistryObserver(Profile* profile)
       : profile_(profile), icon_updates_(0) {
     AppWindowRegistry::Get(profile_)->AddObserver(this);
   }
@@ skipping 87 matching lines @@
 }
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest, WindowsApiSetShapeNoPerm) {
   EXPECT_TRUE(
       RunPlatformAppTest("platform_apps/windows_api_shape/no_permission"))
       << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest,
                        WindowsApiAlphaEnabledHasPermissions) {
   const char* no_alpha_dir =

[tapted, 2016/07/14 23:37:17]

these should be 

const char kNoAlphaDir[] = ..

(or possibly `const char* const`, but currently they are non-const variables,
which isn't right).

[Julien Isorce Samsung, 2016/07/15 06:56:47]

Done.

       "platform_apps/windows_api_alpha_enabled/has_permissions_no_alpha";
-  const char* test_dir = no_alpha_dir;
+  const char* has_alpha_dir =
+      "platform_apps/windows_api_alpha_enabled/has_permissions_has_alpha";
+  const char* test_dir = has_alpha_dir;
+  test_dir = no_alpha_dir;

[tapted, 2016/07/14 23:37:17]

It looks weird to initialize it with has_alpha_dir and immediately replace it
with no_alpha_dir - I'm guessing you did this to get around an unused variable
warning. The less confusing way to do that is

ALLOW_UNUSED_LOCAL(kHasAlphaDir);
const char* test_dir = kNoAlphaDir;

(ALLOW_UNUSED_LOCAL is in base/compiler_specific.h)

[Julien Isorce Samsung, 2016/07/15 06:56:47]

Done.

 
 #if defined(USE_AURA) && (defined(OS_CHROMEOS) || !defined(OS_LINUX))
-  test_dir =
-      "platform_apps/windows_api_alpha_enabled/has_permissions_has_alpha";
+  test_dir = has_alpha_dir;
+
 #if defined(OS_WIN)
   if (!ui::win::IsAeroGlassEnabled()) {
     test_dir = no_alpha_dir;
   }
 #endif  // OS_WIN
 #endif  // USE_AURA && (OS_CHROMEOS || !OS_LINUX)
 
+#if defined(USE_X11) && !defined(OS_CHROMEOS)
+  if (base::CommandLine::ForCurrentProcess()->HasSwitch(
+          switches::kWindowDepth)) {

[tapted, 2016/07/14 23:37:17]

nit: This outer `if` isn't needed. I'd just write

  if (base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
          switches::kWindowDepth) == "32")
    test_dir = kHasAlphaDir

[Julien Isorce Samsung, 2016/07/15 06:56:47]

Done.

+    std::string depth =
+        base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+            switches::kWindowDepth);
+
+    if (depth == "32")
+      test_dir = has_alpha_dir;
+  }
+#endif  // USE_X11 && !OS_CHROMEOS
+
   EXPECT_TRUE(RunPlatformAppTest(test_dir)) << message_;
 }
 
 IN_PROC_BROWSER_TEST_F(PlatformAppBrowserTest,
                        WindowsApiAlphaEnabledNoPermissions) {
   EXPECT_TRUE(RunPlatformAppTest(
       "platform_apps/windows_api_alpha_enabled/no_permissions"))
       << message_;
 }
 
@@ skipping 53 matching lines @@
   command_line->AppendSwitchASCII(switches::kAppId,
                                   "jkghodnilhceideoidjikpgommlajknk");
 
   EXPECT_TRUE(RunComponentExtensionTest(
       "platform_apps/windows_api_ime/forced_app_mode_not_fullscreen"))
       << message_;
 }
 #endif  // OS_CHROMEOS
 
 }  // namespace extensions