diff options
| author | apatrick@chromium.org <apatrick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-08 23:13:03 +0000 |
|---|---|---|
| committer | apatrick@chromium.org <apatrick@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-11-08 23:13:03 +0000 |
| commit | 0ab8d17f32f633ae6a386531deaa03391d3c4693 (patch) | |
| tree | 534c99e3e0d0e082ee1d27af3dc2ef923d8325b4 /chrome/test | |
| parent | 2594c2bef91a8a25e5bfb2c5685d3d9653ddd376 (diff) | |
| download | chromium_src-0ab8d17f32f633ae6a386531deaa03391d3c4693.zip chromium_src-0ab8d17f32f633ae6a386531deaa03391d3c4693.tar.gz chromium_src-0ab8d17f32f633ae6a386531deaa03391d3c4693.tar.bz2 | |
Made OSMesa work on Mac for WebGL.
This required that GPU initialization not fail if GPU info cannot be collected.
The accelerated compositor still needs some work but this will let us run some WebGL browser and ui smoke tests.
Also added --disable-accelerated-compositing for browser and ui tests so they will not attempt to use it.
TEST=try, run ui test locally
BUG=61037, 58343
Review URL: http://codereview.chromium.org/4716002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@65447 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/test')
| -rw-r--r-- | chrome/test/gpu/gpu_uitest.cc | 9 | ||||
| -rw-r--r-- | chrome/test/test_launcher_utils.cc | 7 |
2 files changed, 5 insertions, 11 deletions
diff --git a/chrome/test/gpu/gpu_uitest.cc b/chrome/test/gpu/gpu_uitest.cc index a6c43d0..c22f8f8 100644 --- a/chrome/test/gpu/gpu_uitest.cc +++ b/chrome/test/gpu/gpu_uitest.cc @@ -21,14 +21,7 @@ class GPUUITest : public UITest { FilePath gpu_test_dir_; }; -// TODO(apatrick): OSMesa is flaky on Mac. http://crbug/61037 -#if defined(OS_MACOSX) -#define MAYBE_UITestLaunchedWithOSMesa DISABLED_UITestLaunchedWithOSMesa -#else -#define MAYBE_UITestLaunchedWithOSMesa UITestLaunchedWithOSMesa -#endif - -TEST_F(GPUUITest, MAYBE_UITestLaunchedWithOSMesa) { +TEST_F(GPUUITest, UITestLaunchedWithOSMesa) { // Check the webgl test reports success and that the renderer was OSMesa. // We use OSMesa for tests in order to get consistent results across a // variety of boxes. diff --git a/chrome/test/test_launcher_utils.cc b/chrome/test/test_launcher_utils.cc index 5b93422..aaf7bb9 100644 --- a/chrome/test/test_launcher_utils.cc +++ b/chrome/test/test_launcher_utils.cc @@ -33,12 +33,13 @@ void PrepareBrowserCommandLineForTests(CommandLine* command_line) { command_line->AppendSwitch(switches::kEnableLogging); command_line->AppendSwitchASCII(switches::kLoggingLevel, "1"); // warning - // TODO(apatrick): Other pending changes will fix this for mac. -#if !defined(OS_MACOSX) // Force all tests to use OSMesa if they launch the GPU process. command_line->AppendSwitchASCII(switches::kUseGL, gfx::kGLImplementationOSMesaName); -#endif + + // Mac does not support accelerated compositing with OSMesa. + // http://crbug.com/58343 + command_line->AppendSwitch(switches::kDisableAcceleratedCompositing); } bool OverrideUserDataDir(const FilePath& user_data_dir) { |