Remove suppressions for ScreenCaptureDevice tests.

The bug was fixed in https://code.google.com/p/webrtc/source/detail?r=4239.

BUG=244102

Review URL: https://chromiumcodereview.appspot.com/17391011

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@207235 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

HistoryURLProviderTest: Fix Memory Leak

Before this change, there could be tasks still left on other threads when a test completes.

BUG=250666

Review URL: https://chromiumcodereview.appspot.com/17340002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@207159 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove suppressions for bug 175512

I belive the bug has been already fixed with recent changes in that code.
Beside that the code has been moved so the suppression is not useful
either way.

BUG=175512

Review URL: https://chromiumcodereview.appspot.com/17239010

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@207158 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Exclude two tests that fail natively on win8.

BUG=251517
TBR=zhaoqin@chromium.org

Review URL: https://codereview.chromium.org/17442002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@207150 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Memory Sheriff]

Modified waterfall.sh to pass arguments on to match_suppressions.

Needed for new features (top-symbols/symbol-filter)

R=thestig@chromium.org
BUG=none

Review URL: https://chromiumcodereview.appspot.com/17286004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206976 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix ModuleSystemTest memory leaks and remove suppressions.

R=thakis@chromium.org
BUG=250531
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/17175004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206902 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Memory Sheriff] Suppress leaks.

TBR=thestig@chromium.org, earthdok@chromium.org
NOTRY=true
BUG=251004

Review URL: https://chromiumcodereview.appspot.com/17159003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206856 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 206826 "[Memory Sheriff] Leak suppressions"

dbabic landed r206834 right after, which should fix the issue. removing suppressions to see if the fix works

> [Memory Sheriff] Leak suppressions
>
> NOTRY=true
> TBR=thestig@chromium.org, dbabic@chromium.org
> BUG=251018
>
> Review URL: https://chromiumcodereview.appspot.com/17328003

TBR=groby@chromium.org,mpcomplete@chromium.org

Review URL: https://codereview.chromium.org/17341002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206840 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[Memory Sheriff] Leak suppressions

NOTRY=true
TBR=thestig@chromium.org, dbabic@chromium.org
BUG=251018

Review URL: https://chromiumcodereview.appspot.com/17328003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206826 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove suppressions for EventSender.

They have never been used after October 2012.
http://chromium-build-logs.appspot.com/suppression_query?suppression_query=bug_138510
http://chromium-build-logs.appspot.com/suppression_query?suppression_query=bug_146500

BUG=138510,146500
R=timurrrr@chromium.org

Review URL: https://codereview.chromium.org/17214002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206816 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress leak below MockHostResolverBase.

BUG=250688
TBR=pavely@chromium.org

Review URL: https://codereview.chromium.org/16943004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206659 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Move ScreenCaptureDeviceTest.Capture from media to content tests in valgrind suppressions.

The code itself was moved in http://crrev.com/205655

BUG=250681
TBR=sergeyu@chromium.org

Review URL: https://codereview.chromium.org/16821011

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206658 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress leak in HistoryURLProviderTest with frequency scoring.

BUG=250666
TBR=mpearson@chromium.org

Review URL: https://codereview.chromium.org/17208002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206656 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Slightly widen the jingle_glue suppression, add heapcheck suppression.

BUG=250533
TBR=pavely@chromium.org

Review URL: https://codereview.chromium.org/16861006

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206654 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Widen ModuleSystemTest suppression to include indirect constructors.

Also add a heapcheck suppression.

BUG=250531
TBR=kalman@chromium.org

Review URL: https://codereview.chromium.org/17198002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206653 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress various leaks below jingle_glue.

BUG=250533
TBR=pavely@chromium.org

Review URL: https://codereview.chromium.org/17195002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206652 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Widen an existing suppression slightly.

BUG=250532,20459
TBR=rsleevi@chromium.org

Review URL: https://codereview.chromium.org/17194002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206651 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress several leaks in ModuleSystemTests.

