Skip most tests that are currently broken w/ DrMemory.

Skip most tests that are currently broken w/ DrMemory.

After the switch from GYP->GN, some tests are passing, but most
are still broken. In order to green up the tree, we'll skip the
broken tests for now, and gradually re-enable them.

R=thestig@chromium.org, groby@chromium.org
NOTRY=true
BUG=633693
Committed: https://crrev.com/7c24d26fa15ce41e869c719d572e10deb3d21a31
Cr-Commit-Position: refs/heads/master@{#409869}

[Lei Zhang, 2016-08-04 19:37:16]

lgtm

Slightly heavy handed, but ok for now.

https://codereview.chromium.org/2218513002/diff/1/tools/valgrind/chrome_tests.py
File tools/valgrind/chrome_tests.py (right):

https://codereview.chromium.org/2218513002/diff/1/tools/valgrind/chrome_tests...
tools/valgrind/chrome_tests.py:803: 'cc',
dup?

[Dirk Pranke, 2016-08-04 19:38:22]

Yes, it's clearly heavy handed.

A relatively straightforward next step is to gradually un-skip each test step
and file suppressions for the individual errors, of course.

https://codereview.chromium.org/2218513002/diff/1/tools/valgrind/chrome_tests.py
File tools/valgrind/chrome_tests.py (right):

https://codereview.chromium.org/2218513002/diff/1/tools/valgrind/chrome_tests...
tools/valgrind/chrome_tests.py:803: 'cc',
On 2016/08/04 19:37:16, Lei Zhang wrote:
> dup?

Yup, thx.

[Dirk Pranke, 2016-08-04 19:39:14]

Description was changed from

==========
Skip most tests that are currently broken w/ DrMemory.

After the switch from GYP->GN, some tests are passing, but most
are still broken. In order to green up the tree, we'll skip the
broken tests for now, and gradually re-enable them.

R=thestig@chromium.org, groby@chromium.org
BUG=633693
==========

to

==========
Skip most tests that are currently broken w/ DrMemory.

After the switch from GYP->GN, some tests are passing, but most
are still broken. In order to green up the tree, we'll skip the
broken tests for now, and gradually re-enable them.

R=thestig@chromium.org, groby@chromium.org
NOTRY=true
BUG=633693
==========

[Dirk Pranke, 2016-08-04 19:39:39]

landing NOTRY=true since we don't have any drmem trybots and there's no point in
running any of the others.

[Dirk Pranke, 2016-08-04 19:39:43]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-08-04 19:39:43]

The patchset sent to the CQ was uploaded after l-g-t-m from thestig@chromium.org
Link to the patchset: https://codereview.chromium.org/2218513002/#ps20001
(title: "fix dup")

[commit-bot: I haz the power, 2016-08-04 19:40:21]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-04 19:44:24]

Description was changed from

==========
Skip most tests that are currently broken w/ DrMemory.

After the switch from GYP->GN, some tests are passing, but most
are still broken. In order to green up the tree, we'll skip the
broken tests for now, and gradually re-enable them.

R=thestig@chromium.org, groby@chromium.org
NOTRY=true
BUG=633693
==========

to

==========
Skip most tests that are currently broken w/ DrMemory.

After the switch from GYP->GN, some tests are passing, but most
are still broken. In order to green up the tree, we'll skip the
broken tests for now, and gradually re-enable them.

R=thestig@chromium.org, groby@chromium.org
NOTRY=true
BUG=633693
==========

[commit-bot: I haz the power, 2016-08-04 19:44:25]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-04 19:46:28]

Description was changed from

==========
Skip most tests that are currently broken w/ DrMemory.

After the switch from GYP->GN, some tests are passing, but most
are still broken. In order to green up the tree, we'll skip the
broken tests for now, and gradually re-enable them.

R=thestig@chromium.org, groby@chromium.org
NOTRY=true
BUG=633693
==========

to

==========
Skip most tests that are currently broken w/ DrMemory.

After the switch from GYP->GN, some tests are passing, but most
are still broken. In order to green up the tree, we'll skip the
broken tests for now, and gradually re-enable them.

R=thestig@chromium.org, groby@chromium.org
NOTRY=true
BUG=633693

Committed: https://crrev.com/7c24d26fa15ce41e869c719d572e10deb3d21a31
Cr-Commit-Position: refs/heads/master@{#409869}
==========

[commit-bot: I haz the power, 2016-08-04 19:46:29]

Patchset 2 (id:??) landed as
https://crrev.com/7c24d26fa15ce41e869c719d572e10deb3d21a31
Cr-Commit-Position: refs/heads/master@{#409869}