[Chromoting] Disable a unit test in which win_tsan and mac_valgrind catch leaks.

[Chromoting] Disable a unit test in which win_tsan and mac_valgrind catch use of uninitialized memory.

The unit test was added in r148888.


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=148969

[simonmorris, 2012-07-30 16:13:30]

ptal

[garykac, 2012-07-30 16:14:30]

lgtm

[commit-bot: I haz the power, 2012-07-30 16:14:48]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/simonmorris@chromium.org/10826062/1

[Sergey Ulanov, 2012-07-30 17:36:35]

Don't need to use CQ for things like this, better to dcommit to make the bot
green

http://codereview.chromium.org/10826062/diff/1/remoting/base/decoder_vp8_unit...
File remoting/base/decoder_vp8_unittest.cc (right):

http://codereview.chromium.org/10826062/diff/1/remoting/base/decoder_vp8_unit...
remoting/base/decoder_vp8_unittest.cc:60: // The maximum error is
non-deterministic. The mean error is not too high,
might be better to have separate TODO() and open another bug to track the leak.

[commit-bot: I haz the power, 2012-07-30 17:36:44]

Change committed as 148969

[simonmorris, 2012-07-30 17:58:51]

http://codereview.chromium.org/10826062/diff/1/remoting/base/decoder_vp8_unit...
File remoting/base/decoder_vp8_unittest.cc (right):

http://codereview.chromium.org/10826062/diff/1/remoting/base/decoder_vp8_unit...
remoting/base/decoder_vp8_unittest.cc:60: // The maximum error is
non-deterministic. The mean error is not too high,
On 2012/07/30 17:36:35, sergeyu wrote:
> might be better to have separate TODO() and open another bug to track the
leak.

It's almost certainly the same problem. I should have said "use of uninitialized
memory", not "leak" - I've edited the CL.