BUG=250531
TBR=kalman@chromium.org

Review URL: https://codereview.chromium.org/17192002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206650 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress two leaks in tests.

BUG=250529
TBR=pavely@chromium.org

Review URL: https://codereview.chromium.org/17191003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206647 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove unneeded IndexedDB/Worker leak suppressions.

These appear to have cleaned up circa May 22nd, around the time we
switched to content_shell but before the switch over to the new
Chromium-based IDB back-end which would have invalidated all of the
stacks anyway.

R=thestig@chromium.org
BUG=162829
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/17043009

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206511 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

select subset of browser_tests to run with Dr.Memory
TBR=bruening@chromium.org
BUG=NONE

Review URL: https://codereview.chromium.org/16865015

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@206110 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Update memcheck suppressions after r205655

BUG=244102
TBR=wez@chromium.org

Review URL: https://codereview.chromium.org/16365015

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@205699 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Speculative test excludes in attempt to fix dr-memory bot timeout

BUG=248373
TBR=glider@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/16035024

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@205361 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Exclude failing test on mac valgrind

TBR=thestig@chromium.org
BUG=247601
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15703013

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@205346 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Update suppression for the leak in SimpleIndexFile

TBR=rdsmith@chromium.org
BUG=231969

Review URL: https://codereview.chromium.org/16373009

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@205312 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

We were seeing ActivityLog memory leaks and assorted other errors. It was due to poor handling of error conditions (not cleaning up the DB state if something goes wrong). This CL should fix that.

Added every error-handling measure I could think of:

