summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorccameron@chromium.org <ccameron@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-11-20 21:08:08 +0000
committerccameron@chromium.org <ccameron@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2013-11-20 21:08:08 +0000
commiteb4317e03d6201f00de7d687783b5aea110a7923 (patch)
treedf8331cd0dd4daf33dda159260ccdb675469c226
parent2ca5087652824db1772f38c451391082024d2df8 (diff)
downloadchromium_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.cc27
-rw-r--r--content/browser/gpu/compositor_util_browsertest.cc9
-rw-r--r--gpu/config/software_rendering_list_json.cc31
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"
+ ]
}
]
}