| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
I believe this recent leak is another bit of fallout from leaking
globals_ so I'm expanding the existing suppression:
{
<insert_a_suppression_name_here>
Memcheck:Leak
fun:_Znw*
fun:_ZN9__gnu_cxx13new_allocatorIPNSt8__detail10_Hash_nodeISsLb0EEEE8allocateEmPKv
fun:_ZNSt10_HashtableISsSsSaISsESt9_IdentityISsESt8equal_toISsESt4hashISsENSt8__detail18_Mod_range_hashingENS7_20_Default_ranged_hashENS7_20_Prime_rehash_policyELb0ELb1ELb1EE19_M_allocate_bucketsEm
fun:_ZNSt10_HashtableISsSsSaISsESt9_IdentityISsESt8equal_toISsESt4hashISsENSt8__detail18_Mod_range_hashingENS7_20_Default_ranged_hashENS7_20_Prime_rehash_policyELb0ELb1ELb1EEC2EmRKS6_RKS8_RKS9_RKS4_RKS2_RKS0_
fun:_ZNSt15__unordered_setISsSt4hashISsESt8equal_toISsESaISsELb0EEC2EmRKS1_RKS3_RKS4_
fun:_ZNSt13unordered_setISsSt4hashISsESt8equal_toISsESaISsEEC2EmRKS1_RKS3_RKS4_
fun:_ZN3net18HttpNetworkSession6ParamsC1Ev
fun:_ZN8IOThread34ConstructProxyScriptFetcherContextEPNS_7GlobalsEPN3net6NetLogE
fun:_ZN8IOThread4InitEv
fun:_ZN7content17BrowserThreadImpl4InitEv
fun:_ZN7content21TestBrowserThreadImpl4InitEv
fun:_ZN4base6Thread10ThreadMainEv
fun:_ZN4base12_GLOBAL__N_110ThreadFuncEPv
}
BUG=542543
TBR=thestig@chromium.org
NOTRY=True
Review URL: https://codereview.chromium.org/1584353002
Cr-Commit-Position: refs/heads/master@{#369534}
|
|
|
|
|
|
|
|
|
|
|
| |
AccountInfo constructor now correctly initializes all its members.
Re-enable the affected tests under MSan.
BUG=577333
Review URL: https://codereview.chromium.org/1588913002
Cr-Commit-Position: refs/heads/master@{#369419}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:1 of https://codereview.chromium.org/1574393003/ )
Reason for revert:
It appears that these tests are still triggering errors:
https://build.chromium.org/p/chromium.memory.fyi/builders/Chromium%20OS%20%28valgrind%29%284%29/builds/36425
Original issue's description:
> Revert of Suppress uninit reports in Skia GPU (issue 576259) (patchset #1 id:1 of https://codereview.chromium.org/1579693003/ )
>
> Reason for revert:
> Fix has rolled into Chromium.
>
> Original issue's description:
> > Suppress uninit reports in Skia GPU (issue 576259)
> >
> > BUG=576259
> > TBR=reillyg@chromium.org
> > NOTRY=true
> >
> > Committed: https://crrev.com/bb3cc40c3dfac28bff5a2e8f968e037464ecc7ce
> > Cr-Commit-Position: refs/heads/master@{#368840}
>
> TBR=reillyg@chromium.org,glider@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=576259
>
> Committed: https://crrev.com/ac98f2d4b059a4f1aebba2a5a5fd11328dfe3587
> Cr-Commit-Position: refs/heads/master@{#369192}
TBR=glider@chromium.org,bsalomon@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=576259
Review URL: https://codereview.chromium.org/1583023002 .
Cr-Commit-Position: refs/heads/master@{#369274}
|
|
|
|
|
|
|
|
|
| |
BUG=577333
TBR=bzanotti@chromium.org
Review URL: https://codereview.chromium.org/1582133002 .
Cr-Commit-Position: refs/heads/master@{#369273}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
id:1 of https://codereview.chromium.org/1579693003/ )
Reason for revert:
Fix has rolled into Chromium.
Original issue's description:
> Suppress uninit reports in Skia GPU (issue 576259)
>
> BUG=576259
> TBR=reillyg@chromium.org
> NOTRY=true
>
> Committed: https://crrev.com/bb3cc40c3dfac28bff5a2e8f968e037464ecc7ce
> Cr-Commit-Position: refs/heads/master@{#368840}
TBR=reillyg@chromium.org,glider@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=576259
Review URL: https://codereview.chromium.org/1574393003
Cr-Commit-Position: refs/heads/master@{#369192}
|
|
|
|
|
|
|
|
|
|
|
|
| |
562718)
BUG=562718
TBR=reillyg@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1575363003
Cr-Commit-Position: refs/heads/master@{#368850}
|
|
|
|
|
|
|
|
|
|
| |
BUG=576259
TBR=reillyg@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1579693003
Cr-Commit-Position: refs/heads/master@{#368840}
|
|
|
|
|
|
|
|
|
|
| |
This shouldn't be relevant anymore since neither the WorkerPrioritySetters that were leaking, nor WorkerServiceImpl exist anymore.
BUG=245714
Review URL: https://codereview.chromium.org/1573263002
Cr-Commit-Position: refs/heads/master@{#368661}
|
|
|
|
|
|
|
|
|
|
| |
BUG=573604
TBR=eustas@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1553023002
Cr-Commit-Position: refs/heads/master@{#367264}
|
|
|
|
|
|
|
|
|
| |
BUG=571543
TBR=glider@chromium.org
Review URL: https://codereview.chromium.org/1548603002
Cr-Commit-Position: refs/heads/master@{#366553}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Could not reproduce Valgrind error locally. Suppression might not be
needed anymore. In any case, I think I know how to fix the
SequencedWorkerPool pattern of shutdown leak false alarms.
BUG=450228
Review URL: https://codereview.chromium.org/1412443007
Cr-Commit-Position: refs/heads/master@{#366440}
|
|
|
|
|
|
|
|
|
|
|
| |
Also widen a Valgrind suppression.
BUG=557993,571272
TBR=bruening@chromium.org
Review URL: https://codereview.chromium.org/1540153002
Cr-Commit-Position: refs/heads/master@{#366305}
|
|
|
|
|
|
|
|
|
| |
BUG=571278,571279,571280
TBR=bruening@chromium.org
Review URL: https://codereview.chromium.org/1537253002
Cr-Commit-Position: refs/heads/master@{#366300}
|
|
|
|
|
|
|
|
|
| |
BUG=558341,571271,571272
TBR=glider@chromium.org
Review URL: https://codereview.chromium.org/1540133002
Cr-Commit-Position: refs/heads/master@{#366298}
|
|
|
|
|
|
|
|
|
|
|
| |
DevTools source code was changed and mentioned stack traces can't happen.
BUG=388013
TBR=pfeldman@chromium.org
Review URL: https://codereview.chromium.org/1540673005
Cr-Commit-Position: refs/heads/master@{#366234}
|
|
|
|
|
|
|
|
|
| |
BUG=555798, 569736
TEST=valgrind locally passes
Review URL: https://codereview.chromium.org/1522273005
Cr-Commit-Position: refs/heads/master@{#365480}
|
|
|
|
|
|
|
|
|
| |
BUG=514439,521656,522463,522468,522514,522524,522620,542563,542875
TBR=reillyg@chromium.org
Review URL: https://codereview.chromium.org/1514573004
Cr-Commit-Position: refs/heads/master@{#364219}
|
|
|
|
|
|
|
|
|
|
|
|
| |
It looks like the stack trace through the blink::PaintLayer destructor
has changed slightly.
BUG=562718
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1518513004 .
Cr-Commit-Position: refs/heads/master@{#364158}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Comment #3 on issue 542543 wondered why we weren't seeing a leak report
for the globals_ object itself. Here it is.
BUG=542543
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1504123002
Cr-Commit-Position: refs/heads/master@{#363558}
|
|
|
|
|
|
|
|
|
|
| |
The class has been renamed for a while, so the suppressions cannot apply anymore.
BUG=195160
Review URL: https://codereview.chromium.org/1488613008
Cr-Commit-Position: refs/heads/master@{#362681}
|
|
|
|
|
|
|
|
|
|
| |
BUG=242672
TBR=benwells@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1487993002
Cr-Commit-Position: refs/heads/master@{#362378}
|
|
|
|
|
|
|
|
|
|
| |
BUG=234845
TBR=benwells@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1490733002
Cr-Commit-Position: refs/heads/master@{#362372}
|
|
|
|
|
|
|
|
|
|
|
| |
Also suppress a couple flaky errors.
BUG=515330,562712,562718
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1481233002
Cr-Commit-Position: refs/heads/master@{#362044}
|
|
|
|
|
|
|
|
|
| |
BUG=90487,562703
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1485523002
Cr-Commit-Position: refs/heads/master@{#362042}
|
|
|
|
|
|
|
|
|
| |
BUG=534718,561081,561085,561088
TBR=groby@chromium.org
Review URL: https://codereview.chromium.org/1469153005
Cr-Commit-Position: refs/heads/master@{#361496}
|
|
|
|
|
|
|
|
|
|
|
|
| |
proper section.
BUG=542575
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1467203002
Cr-Commit-Position: refs/heads/master@{#361091}
|
|
|
|
|
|
|
|
|
|
| |
BUG=558179
TBR=lazyboy@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1459053002
Cr-Commit-Position: refs/heads/master@{#360505}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This feature was previously requested by the mandoline project. Now the project doesn't need it anymore.
This reverts the following commits with conflict resolution:
da8f981cd01fc15b878c6bfd692adbd3c151855a
455f7e6bde0882fcc8fcfeac1abaeb35e28709e3
2338784958b986a1280d7aa67e34dc0766b746d2
BUG=498986
Review URL: https://codereview.chromium.org/1449553003
Cr-Commit-Position: refs/heads/master@{#360360}
|
|
|
|
|
|
|
|
|
|
| |
BUG=557778
TBR=marpan@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1455163003
Cr-Commit-Position: refs/heads/master@{#360340}
|
|
|
|
|
|
|
|
|
| |
BUG=510088,555798,556346
TEST=new unittests pass
Review URL: https://codereview.chromium.org/1447183004
Cr-Commit-Position: refs/heads/master@{#360206}
|
|
|
|
|
|
|
|
|
|
|
|
| |
The code path no longer exists, and crbug.com/387993 was fixed
six months ago.
R=halcanary@google.com,mtklein@google.com
BUG=387993
Review URL: https://codereview.chromium.org/1447913003
Cr-Commit-Position: refs/heads/master@{#359872}
|
|
|
|
|
|
|
|
|
| |
BUG=555798
TEST=valgrind passes locally
Review URL: https://codereview.chromium.org/1444523008
Cr-Commit-Position: refs/heads/master@{#359751}
|
|
|
|
|
|
|
|
|
|
|
|
| |
SequencedWorkerPool destroys itself asynchronously, so we
have to wrap it in a SequencedWorkerPoolOwner in unit tests
to avoid leaks.
BUG=550566
Review URL: https://codereview.chromium.org/1429113002
Cr-Commit-Position: refs/heads/master@{#357628}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
under Valgrind.
Leak_DefinitelyLost
1,580 (496 direct, 1,084 indirect) bytes in 1 blocks are definitely lost in loss record 12,324 of 12,407
operator new(unsigned long) (m_replacemalloc/vg_replace_malloc.c:1140)
base::SequencedWorkerPool::SequencedWorkerPool(unsigned long, std::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) (base/threading/sequenced_worker_pool.cc:1196)
base::(anonymous namespace)::SequencedTaskRunnerHandleTest_FromSequencedWorkerPool_Test::TestBody() (base/threading/sequenced_task_runner_handle_unittest.cc:54)
BUG=550566
TBR=bauerb@chromium.org
Review URL: https://codereview.chromium.org/1414333010
Cr-Commit-Position: refs/heads/master@{#357434}
|
|
|
|
|
|
|
|
|
| |
BUG=547391,64887
TBR=thestig@chromium.org
Review URL: https://codereview.chromium.org/1414833004
Cr-Commit-Position: refs/heads/master@{#355979}
|
|
|
|
|
|
|
|
|
|
|
| |
This suppresses memory leak warnings caused by r354658.
BUG=545259
TBR=nasko@chromium.org
Review URL: https://codereview.chromium.org/1413303002
Cr-Commit-Position: refs/heads/master@{#354935}
|
|
|
|
|
|
|
|
|
| |
TBR=haraken@chromium.org
BUG=340752, 521656
Review URL: https://codereview.chromium.org/1412523002
Cr-Commit-Position: refs/heads/master@{#354415}
|
|
|
|
|
|
|
|
| |
BUG=542563
Review URL: https://codereview.chromium.org/1409453002
Cr-Commit-Position: refs/heads/master@{#354357}
|
|
|
|
|
|
|
|
|
|
| |
Also widen an existing layout test error.
BUG=492821,521656,542875
Review URL: https://codereview.chromium.org/1391743005
Cr-Commit-Position: refs/heads/master@{#353886}
|
|
|
|
|
|
|
|
|
|
| |
Move the good suppressions into its place.
BUG=542502
Review URL: https://codereview.chromium.org/1403703004
Cr-Commit-Position: refs/heads/master@{#353818}
|
|
|
|
|
|
|
|
|
| |
BUG=542543,542563,542575
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1401893002
Cr-Commit-Position: refs/heads/master@{#353708}
|
|
|
|
|
|
|
|
|
|
|
|
| |
DrMemory: Writing to readonly memory error
Valgrind Mac: Leaks in ScopedSendingEventTest
BUG=542501,542502
TBR=thakis@chromium.org
Review URL: https://codereview.chromium.org/1399363003
Cr-Commit-Position: refs/heads/master@{#353676}
|
|
|
|
|
|
|
|
|
| |
BUG=517914
TBR=thestig
Review URL: https://codereview.chromium.org/1401653002
Cr-Commit-Position: refs/heads/master@{#353432}
|
|
|
|
|
|
|
|
|
| |
BUG=537626, 537265
TEST=run tests of valgrind and MSAN build without any error
Review URL: https://codereview.chromium.org/1376243004
Cr-Commit-Position: refs/heads/master@{#352335}
|
|
|
|
|
|
|
|
|
|
| |
It seems overly aggressive to suppress all of RenderProcessHostImpl::Init
BUG=536907
Review URL: https://codereview.chromium.org/1381313003
Cr-Commit-Position: refs/heads/master@{#352333}
|
|
|
|
|
|
|
|
|
|
| |
TBR=yoshiki@chromium.org
BUG=537626
NOTRY=true
Review URL: https://codereview.chromium.org/1375273002
Cr-Commit-Position: refs/heads/master@{#351566}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Valgrind bots
TBR=creis@chromium.org
BUG=536907
NOTRY=true
Review URL: https://codereview.chromium.org/1373973002
Cr-Commit-Position: refs/heads/master@{#351122}
|
|
|
|
|
|
|
|
|
| |
BUG=512466
TBR=yzshen@chromium.org
Review URL: https://codereview.chromium.org/1355413007
Cr-Commit-Position: refs/heads/master@{#350885}
|
|
|
|
|
|
|
|
|
|
|
| |
Many of these have been fixed already.
BUG=400547,402257,403774,403775,404182,416729,424056,424575,436110,457557,487706
TBR=zhaoqin@chromium.org
Review URL: https://codereview.chromium.org/1367853003
Cr-Commit-Position: refs/heads/master@{#350745}
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=mp3 fast seek still works.
Review URL: https://codereview.chromium.org/1362693002
Cr-Commit-Position: refs/heads/master@{#350278}
|