1. Moves the error handling into the ActivityDatabase class so that we can immediately (synchronously) kill the database, instead of asynchronously doing it from the ActivityLog
2. Closes the db even for non-catastrophic errors, so that we aren't just hammering away when the I/O thread has gone wonky
3. Adds checks in ActivityDatabase to see if the db is valid so that we aren't constantly continuing to try to write to a closed database
4. Removes dead code that I had superstitiously added to the activity_log_unittest to see if adding it would remove the memory leaks (it didn't)
5. Failures in Init explicitly call the error handling code, in case it isn't automatically triggered
6. Record now returns a bool, so we can stop trying to record actions if one has failed
7. In the case of unittests where everything is running on one thread anyway, ActivityLog synchronously kills the Activity Database
8. ActivityLog does cleanup on Shutdown, which happens before references start dying

BUG=246825

Review URL: https://chromiumcodereview.appspot.com/16510002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@205059 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix memory leak in TemplateURLService::ResetNonExtensionURLs

BUG=246560

Review URL: https://chromiumcodereview.appspot.com/16480003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@205058 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Move FileEnumerator to its own file, do some refactoring.

It creates a class FileInfo to contain the details rather than using a platform-specific typedef. This allows the accessors GetName, GetSize, etc. to be moved directly to this class (previously they were static helpers on the FileEnumerator class) which makes a bunch of code much cleaner. It also gives reasonable getting and initialization which the previous version lacked.

BUG=175002

Reland of 198820 and 298824
Original review = https://codereview.chromium.org/13165005
R=rvargas@chromium.org

Review URL: https://codereview.chromium.org/16392011

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@205019 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Update suppression for 87629

BUG=87629
TEST=none
TBR=kouhei@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16667008

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204955 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Retire Valgrind suppressions

A number of memory leaks and reads of uninitialized memory are no longer
occurring.

BUG=31800,103467,107503,107698,169311

Review URL: https://chromiumcodereview.appspot.com/15771010

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204937 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 204110 "Retire leak suppression for CSS rare data"

> Retire leak suppression for CSS rare data
>
> BUG=241892
>
> Review URL: https://chromiumcodereview.appspot.com/15816007

TBR=ericwilligers@chromium.org

Review URL: https://codereview.chromium.org/16514003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204929 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove suppression in mirror window controller
This is fixed in 202985

TBR=bruening
BUG=244105
TEST=none
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16667003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204921 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix for uninitialized memory valgrind alerts in IndexedDB unit tests

Valgrind doesn't like clang's optimized code in release builds for:

int foo;
if (!InitOrReturnFalse(&foo) || foo < 0) {
  return;
}

The emitted code is checking the value of foo even if the
InitOrReturnFalse() call is returning false, presumably to avoid a
branch. It's a safe optimiation for the compiler to make, but throws
valgrind for a loop.

Initialize foo with a dummy value to suppress the alert.

BUG=247474
R=dgrogan@chromium.org,thestig@chromium.org

Review URL: https://chromiumcodereview.appspot.com/15643015

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204906 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind: Remove an old suppression and disable a failing test on Mac.

BUG=247601
NOTRY=true
TBR=oshima

Review URL: https://chromiumcodereview.appspot.com/16647005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204864 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Retire obsolete Valgrind suppressions

BUG=108516,70599,107698

Review URL: https://chromiumcodereview.appspot.com/16531003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204839 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind Mac: Suppress a couple more leaks following r204514.

BUG=247506
TBR=tapted
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15789018

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204720 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind: Remove a couple confirmed fixed CSS leaks; Widen suppression for TestBrowserThreadBundle; Disable a failing WebRTCAudioDeviceTest on Mac.

BUG=100981,109470,247163,247525
TBR=ajwong@chromium.org,ericwilligers@chromium.org,glider@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/16485004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204699 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppressions for leaks caused by TestBrowserThreadBundle in r204603

TBR=thestig
BUG=247525

Review URL: https://codereview.chromium.org/16268014

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204637 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind Mac: Move a few Mac-only IndexedDB suppressions into the right file.

BUG=247474
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16005012

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204628 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind Mac: Suppress a bunch of new leaks from r204514, or widen existing suppressions that match. Disable another failing VideoCaptureManagerTest as well.

BUG=101359,105527,247163,247506
TBR=tapted@chromium.org,glider@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16603002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204618 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Uninitialized memory suppressions

Introduced in r247474, look like they're just from the values held
in test case lists in the unit test itself, but will fix ASAP.

BUG=247474
NOTRY=true
R=thestig@chromium.org

Review URL: https://chromiumcodereview.appspot.com/16049012

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204600 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind: Disable failing NotificationAudioControllerTest on CrOS; Disable failing VideoCaptureManagerTest on Mac.

BUG=247163,247440
TBR=glider@chromium.org
NOTRY=true

Review URL: https://codereview.chromium.org/16585002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204568 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Roll webrtc 4179:4187

This roll fixes a memory leak in DesktopRegion and contains
no other changes. Also removed suppressions for the leak.

BUG=246870

Review URL: https://chromiumcodereview.appspot.com/16503003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204477 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove obsolete suppression about v8::internal::CopyBytes.

R=jkummerow@chromium.org
BUG=chromium:222628,chromium:233500

Review URL: https://chromiumcodereview.appspot.com/15793011

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204452 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind Mac: Update a disk_cache suppression and disable more failing media tests.

BUG=231969,247163
TBR=glider@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16527002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204434 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind/Heapcheck: Add more suppressions for a V8 leak.

BUG=246567
TBR=zhaoqin@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16492004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204364 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind: Update a leveldb suppression after r204301

BUG=111186
TBR=dgrogan
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16488005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204362 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Flush blocking pool during teardown of ManagedUserRegistrationServiceTest.

BUG=245241

Review URL: https://chromiumcodereview.appspot.com/15934010

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204279 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add more TSan v2 suppressions:
 - races on WebRTC trace counters (issue 244774)
 - race in webrtc::VideoCodingModuleImpl::RegisterPacketRequestCallback (issue 246968)
 - race in the webrtc::EventPosix class (issue 246970)
 - race in GpuWatchdogThread (issue 246974)

BUG=244774,246968,246970,246974
TBR=hclam@chromium.org

Review URL: https://codereview.chromium.org/16454004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204245 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Heapcheck: Suppress a leak in V8.

BUG=246567
TBR=zhaoqin@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16436002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204211 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind: Widen the suppression from r204128.

BUG=246870
TBR=sergeyu@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16413003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204137 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add valgrind suppressions for bug 246870

BUG=246870
TBR=thestig@chromium.org

Review URL: https://codereview.chromium.org/16068005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204128 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind/Heapcheck: Suppress more ActivityLog leaks.

BUG=246825
TBR=felt@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16077012

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204112 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Retire leak suppression for CSS rare data

BUG=241892

Review URL: https://chromiumcodereview.appspot.com/15816007

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204110 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppressing ActivityLog Memory errors

We're seeing Activity Log related memory leaks. This patch suppresses the leaks so that the tests can keep running while we investigate.

NOTRY=true
BUG=246825

Review URL: https://chromiumcodereview.appspot.com/16212010

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204073 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

[content shell] don't leak the devtools delegate

BUG=246359
R=pfeldman@chromium.org

Review URL: https://chromiumcodereview.appspot.com/16360007

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@204033 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

supprress memory leak from TemplateURLServiceTest::AddKeywordWithDate
TBR=Samusaaron3@gmail.com
BUG=246560
TEST=waterfall

Review URL: https://codereview.chromium.org/15934011

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203943 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix leak in pnacl translation cache and remove check suppressions

R=jvoung@chromium.org
BUG= https://code.google.com/p/chromium/issues/detail?id=246092
BUG= https://code.google.com/p/nativeclient/issues/detail?id=3372
TEST= unit_tests --gtest_filter=PNaClTranslationCacheTest.*

Review URL: https://chromiumcodereview.appspot.com/16295020

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203929 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix uninitialized read in the SimpleCache tests.

DiskCacheEntryTest.SimpleCacheOptimisticWriteReleases did not initialize
its buffer. It was immaterial to the test, but less so to valgrind.

R=zhaoqin@chromium.org
BUG=245952

Review URL: https://chromiumcodereview.appspot.com/16306003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203887 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress memory leak from ShellBrowserMainParts::PreMainMessageLoopRun
TBR=nileshagrawal@chromium.org
BUG=246359

Review URL: https://codereview.chromium.org/15894029

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203779 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress memory leak content::ShellBrowserMainParts::PreMainMessageLoopRun
TBR=ajwong@chromium.org
BUG=245828
TEST=waterfall

Review URL: https://codereview.chromium.org/16057020

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203772 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress memory leak from ManagedUserRegistrationService::Register
TBR=bauerb@chromium.org
BUG=246306
TEST=waterfall

Review URL: https://codereview.chromium.org/16336019

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203755 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress uninit read from content::RenderViewImpl
TBR=nasko@chromium.org
BUG=246153
TEST=waterfall

Review URL: https://codereview.chromium.org/15700015

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203705 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress memory leak from content::RenderViewHostImpl
TBR=nasko@chromium.org
BUG=246148
TEST=waterfall

Review URL: https://codereview.chromium.org/16156012

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203702 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress memory leak from nacl_cache::PNaClTranslationCache
TBR=dschuff@chromium.org
BUG=246092
TEST=waterfall

Review URL: https://codereview.chromium.org/16325004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203694 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress uninit read from disk_cache
TBR=gavinp@chromium.org
TEST=waterfall
BUG=245952

Review URL: https://codereview.chromium.org/16307003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203621 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress memory leak from content::Zygote::HandleReapRequest
TBR=brettw@chromium.org
BUG=245866
TEST=waterfall.sh

Review URL: https://codereview.chromium.org/16105016

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203577 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix leak in WeakPtr test introduced in r202038, and remove suppression.

TBR=darin
BUG=243757,232143

Review URL: https://chromiumcodereview.appspot.com/16035004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203547 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

suppress memory leak from content::BrowserThread::PostTaskAndReply
TBR=ajwong@chromium.org
BUG=245828
TEST=waterfall

Review URL: https://codereview.chromium.org/15774007

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203467 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

exclude test ClientSideDetectionHostTest.NavigationCancelsShouldClassifyUrl
TBR=kewang@chromium.org
BUG=245797

Review URL: https://codereview.chromium.org/16264002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203450 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress memory leak from WorkerServiceImpl::WorkerServiceImpl
TBR=simonhatch@chromium.org
BUG=245714
TEST=waterfall

Review URL: https://codereview.chromium.org/16240012

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203421 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Bump the version of asan_symbolize.py to 183006.

BUG=245262
R=timurrrr@chromium.org

Review URL: https://codereview.chromium.org/15715020

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203407 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Roll asan_symbolize.py r182915 from the LLVM trunk.

BUG=244668
R=timurrrr@chromium.org

Review URL: https://codereview.chromium.org/16134005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@203130 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

More suppressions for TSan v2:
 - fixed the suppression for libevent to contain the correct filename (issue 23244)
 - removed the race:base::Thread::ThreadMain suppression, which was too broad (issue 115540)
 - suppressed a race in skia::BeginPlatformPaint (issue 244368)
 - suppressed a race in unixTempFileDir (issue 244385)

BUG=23244,115540,244368,244385
R=timurrrr@chromium.org

Review URL: https://codereview.chromium.org/15771007

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202890 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Expand suppression 166709c to also cover bug 244800.

BUG=166709,244800
R=timurrrr@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16194006

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202862 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Exclude one more crashing test from Valgrind/Mac runs
BUG=244102
TBR=bruening
NOTRY=true

Review URL: https://codereview.chromium.org/16072011

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202845 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Un-exclude a few non-Mac tests to check if any of them still hang
TBR=bruening
BUG=163314
NOTRY=true

Review URL: https://codereview.chromium.org/15924008

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202844 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Add a comment explaining why readline is the only way we can avoid buffering
BUG=163314
R=glider@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15870005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202841 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Run the MessageLoop longer in order to clear out extra DeleteSoon()s.

Also revert the leak suppression (r202589) so we'll know if this
didn't fix the problem.

BUG=244446

Review URL: https://chromiumcodereview.appspot.com/16133011

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202768 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Valgrind: Do not use the sandbox for layout tests.

Recently layout tests changed to use content_shell instead of DRT. Content_shell attempts to use the sandbox if it can, and that made the Valgrind layout test bot unhappy. All other Valgrind bots disable the sandbox, and layout tests should too.

TBR=timurrrr@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16087005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202732 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress flaky memory leak in DeclarativeContentRulesRegistryTest.ActiveRulesDoesntGrow on Chrome OS

BUG=244446
R=timurrrr@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16060003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202589 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress uninitialized read report in WebRtcAudioRenderer::SourceCallback on MacOS

BUG=244437
R=timurrrr@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15658008

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202576 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress uninitialized read report in VideoDestinationHandlerTest.Open on Mac OS

BUG=244420
R=timurrrr@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15912008

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202567 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Fix and widen a suppression to make reliability_tests greener
BUG=222876
TBR=bruening
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/16151002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202555 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Widen one suppression to account for a recent rename of a class
BUG=96684
TBR=bruening@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15975006

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202550 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Try to read the stdout pipe to the Valgrind process differently
Should fix reliability_tests under Valgrind

BUG=163314
TBR=glider@chromium.org
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15675005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202523 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Removed suppression for bug 243963.

This was a race because PolicyDomainDescriptors are passed from the UI to the
FILE thread in scoped_refptrs, but the class derived from RefCounted instead
of RefCountedThreadSafe. The fix was in http://crrev.com/202452.

BUG=243963

Review URL: https://chromiumcodereview.appspot.com/16018007

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202520 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Exclude one test that hangs under Valgrind/Mac
BUG=244257
TBR=bruening
NOTRY=true

Review URL: https://chromiumcodereview.appspot.com/15780007

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202444 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove some dead code from tools/valgrind/common.py

Also, hopefully improve the buffering behaviour, see
BUG=163314

NOTRY=true
R=glider@chromium.org

Review URL: https://chromiumcodereview.appspot.com/16103005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202422 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Disable ScreenCaptureDeviceTest.Capture under Valgrind on Mac

BUG=244102
NOTRY=true
R=timurrrr@chromium.org

ScreenCaptureDeviceTest.Capture causes SEGV_MAPERR under Mac Valgrind. Bug
244102.

Review URL: https://chromiumcodereview.appspot.com/16104003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202417 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress leak in ash::internal::MirrorWindowController::UpdateWindow

TBR=oshima@chromium.org
BUG=244105

Review URL: https://codereview.chromium.org/15662004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202350 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress invalid read/write in Mac screen capture test

BUG=244102
TBR=sergeyu@chromium.org

Review URL: https://codereview.chromium.org/16076002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202349 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Update valgrind suppressions for ConvertYUVToRGB32.

Function was recently converted from extern C to C++, signature
needed updating.

BUG=113076
TEST=none
TBR=bruening

Review URL: https://codereview.chromium.org/15729006

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202253 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress race involving policy::BrowserPolicyConnector::CreatePolicyService

TBR=joaodasilva@chromium.org
BUG=243963

Review URL: https://codereview.chromium.org/15919005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202249 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Delete Automation[Tab/Renderer]Helper and fix users.

For Chrome Endure, use the memory benchmarking V8 extension instead.
For the old ChromeDriver, just mark the messages as deprecated on the client side. I'll remove all the old ChromeDriver code later. I want to keep it here since we release ChromeDriver from HEAD, and we might need to do critical fixes for Chrome 28 and below still.
BUG=240580

Review URL: https://chromiumcodereview.appspot.com/14977013

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202205 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress leaks in base::WeakPtrDeathTest_NonOwnerThreadDeletesWeakPtrAfterReference_Test::TestBody

TBR=wez@chromium.org
BUG=243757

Review URL: https://codereview.chromium.org/15937006

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202108 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Suppress leaks in fileapi::SandboxMountPointProviderTest::GetRootPath

TBR=kinuko@chromium.org
BUG=243753

Review URL: https://codereview.chromium.org/16021004

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202090 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Remove references to TopSitesTest

TopSitesTest was renamed to TopSitesImplTest in crrev.com/197127,
hence references to TopSitesTest no longer make sense.

BUG=180467
TEST=none

Review URL: https://chromiumcodereview.appspot.com/15823002

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@202014 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Revert 201955 "Allow multiple base::MessagePumpForUI instances t..."

Broke browser_tests on XP Tests (dbg)(4):
http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%28dbg%29%284%29/builds/33253

[844:3260:0523/194017:1212046:FATAL:message_window.cc(28)] Check failed: CalledOnValidThread().
Backtrace:
  base::Histogram::GetCountAndBucketData [0x08408FD1+2262176]
  base::Histogram::GetCountAndBucketData [0x0827FBBE+651405]
  base::Histogram::GetCountAndBucketData [0x082E1E60+1053487]
  base::Histogram::GetCountAndBucketData [0x0825CC46+508181]
  base::Histogram::GetCountAndBucketData [0x0825BB66+503861]
  base::Histogram::GetCountAndBucketData [0x0825A534+498179]
  base::Histogram::GetCountAndBucketData [0x08256C16+483557]
  base::Histogram::GetCountAndBucketData [0x08256C82+483665]
  base::Histogram::GetCountAndBucketData [0x08256324+481267]
  base::Histogram::GetCountAndBucketData [0x081F59AF+85630]
  base::Histogram::GetCountAndBucketData [0x081F84AC+96635]
  base::Histogram::GetCountAndBucketData [0x081F5917+85478]
  base::Histogram::GetCountAndBucketData [0x08249FD3+431266]
  base::Histogram::GetCountAndBucketData [0x08249255+427812]
  base::Histogram::GetCountAndBucketData [0x082462CD+415644]
  base::Histogram::GetCountAndBucketData [0x0831D883+1297746]
  base::Histogram::GetCountAndBucketData [0x0831D602+1297105]
  ViewHostMsg_TextInputStateChanged::Read [0x0D3DD6F9+6214944]
  ViewHostMsg_TextInputStateChanged::Read [0x0D3DD584+6214571]
  ViewHostMsg_TextInputStateChanged::Read [0x0D33048E+5505717]
  ViewHostMsg_TextInputStateChanged::Read [0x0D3303FA+5505569]
  ViewHostMsg_TextInputStateChanged::Read [0x0D32DDEA+5495825]
  ViewHostMsg_TextInputStateChanged::Read [0x0D32BAC7+5486830]
  ViewHostMsg_TextInputStateChanged::Read [0x0D3DC9B6+6211549]
  ViewHostMsg_TextInputStateChanged::Read [0x0D367071+5729944]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x03E93429+30896697]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x025BD0DF+4853487]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x02701C3F+6183503]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x026EECCB+6105819]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x026EF6DD+6108397]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x026EFEAF+6110399]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x026F673D+6137165]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x02702637+6186055]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x026F4FA0+6131120]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x0262E070+5316224]
  (No symbol) [0x00632BF5]
  RelaunchChromeBrowserWithNewCommandLineIfNeeded [0x03E2BD73+30473091]
  (No symbol) [0x00632975]
  std::_Init_locks::operator= [0x03F5574F+5791]
  std::_Init_locks::operator= [0x03F5557F+5327]
  RegisterWaitForInputIdle [0x7C817077+73]

