diff options
author | ccameron@chromium.org <ccameron@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-11-20 21:08:08 +0000 |
---|---|---|
committer | ccameron@chromium.org <ccameron@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-11-20 21:08:08 +0000 |
commit | eb4317e03d6201f00de7d687783b5aea110a7923 (patch) | |
tree | df8331cd0dd4daf33dda159260ccdb675469c226 | |
parent | 2ca5087652824db1772f38c451391082024d2df8 (diff) | |
download | chromium_src-eb4317e03d6201f00de7d687783b5aea110a7923.zip chromium_src-eb4317e03d6201f00de7d687783b5aea110a7923.tar.gz chromium_src-eb4317e03d6201f00de7d687783b5aea110a7923.tar.bz2 |
Remove blacklisting of Mac OS 10.6 and 10.7.
Add blacklisting of VMWare driver on 10.6 and 10.7 because it hangs.
Disable compositing expectation test on 10.6 and 10.7 because it will
give different results depending on whether it is running on
a VM or real hardware.
Expand disabling of PhishingClassifierTest because it
flakes on all Mac versions now.
TBR=vangelis
BUG=266186
Review URL: https://codereview.chromium.org/66153005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@236289 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | chrome/renderer/safe_browsing/phishing_classifier_browsertest.cc | 27 | ||||
-rw-r--r-- | content/browser/gpu/compositor_util_browsertest.cc | 9 | ||||
-rw-r--r-- | gpu/config/software_rendering_list_json.cc | 31 |
3 files changed, 37 insertions, 30 deletions
diff --git a/chrome/renderer/safe_browsing/phishing_classifier_browsertest.cc b/chrome/renderer/safe_browsing/phishing_classifier_browsertest.cc index c9c689b..2a44c6e 100644 --- a/chrome/renderer/safe_browsing/phishing_classifier_browsertest.cc +++ b/chrome/renderer/safe_browsing/phishing_classifier_browsertest.cc @@ -28,10 +28,6 @@ #include "testing/gmock/include/gmock/gmock.h" #include "url/gurl.h" -#if defined(OS_MACOSX) -#include "base/mac/mac_util.h" -#endif - using ::testing::AllOf; using ::testing::Contains; using ::testing::Not; @@ -201,8 +197,14 @@ class PhishingClassifierTest : public InProcessBrowserTest { const std::string page_term_login_; }; -// Flaky on Mac Blink bots: crbug.com/316709 -IN_PROC_BROWSER_TEST_F(PhishingClassifierTest, TestClassification) { +// This test flakes on Mac with force compositing mode. +// http://crbug.com/316709 +#if defined(OS_MACOSX) +#define MAYBE_TestClassification DISABLED_TestClassification +#else +#define MAYBE_TestClassification TestClassification +#endif +IN_PROC_BROWSER_TEST_F(PhishingClassifierTest, MAYBE_TestClassification) { host_resolver()->AddRule("*", "127.0.0.1"); // No scorer yet, so the classifier is not ready. @@ -272,15 +274,14 @@ IN_PROC_BROWSER_TEST_F(PhishingClassifierTest, TestClassification) { EXPECT_EQ(PhishingClassifier::kInvalidScore, phishy_score); } -IN_PROC_BROWSER_TEST_F(PhishingClassifierTest, DisableDetection) { +// This test flakes on Mac with force compositing mode. +// http://crbug.com/316709 #if defined(OS_MACOSX) - if (base::mac::IsOSMountainLionOrLater()) { - // This test flakes on 10.8 only for an unknown reason. - // http://crbug.com/316709 - return; - } +#define MAYBE_DisableDetection DISABLED_DisableDetection +#else +#define MAYBE_DisableDetection DisableDetection #endif - +IN_PROC_BROWSER_TEST_F(PhishingClassifierTest, MAYBE_DisableDetection) { // No scorer yet, so the classifier is not ready. EXPECT_FALSE(classifier_->is_ready()); diff --git a/content/browser/gpu/compositor_util_browsertest.cc b/content/browser/gpu/compositor_util_browsertest.cc index 3bb8eea..ac39dac 100644 --- a/content/browser/gpu/compositor_util_browsertest.cc +++ b/content/browser/gpu/compositor_util_browsertest.cc @@ -29,8 +29,13 @@ IN_PROC_BROWSER_TEST_F(CompositorUtilTest, CompositingModeAsExpected) { #elif defined(OS_ANDROID) expected_mode = THREADED; #elif defined(OS_MACOSX) - if (base::mac::IsOSMountainLionOrLater()) - expected_mode = THREADED; + expected_mode = THREADED; + // Lion and SnowLeopard have compositing blacklisted when using the Apple + // software renderer, so results will vary depending if this test is being + // run in a VM versus actual hardware. + // http://crbug.com/230931 + if (base::mac::IsOSLionOrEarlier()) + return; #elif defined(OS_WIN) if (base::win::GetVersion() >= base::win::VERSION_VISTA) expected_mode = THREADED; diff --git a/gpu/config/software_rendering_list_json.cc b/gpu/config/software_rendering_list_json.cc index ae3f014..c583714 100644 --- a/gpu/config/software_rendering_list_json.cc +++ b/gpu/config/software_rendering_list_json.cc @@ -773,21 +773,6 @@ const char kSoftwareRenderingListJson[] = LONG_STRING_CONST( ] }, { - "id": 66, - "description": "Force compositing mode is unstable in MacOSX earlier than 10.8.", - "cr_bugs": [174101], - "os": { - "type": "macosx", - "version": { - "op": "<", - "value": "10.8" - } - }, - "features": [ - "force_compositing_mode" - ] - }, - { "id": 68, "description": "VMware Fusion 4 has corrupt rendering with Win Vista+.", "cr_bugs": [169470], @@ -970,6 +955,22 @@ LONG_STRING_CONST( "features": [ "texture_sharing" ] + }, + { + "id": 81, + "description": "Apple software renderer used under VMWare hangs on Mac OS 10.6 and 10.7.", + "cr_bugs": [230931], + "os": { + "type": "macosx", + "version": { + "op": "<=", + "value": "10.7" + } + }, + "vendor_id": "0x15ad", + "features": [ + "force_compositing_mode" + ] } ] } |