Commit 7e6bbeaa authored by Nico Weber's avatar Nico Weber Committed by Commit Bot

run components_browsertests on the memory bots

I added them to the clang/tot memory bots in
https://chromium-review.googlesource.com/1060293 and they mostly pass.
Disable the two tests that don't pass.

TBR=nyquist,dmazzoni

Bug: 843511
Change-Id: Iefc4f1450e3da57ad7854513587e1c9e2744a75e
Reviewed-on: https://chromium-review.googlesource.com/1067168Reviewed-by: default avatarTommy Nyquist <nyquist@chromium.org>
Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560329}
parent 459ce346
...@@ -101,7 +101,13 @@ class DomDistillerJsTest : public content::ContentBrowserTest { ...@@ -101,7 +101,13 @@ class DomDistillerJsTest : public content::ContentBrowserTest {
} }
}; };
IN_PROC_BROWSER_TEST_F(DomDistillerJsTest, RunJsTests) { #if defined(MEMORY_SANITIZER)
// https://crbug.com/845180
#define MAYBE_RunJsTests DISABLED_RunJsTests
#else
#define MAYBE_RunJsTests RunJsTests
#endif
IN_PROC_BROWSER_TEST_F(DomDistillerJsTest, MAYBE_RunJsTests) {
// TODO(jaebaek): Revisit this code when the --use-zoom-for-dsf feature on // TODO(jaebaek): Revisit this code when the --use-zoom-for-dsf feature on
// Android is done. If we remove this code (i.e., enable --use-zoom-for-dsf), // Android is done. If we remove this code (i.e., enable --use-zoom-for-dsf),
// HTMLImageElement::LayoutBoxWidth() returns a value that has a small error // HTMLImageElement::LayoutBoxWidth() returns a value that has a small error
......
...@@ -107,7 +107,13 @@ class PdfAccessibilityTreeTest : public content::RenderViewTest { ...@@ -107,7 +107,13 @@ class PdfAccessibilityTreeTest : public content::RenderViewTest {
std::vector<PP_PrivateAccessibilityCharInfo> chars_; std::vector<PP_PrivateAccessibilityCharInfo> chars_;
}; };
TEST_F(PdfAccessibilityTreeTest, TestEmptyPDFPage) { #if defined(MEMORY_SANITIZER)
// https://crbug.com/845143
#define MAYBE_TestEmptyPDFPage DISABLED_TestEmptyPDFPage
#else
#define MAYBE_TestEmptyPDFPage TestEmptyPDFPage
#endif
TEST_F(PdfAccessibilityTreeTest, MAYBE_TestEmptyPDFPage) {
content::RenderFrame* render_frame = view_->GetMainRenderFrame(); content::RenderFrame* render_frame = view_->GetMainRenderFrame();
render_frame->SetAccessibilityModeForTest(ui::AXMode::kWebContents); render_frame->SetAccessibilityModeForTest(ui::AXMode::kWebContents);
ASSERT_TRUE(render_frame->GetRenderAccessibility()); ASSERT_TRUE(render_frame->GetRenderAccessibility());
......
...@@ -807,6 +807,16 @@ ...@@ -807,6 +807,16 @@
}, },
"test": "chromedriver_unittests" "test": "chromedriver_unittests"
}, },
{
"args": [
"--test-launcher-batch-limit=1",
"--test-launcher-print-test-stdio=always"
],
"swarming": {
"can_use_on_swarming_builders": true
},
"test": "components_browsertests"
},
{ {
"args": [ "args": [
"--test-launcher-batch-limit=1", "--test-launcher-batch-limit=1",
...@@ -1365,6 +1375,12 @@ ...@@ -1365,6 +1375,12 @@
}, },
"test": "browser_tests" "test": "browser_tests"
}, },
{
"swarming": {
"can_use_on_swarming_builders": true
},
"test": "components_browsertests"
},
{ {
"swarming": { "swarming": {
"can_use_on_swarming_builders": true "can_use_on_swarming_builders": true
...@@ -1972,6 +1988,12 @@ ...@@ -1972,6 +1988,12 @@
}, },
"test": "chromeos_unittests" "test": "chromeos_unittests"
}, },
{
"swarming": {
"can_use_on_swarming_builders": true
},
"test": "components_browsertests"
},
{ {
"swarming": { "swarming": {
"can_use_on_swarming_builders": true "can_use_on_swarming_builders": true
...@@ -2427,6 +2449,12 @@ ...@@ -2427,6 +2449,12 @@
}, },
"test": "chromeos_unittests" "test": "chromeos_unittests"
}, },
{
"swarming": {
"can_use_on_swarming_builders": true
},
"test": "components_browsertests"
},
{ {
"swarming": { "swarming": {
"can_use_on_swarming_builders": true "can_use_on_swarming_builders": true
...@@ -2835,6 +2863,12 @@ ...@@ -2835,6 +2863,12 @@
}, },
"test": "chromedriver_unittests" "test": "chromedriver_unittests"
}, },
{
"swarming": {
"can_use_on_swarming_builders": true
},
"test": "components_browsertests"
},
{ {
"swarming": { "swarming": {
"can_use_on_swarming_builders": true "can_use_on_swarming_builders": true
...@@ -3235,6 +3269,12 @@ ...@@ -3235,6 +3269,12 @@
}, },
"test": "chromedriver_unittests" "test": "chromedriver_unittests"
}, },
{
"swarming": {
"can_use_on_swarming_builders": true
},
"test": "components_browsertests"
},
{ {
"swarming": { "swarming": {
"can_use_on_swarming_builders": true "can_use_on_swarming_builders": true
......
...@@ -685,13 +685,6 @@ ...@@ -685,13 +685,6 @@
# chromium.win # chromium.win
'Win7 Tests (dbg)(1)', 'Win7 Tests (dbg)(1)',
'Win10 Tests x64 (dbg)', 'Win10 Tests x64 (dbg)',
# chromium.memory
'Linux ASan LSan Tests (1)',
'Linux ASan Tests (sandboxed)',
'Linux Chromium OS ASan LSan Tests (1)',
'Linux ChromiumOS MSan Tests',
'Linux MSan Tests',
'Linux TSan Tests',
], ],
'modifications': { 'modifications': {
'android-marshmallow-arm64-rel': { 'android-marshmallow-arm64-rel': {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment