Disable delete mismatch on Dr.Memory bot

Disable delete mismatch on Dr.Memory bot
xref https://code.google.com/p/drmemory/issues/detail?id=1058

R=bruening@chromium.org
BUG=413215
NOTRY=true

Committed: https://crrev.com/e98b2dc1964cc237248ebb1627be52242d1c55ca
Cr-Commit-Position: refs/heads/master@{#295775}

[Derek Bruening, 2014-09-16 17:14:02]

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
File tools/valgrind/drmemory/suppressions.txt (right):

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
tools/valgrind/drmemory/suppressions.txt:261: *!scoped_ptr<>*
Should this be removed too?  It seems awfully wide.

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
File tools/valgrind/drmemory/suppressions_full.txt (left):

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
tools/valgrind/drmemory/suppressions_full.txt:2201: INVALID HEAP ARGUMENT
Why was this duplicated here?

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

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/valgrind_test...
tools/valgrind/valgrind_test.py:942: proc += ["-no_check_delete_mismatch"]
You need to check that this is running on out/Release before disabling, right?

[zhaoqin, 2014-09-16 18:22:00]

PTAL

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
File tools/valgrind/drmemory/suppressions.txt (right):

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
tools/valgrind/drmemory/suppressions.txt:261: *!scoped_ptr<>*
On 2014/09/16 17:14:01, Derek Bruening wrote:
> Should this be removed too?  It seems awfully wide.

Done.

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
File tools/valgrind/drmemory/suppressions_full.txt (left):

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
tools/valgrind/drmemory/suppressions_full.txt:2201: INVALID HEAP ARGUMENT
On 2014/09/16 17:14:01, Derek Bruening wrote:
> Why was this duplicated here?

the sheriff not sure how suppression file works?

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

https://codereview.chromium.org/574643006/diff/1/tools/valgrind/valgrind_test...
tools/valgrind/valgrind_test.py:942: proc += ["-no_check_delete_mismatch"]
On 2014/09/16 17:14:02, Derek Bruening wrote:
> You need to check that this is running on out/Release before disabling, right?

Done.

[zhaoqin, 2014-09-17 14:26:13]

On 2014/09/16 18:22:00, zhaoqin wrote:
> PTAL
> 
>
https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
> File tools/valgrind/drmemory/suppressions.txt (right):
> 
>
https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
> tools/valgrind/drmemory/suppressions.txt:261: *!scoped_ptr<>*
> On 2014/09/16 17:14:01, Derek Bruening wrote:
> > Should this be removed too?  It seems awfully wide.
> 
> Done.
> 
>
https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
> File tools/valgrind/drmemory/suppressions_full.txt (left):
> 
>
https://codereview.chromium.org/574643006/diff/1/tools/valgrind/drmemory/supp...
> tools/valgrind/drmemory/suppressions_full.txt:2201: INVALID HEAP ARGUMENT
> On 2014/09/16 17:14:01, Derek Bruening wrote:
> > Why was this duplicated here?
> 
> the sheriff not sure how suppression file works?
> 
>
https://codereview.chromium.org/574643006/diff/1/tools/valgrind/valgrind_test.py
> File tools/valgrind/valgrind_test.py (right):
> 
>
https://codereview.chromium.org/574643006/diff/1/tools/valgrind/valgrind_test...
> tools/valgrind/valgrind_test.py:942: proc += ["-no_check_delete_mismatch"]
> On 2014/09/16 17:14:02, Derek Bruening wrote:
> > You need to check that this is running on out/Release before disabling,
right?
> 
> Done.

ping

[Derek Bruening, 2014-09-19 20:57:04]

LGTM

[zhaoqin, 2014-09-19 21:01:16]

The CQ bit was checked by zhaoqin@google.com

[commit-bot: I haz the power, 2014-09-19 21:02:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/574643006/20001

[commit-bot: I haz the power, 2014-09-19 21:04:07]

Committed patchset #2 (id:20001) as 42821b1d481452b037a3e5349f739952e5555b4c

[commit-bot: I haz the power, 2014-09-19 21:04:08]

Failed to apply patch for tools/valgrind/drmemory/suppressions.txt:
While running git apply --index -p1;
  error: patch failed: tools/valgrind/drmemory/suppressions.txt:257
  error: tools/valgrind/drmemory/suppressions.txt: patch does not apply

Patch:       tools/valgrind/drmemory/suppressions.txt
Index: tools/valgrind/drmemory/suppressions.txt
diff --git a/tools/valgrind/drmemory/suppressions.txt
b/tools/valgrind/drmemory/suppressions.txt
index
4362445d30e79b099ff5da393d907f2770a8b685..e9a76f3e58b192bd601af09f590ac3921b5b01d7
100644
--- a/tools/valgrind/drmemory/suppressions.txt
+++ b/tools/valgrind/drmemory/suppressions.txt
@@ -257,20 +257,16 @@ instruction=mov    0xfffffffc(%esp) -> %eax
 chrome.dll!blink::RenderStyle::resetBorder*
 
 INVALID HEAP ARGUMENT
-name=http://crbug.com/101537, http://crbug.com/101717 (1)
-*!scoped_ptr<>*
-
-INVALID HEAP ARGUMENT
-name=http://crbug.com/101717 (2)
+name=http://crbug.com/101717 (1)
 *!sandbox::PolicyBase::~PolicyBase
 
 INVALID HEAP ARGUMENT
-name=http://crbug.com/101717 (3)
+name=http://crbug.com/101717 (2)
 *!scoped_ptr<>::~scoped_ptr<>
 *!sandbox::GetHandleName
 
 INVALID HEAP ARGUMENT
-name=http://crbug.com/101717 (4)
+name=http://crbug.com/101717 (3)
 *!scoped_ptr<>::~scoped_ptr<>
 *!sandbox::GetPathFromHandle
 
@@ -653,11 +649,6 @@ base.dll!base::MessageLoop::RunTask
 base.dll!base::Thread::StopSoon
 base.dll!base::MessageLoop::DeferOrRunPendingTask
 
-INVALID HEAP ARGUMENT
-name=http://crbug.com/413215
-*!replace_operator_delete*
-v8.dll!*
-
 UNADDRESSABLE ACCESS
 name=http://crbug.com/414675
 blink_web.dll!blink::toCoreFrame

[commit-bot: I haz the power, 2014-09-19 21:05:35]

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