| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
| |
BUG=576259
TBR=bsalomon@chromium.org
Review URL: https://codereview.chromium.org/1813283002
Cr-Commit-Position: refs/heads/master@{#382082}
|
|
|
|
|
|
|
|
|
| |
BUG=595951
TBR=reed@chromium.org
Review URL: https://codereview.chromium.org/1815443003
Cr-Commit-Position: refs/heads/master@{#381877}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The DrMemory update process only runs when it thinks that it needs to
but its heuristic is not accurate. A better thing to do is to always
update.
This change also adds setlocal so that environment variables don't
leak outside. This may also help with making this script more robust
in the future, once it has been in place for long enough for all
machines affected by the environment variable leakage to have rebooted.
BUG=595867
R=scottmg@chromium.org
Review URL: https://codereview.chromium.org/1807013004 .
Cr-Commit-Position: refs/heads/master@{#381870}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In VS 2015 builds every call to realloc triggers a warning like this:
Error #1: INVALID HEAP ARGUMENT: allocated with Windows API layer, queried with C library layer
0 replace_malloc_usable_size
1 _recalloc_base
2 <lambda_4e60a939b0d047cfe11ddc22648dfba9>::operator()
3 __crt_seh_guarded_call<>::operator()<>
4 __acrt_lock_and_call<>
5 _register_onexit_function
6 _crt_atexit
7 _onexit
8 atexit
9 testing::`dynamic initializer for 'FLAGS_gmock_verbose''
This mismatch is harmless (because the CRT always uses the Windows heap)
and out of our control. Therefore it should be suppressed.
With this suppression I now get a clean run on this previously failing
test:
tools\valgrind\chrome_tests.bat -t content_browsertests --tool \
drmemory_light --build-dir=out\Release \
--gtest_filter=ManifestBrowserTest.DummyManifest
BUG=440500,594808
Review URL: https://codereview.chromium.org/1813063002
Cr-Commit-Position: refs/heads/master@{#381840}
|
|
|
|
|
|
|
|
|
| |
BUG=595490
TBR=zhaoqin@chromium.org
Review URL: https://codereview.chromium.org/1812963003
Cr-Commit-Position: refs/heads/master@{#381758}
|
|
|
|
|
|
|
|
|
| |
BUG=595490
TBR=zhaoqin@chromium.org
Review URL: https://codereview.chromium.org/1809883002
Cr-Commit-Position: refs/heads/master@{#381602}
|
|
|
|
|
|
|
|
|
| |
BUG=595408
TBR=bcwhite@chromium.org
Review URL: https://codereview.chromium.org/1813543002
Cr-Commit-Position: refs/heads/master@{#381592}
|
|
|
|
|
|
|
|
|
|
|
| |
been specified get a default value in tests.
BUG=508794
R=dmazzoni@chromium.org
Review URL: https://codereview.chromium.org/1806813003
Cr-Commit-Position: refs/heads/master@{#381568}
|
|
|
|
|
|
| |
Review URL: https://codereview.chromium.org/1783133002
Cr-Commit-Position: refs/heads/master@{#381565}
|
|
|
|
|
|
|
|
|
| |
BUG=595408
TBR=bcwhite@chromium.org
Review URL: https://codereview.chromium.org/1806043003
Cr-Commit-Position: refs/heads/master@{#381527}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This should cover the rest of the suppressions needed to clean up the
Dr. Memory bots.
BUG=595146,595149,595156,595158
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1807513003
Cr-Commit-Position: refs/heads/master@{#381355}
|
|
|
|
|
|
|
|
|
|
| |
BUG=595105
TBR=glider@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1805223002
Cr-Commit-Position: refs/heads/master@{#381338}
|
|
|
|
|
|
|
|
|
|
| |
BUG=595093
TBR=glider@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1800303002
Cr-Commit-Position: refs/heads/master@{#381311}
|
|
|
|
|
|
|
|
|
|
| |
BUG=571553
TBR=glider@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1804203002
Cr-Commit-Position: refs/heads/master@{#381298}
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changes to base::Bind changed the symbols.
BUG=588849
TBR=zea@chromium.org,stanisc@chromium.org,pavely@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1801253002
Cr-Commit-Position: refs/heads/master@{#381263}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Dr. Memory complains that a parameter to the NtFsControlFile system call
is not initialized.
BUG=595052
TBR=benwells@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1803063003
Cr-Commit-Position: refs/heads/master@{#381257}
|
|
|
|
|
|
|
|
|
|
| |
BUG=594829
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1796973005
Cr-Commit-Position: refs/heads/master@{#381122}
|
|
|
|
|
|
|
|
|
|
| |
BUG=594808
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1802823002
Cr-Commit-Position: refs/heads/master@{#381106}
|
|
|
|
|
|
|
|
|
|
| |
BUG=594781,594785
TBR=zhaoqin@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1802143002
Cr-Commit-Position: refs/heads/master@{#381104}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
MSVC120.dll has been replaced by ucrtbase.dll and the stack is slightly
different.
BUG=508794
TBR=zhaoqin@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1804713002
Cr-Commit-Position: refs/heads/master@{#381056}
|
|
|
|
|
|
|
|
|
|
| |
BUG=594657
TBR=zhaoqin@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1799833002
Cr-Commit-Position: refs/heads/master@{#381051}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
These errors have been seen coming from more modules than just cookie
storage.
BUG=594618
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1801473004
Cr-Commit-Position: refs/heads/master@{#381017}
|
|
|
|
|
|
|
|
|
|
| |
BUG=594614,594618
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1795303003
Cr-Commit-Position: refs/heads/master@{#381009}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Followup to https://codereview.chromium.org/1771283002/
We must clear request_context_getter_ on the IO thread or the valgrind
bots cry. Also, call RemoveObserver in ShutdownOnIO in case OnContextIsShuttingDown
is never called, as is the case on content_shell.
BUG=587621,593913
Review URL: https://codereview.chromium.org/1779353003
Cr-Commit-Position: refs/heads/master@{#380924}
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GpuScheduler has been a misnomer for what the class does. This CL is a
simple refactoring to rename gpu::GpuScheduler to gpu::CommandExecutor.
This allows the GpuScheduler name to be used for the upcoming scheduler
and at the very least avoid confusion.
BUG=514813
CQ_INCLUDE_TRYBOTS=tryserver.chromium.win:win_optional_gpu_tests_rel;tryserver.chromium.mac:mac_optional_gpu_tests_rel
Review URL: https://codereview.chromium.org/1794463004
Cr-Commit-Position: refs/heads/master@{#380851}
|
|
|
|
|
|
|
|
|
| |
BUG=593913,593594
TBR=reillyg@chromium.org
Review URL: https://codereview.chromium.org/1785873002
Cr-Commit-Position: refs/heads/master@{#380496}
|
|
|
|
|
|
|
|
| |
BUG=570102
Review URL: https://codereview.chromium.org/1784643002
Cr-Commit-Position: refs/heads/master@{#380472}
|
|
|
|
|
|
|
|
|
| |
BUG=86301
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1784643004
Cr-Commit-Position: refs/heads/master@{#380346}
|
|
|
|
|
|
|
|
|
|
|
| |
Also format suppressions_linux.txt to match other suppression files.
BUG=386418,415092,588849
TBR=reillyg@chromium.org
Review URL: https://codereview.chromium.org/1778213002
Cr-Commit-Position: refs/heads/master@{#380344}
|
|
|
|
|
|
|
|
|
| |
BUG=431213
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1778183002
Cr-Commit-Position: refs/heads/master@{#380319}
|
|
|
|
|
|
|
|
|
| |
BUG=585942
R=ben@chromium.org
Review URL: https://codereview.chromium.org/1765243002 .
Cr-Commit-Position: refs/heads/master@{#379839}
|
|
|
|
|
|
|
|
|
|
|
|
| |
- a Braille test which is flaky on ChromeOS MSAN
- loads of tests which are timing out on DrMemory.
TBR=thestig@chromium.org
BUG=592900, 592893
Review URL: https://codereview.chromium.org/1777463002
Cr-Commit-Position: refs/heads/master@{#379779}
|
|
|
|
|
|
|
|
|
| |
TBR=thestig@chromium.org
BUG=592320
Review URL: https://codereview.chromium.org/1767023002
Cr-Commit-Position: refs/heads/master@{#379520}
|
|
|
|
|
|
|
|
|
| |
BUG=131094
TBR=isherman@chromium.org
Review URL: https://codereview.chromium.org/1759903002
Cr-Commit-Position: refs/heads/master@{#378857}
|
|
|
|
|
|
|
|
|
| |
BUG=562431
TBR=task@google.com
Review URL: https://codereview.chromium.org/1755203002
Cr-Commit-Position: refs/heads/master@{#378787}
|
|
|
|
|
|
|
|
|
| |
BUG=562431
TBR=task@google.com
Review URL: https://codereview.chromium.org/1758483002
Cr-Commit-Position: refs/heads/master@{#378617}
|
|
|
|
|
|
|
|
|
| |
BUG=576251
TBR=sergeyu@chromium.org
Review URL: https://codereview.chromium.org/1757483003
Cr-Commit-Position: refs/heads/master@{#378498}
|
|
|
|
|
|
|
|
|
| |
BUG=591092
TBR=benwells@chromium.org
Review URL: https://codereview.chromium.org/1753833002
Cr-Commit-Position: refs/heads/master@{#378470}
|
|
|
|
|
|
|
|
|
|
| |
These symbols don't appear in the codebase anymore.
BUG=42842
Review URL: https://codereview.chromium.org/1740023003
Cr-Commit-Position: refs/heads/master@{#377827}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Valgrind bots seem having more overhead than other try bots.
It looks like when refresh interval is too small. The first
refresh of the second list has overlap with the second
refresh of the first list.
BUG=582372
Review URL: https://codereview.chromium.org/1729663003
Cr-Commit-Position: refs/heads/master@{#377387}
|
|
|
|
|
|
|
|
|
|
| |
BUG=586668
TBR=rockot@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1724313002
Cr-Commit-Position: refs/heads/master@{#377174}
|
|
|
|
|
|
|
|
|
|
|
|
| |
This test is being flaky.
BUG=589174
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1724073002
Cr-Commit-Position: refs/heads/master@{#377065}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Remove base::internal::RunnableAdapter<>::Run from the stack trace as it
is unstable.
BUG=571551
TBR=thestig@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1719903003
Cr-Commit-Position: refs/heads/master@{#376886}
|
|
|
|
|
|
|
|
|
|
| |
BUG=588849
TBR=zea@chromium.org,stanisc@chromium.org,pavely@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1722533003
Cr-Commit-Position: refs/heads/master@{#376839}
|
|
|
|
|
|
|
|
|
|
| |
As we fixed the bug, so we can remove the suppression and verify whether the fix works.
BUG=571280
Review URL: https://codereview.chromium.org/1725513002
Cr-Commit-Position: refs/heads/master@{#376835}
|
|
|
|
|
|
|
|
| |
BUG=542575, 573604
Review URL: https://codereview.chromium.org/1554853002
Cr-Commit-Position: refs/heads/master@{#376790}
|
|
|
|
|
|
|
|
|
|
| |
BUG=588788
TBR=dtseng@chromium.org,dmazzoni@chromium.org
NOTRY=true
Review URL: https://codereview.chromium.org/1724463002
Cr-Commit-Position: refs/heads/master@{#376770}
|
|
|
|
|
|
|
|
|
| |
BUG=542543
TBR=thestig@chromium.org
Review URL: https://codereview.chromium.org/1717623003
Cr-Commit-Position: refs/heads/master@{#376589}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ProcessUtilTest.MacMallocFailureDoesNotTerminate.
The test does not exist anymore.
BUG=122080
R=dcheng@chromium.org
Review URL: https://codereview.chromium.org/1717813002 .
Cr-Commit-Position: refs/heads/master@{#376548}
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In particular the method SkPictureRecord::getFlatPaintData appears
in the stack trace of this suppression, but this method no longer
exists, as it was removed with
https://skia.googlesource.com/skia/+/46616af01b412ea984a516fda1ed8ec08e689f29
BUG=chromium:381156
Review URL: https://codereview.chromium.org/1709123002
Cr-Commit-Position: refs/heads/master@{#376195}
|