> Allow multiple base::MessagePumpForUI instances to be created simultanenously on Windows.
>
> The current implementation of base::MessagePumpForUI on Windows registers a window class with a predefined name in order to create a message-only window. The window class is unregistered when base::MessagePumpForUI is deleted. This causes issues when two or more instances of base::MessagePumpForUI are created/destroyed simultanenously on different threads. For instance once thread can unregister the window class right before the other thread is trying to create a window using this class.
>
> The CL addresses this problem by switching MessageWindow to implement a message-only window. It also moves MessageWindow from remoting/host/win to base/win along with the corresponding unit test.
>
> MessageWindow registers a uniquely named window class per MessageWindow instance making sure that different MessageWindow objects do not share any resources. In the future this can be optimized further by registering a common window class shared by all MessageWindow objects in a thread-safe manner (by using LazyInstance for example).
>
> BUG=241939
>
> Review URL: https://chromiumcodereview.appspot.com/15261005

TBR=alexeypa@chromium.org

Review URL: https://codereview.chromium.org/15973003

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@201974 4ff67af0-8c30-449e-8e8b-ad334ec8d88c

Allow multiple base::MessagePumpForUI instances to be created simultanenously on Windows.

The current implementation of base::MessagePumpForUI on Windows registers a window class with a predefined name in order to create a message-only window. The window class is unregistered when base::MessagePumpForUI is deleted. This causes issues when two or more instances of base::MessagePumpForUI are created/destroyed simultanenously on different threads. For instance once thread can unregister the window class right before the other thread is trying to create a window using this class.

The CL addresses this problem by switching MessageWindow to implement a message-only window. It also moves MessageWindow from remoting/host/win to base/win along with the corresponding unit test.

MessageWindow registers a uniquely named window class per MessageWindow instance making sure that different MessageWindow objects do not share any resources. In the future this can be optimized further by registering a common window class shared by all MessageWindow objects in a thread-safe manner (by using LazyInstance for example).

BUG=241939

Review URL: https://chromiumcodereview.appspot.com/15261005

git-svn-id: http://src.chromium.org/svn/trunk/src/tools/valgrind@201955 4ff67af0-8c30-449e-8e8b-ad334ec8d88c