Back to build
Raw log | Switch to full mode | Login

/b/s/w/ir/cache/builder/src/build/tree_truth.sh /b/s/w/ir/cache/builder/src src in dir /b/s/w/ir/k: allow_subannotations: False cmd: ['/b/s/w/ir/cache/builder/src/build/tree_truth.sh', '/b/s/w/ir/cache/builder/src', 'src'] env: FrozenDict([('CHROME_HEADLESS', '1')]) env_prefixes: {'mapping': FrozenDict([]), 'pathsep': ':'} env_suffixes: {'mapping': FrozenDict([]), 'pathsep': ':'} infra_step: False name_tokens: ['tree truth steps'] ok_ret: frozenset([0]) trigger_specs: [] full environment: BOTO_CONFIG: /b/s/w/ir/tmp/gsutil-task/.boto BUILDBUCKET_EXPERIMENTAL: FALSE CHROME_HEADLESS: 1 CIPD_CACHE_DIR: /b/s/cipd_cache/cache CIPD_PROTOCOL: v2 DEVSHELL_CLIENT_PORT: 34369 DOCKER_CONFIG: /b/s/w/ir/tmp/docker-cfg-task DOCKER_TMPDIR: /b/s/w/ir/tmp/docker-tmp-task GIT_CONFIG_NOSYSTEM: 1 GIT_TERMINAL_PROMPT: 0 HOME: /home/chrome-bot INFRA_GIT_WRAPPER_HOME: /b/s/w/ir/tmp/git-home-task LOGDOG_COORDINATOR_HOST: logs.chromium.org LOGDOG_STREAM_PREFIX: buildbucket/cr-buildbucket.appspot.com/8915175036385407984 LOGDOG_STREAM_PROJECT: chromium LOGDOG_STREAM_SERVER_PATH: unix:/b/s/w/ir/tmp/ld.sock LOGNAME: chrome-bot LUCI_CONTEXT: /b/s/w/ir/tmp/luci_context.733726127 MAC_CHROMIUM_TMPDIR: /b/s/w/ir/tmp/t MAIL: /var/mail/chrome-bot NO_GCE_CHECK: False PATH: /b/s/w/ir/cipd_bin_packages:/b/s/w/ir/cipd_bin_packages/bin:/b/s/cipd_cache/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin PWD: /b/s/w/ir/k PYTHONIOENCODING: UTF-8 PYTHONUNBUFFERED: 1 SHELL: /bin/bash SHLVL: 1 SUDO_COMMAND: /usr/bin/python /b/s/swarming_bot.zip start_bot SUDO_GID: 0 SUDO_UID: 0 SUDO_USER: root SWARMING_BOT_ID: luci-chromium-ci-trusty-8-1-tqz7 SWARMING_EXTERNAL_BOT_SETUP: 1 SWARMING_HEADLESS: 1 SWARMING_SERVER: https://chromium-swarm.appspot.com SWARMING_TASK_ID: 446ef8a2e2062811 TEMP: /b/s/w/ir/tmp/t TEMPDIR: /b/s/w/ir/tmp/t TERM: linux TMP: /b/s/w/ir/tmp/t TMPDIR: /b/s/w/ir/tmp/t USER: chrome-bot USERNAME: chrome-bot VPYTHON_VIRTUALENV_ROOT: /b/s/w/ir/cache/vpython _: /b/s/w/ir/cipd_bin_packages/vpython !@@@BUILD_STEP Brief summary of recent CLs in every branch@@@ src: a11eba89f2630c1a1c7505513bcd624af1058b07 Fix UAF in NavigationPredictor test. Alexei Svitkine, Thu Apr 25 20:17:22 2019 +0000 f774bd5c23bee52741419cac25de850955c32010 [omnibox] Update the icon's colors on theme change Kevin Bailey, Thu Apr 25 20:16:50 2019 +0000 0f1cb59fce46e5ab8a8ed4124ec26a1b3ab41195 [Animation Worklet] Support effects with no target Yi Gu, Thu Apr 25 20:15:11 2019 +0000 867e4320ab34b0f24128d334dbbac0965973d57d Disable the flaky WMI tests Sebastien Marchand, Thu Apr 25 20:12:40 2019 +0000 62fb8a8ff932e7e83912c5f6aaf94a09115983bc [ios] Add displayBadge: and animated: method parameter to InfoBarBadgeButton Chris Lu, Thu Apr 25 20:10:17 2019 +0000 d9ac2f15afb3ea64d99cf6d15a2fd8ee740428fa Revert "[Touchless] Update placeholder text and add special case for first run" Brandon Wylie, Thu Apr 25 20:09:57 2019 +0000 e53db1f02b29d568f5a5101a62ef4a8a7da91d96 Reland "Make Zombies ignore GWP-ASan backed allocations" Vlad Tsyrklevich, Thu Apr 25 20:06:19 2019 +0000 6ae342a3f37127542b2009d9168b21b4e3d2099f Adding field trial testing config for manual translate trigger. Alexandre Frechette, Thu Apr 25 20:04:37 2019 +0000 83cbc0f44ae5b5316c404fce43c0a93527cd81ed viz: Safe downcast from OutputSurface to SkiaOutputSurface Sunny Sachanandani, Thu Apr 25 20:03:37 2019 +0000 ad14b36f443aa4e4f1661ecf7d4c1d20dfc3a5dd Roll src/third_party/swiftshader 3c203531c7d7..244660437f4b (3 commits) chromium-autoroll, Thu Apr 25 19:58:12 2019 +0000 ================================================================= !@@@BUILD_STEP New commits in repo src@@@ commit a11eba89f2630c1a1c7505513bcd624af1058b07 Author: Alexei Svitkine <asvitkine@chromium.org> Date: Thu Apr 25 20:17:22 2019 +0000 Fix UAF in NavigationPredictor test. The test was initializing two nested ScopedFeatureLists in a way that caused UAF on FieldTrial objects. The nested lists are not actually required and this CL changes the test not to do this. In separate CL, I'm going to look at a way to prevent this issue to begin with, for example via a DCHECK, but this requires seeing if other tests may be affected. Bug: 955643 Change-Id: Ic63543786b45b0fccde51d1287f42de23f183765 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584889 Reviewed-by: Tarun Bansal <tbansal@chromium.org> Reviewed-by: Bruce Dawson <brucedawson@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#654169} .../navigation_predictor_unittest.cc | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) commit f774bd5c23bee52741419cac25de850955c32010 Author: Kevin Bailey <krb@chromium.org> Date: Thu Apr 25 20:16:50 2019 +0000 [omnibox] Update the icon's colors on theme change Icons, like a bookmark star, weren't getting updated when flipping between light and dark e.g. on a Mac. This CL changes the behavior to update them on a theme change. Bug: 935265 Change-Id: I3b7a3827690c76815772a45c385ce08fa7dce73d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584401 Reviewed-by: Tommy Li <tommycli@chromium.org> Commit-Queue: Kevin Bailey <krb@chromium.org> Cr-Commit-Position: refs/heads/master@{#654168} chrome/browser/ui/views/location_bar/location_bar_view.cc | 2 ++ 1 file changed, 2 insertions(+) commit 0f1cb59fce46e5ab8a8ed4124ec26a1b3ab41195 Author: Yi Gu <yigu@chromium.org> Date: Thu Apr 25 20:15:11 2019 +0000 [Animation Worklet] Support effects with no target Keyframe effect with no target element is allowed in WebAnimation spec [1]. This patch adds the support and makes sure that adding or removing the target while the animation is running doesn't affect the result. [1] https://drafts.csswg.org/web-animations/#dom-keyframeeffect-target Bug: 781816 Change-Id: I23c5c005e61bb988c51aba60d07fe3f2589dbff1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572260 Commit-Queue: Yi Gu <yigu@chromium.org> Reviewed-by: Majid Valipour <majidvp@chromium.org> Cr-Commit-Position: refs/heads/master@{#654167} .../modules/animationworklet/worklet_animation.cc | 39 +++++++---- .../modules/animationworklet/worklet_animation.h | 5 +- .../worklet-animation-creation.https.html | 15 ----- .../worklet-animation-without-target.https.html | 76 ++++++++++++++++++++++ 4 files changed, 105 insertions(+), 30 deletions(-) commit 867e4320ab34b0f24128d334dbbac0965973d57d Author: Sebastien Marchand <sebmarchand@chromium.org> Date: Thu Apr 25 20:12:40 2019 +0000 Disable the flaky WMI tests This also remove one usage of a base::win::ScopedCOMInitializer(kMTA) from the MetricEvaluatorsHelperWinTest test fixture as it's apparently not good to do this (according to etienneb@ in this CL: https://chromium-review.googlesource.com/c/chromium/src/+/1582322). Bug: 956638 Change-Id: I67039e8fb0349561156fc5cdd6f5a8d6ad4dab21 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584820 Auto-Submit: Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by: Etienne Bergeron <etienneb@chromium.org> Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Cr-Commit-Position: refs/heads/master@{#654166} .../metric_evaluator_helper_win_unittest.cc | 18 +++++++++++------- .../performance_monitor/wmi_refresher_unittest.cc | 5 ++++- 2 files changed, 15 insertions(+), 8 deletions(-) commit 62fb8a8ff932e7e83912c5f6aaf94a09115983bc Author: Chris Lu <thegreenfrog@chromium.org> Date: Thu Apr 25 20:10:17 2019 +0000 [ios] Add displayBadge: and animated: method parameter to InfoBarBadgeButton - displayBadge: method allows for showing and hiding the badge - Adds animated: param to other methods to follow proper convention. Bug: 954624 Change-Id: I0702f841478ff7a9a2b4ab68ff1b15b475388519 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584379 Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Reviewed-by: Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#654165} .../ui/infobars/badge/infobar_badge_button.h | 11 ++-- .../ui/infobars/badge/infobar_badge_button.mm | 60 ++++++++++++++++------ 2 files changed, 51 insertions(+), 20 deletions(-) commit d9ac2f15afb3ea64d99cf6d15a2fd8ee740428fa Author: Brandon Wylie <wylieb@chromium.org> Date: Thu Apr 25 20:09:57 2019 +0000 Revert "[Touchless] Update placeholder text and add special case for first run" This reverts commit 91720b9ea5770a6167b0410264dccdee854c26ef. Reason for revert: Crashes on ToT. Original change's description: > [Touchless] Update placeholder text and add special case for first run > > Bug: 954529 > Change-Id: I9cba7d40d2bc024226274debf1ffc1c5e60b5241 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574297 > Reviewed-by: Matthew Jones <mdjones@chromium.org> > Reviewed-by: Justin DeWitt <dewittj@chromium.org> > Commit-Queue: Brandon Wylie <wylieb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#654064} TBR=dewittj@chromium.org,mdjones@chromium.org,wylieb@chromium.org Change-Id: Ie197a82e1285a21fa47d9fadef47db757f971918 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 954529 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1585105 Reviewed-by: Brandon Wylie <wylieb@chromium.org> Commit-Queue: Brandon Wylie <wylieb@chromium.org> Cr-Commit-Position: refs/heads/master@{#654164} .../chromium/chrome/browser/touchless/OpenLastTabMediator.java | 10 ---------- .../chrome/browser/touchless/OpenLastTabProperties.java | 3 --- .../org/chromium/chrome/browser/touchless/OpenLastTabView.java | 9 --------- .../chrome/browser/touchless/OpenLastTabViewBinder.java | 2 -- chrome/android/touchless/java/strings/touchless_strings.grd | 7 ++----- 5 files changed, 2 insertions(+), 29 deletions(-) commit e53db1f02b29d568f5a5101a62ef4a8a7da91d96 Author: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Date: Thu Apr 25 20:06:19 2019 +0000 Reland "Make Zombies ignore GWP-ASan backed allocations" This is a reland of 28000bb230411bf866115fe2e3695721db67bea7 with a fix for a gn gen --check failure for an #include in an #if block. Original change's description: > Make Zombies ignore GWP-ASan backed allocations > > Zombies is an macOS-specific tool for detecting Objective-C > use-after-frees by delaying Obj-C object deallocation and replacing > those objects' class information with a debug class that will cause a > crash on any attempt to call a method on the freed object. > > Zombies interferes with GWP-ASan on account of the fact that by delaying > the object's deallocation GWP-ASan does not get a correct deallocation > stack trace, nor can it detect errors during the time the object is in > Zombies' quarantine. > > Change Zombies to immediately deallocate Obj-C objects backed by a > GWP-ASan allocation. > > Bug: 944727 > Change-Id: Ifa452bce7f6678ed20a4cc51000748d11d366a59 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1570631 > Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org> > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Reviewed-by: Vitaly Buka <vitalybuka@chromium.org> > Commit-Queue: Vlad Tsyrklevich <vtsyrklevich@chromium.org> > Cr-Commit-Position: refs/heads/master@{#653844} TBR=rsesek@chromium.org,vitalybuka@chromium.org Bug: 944727 Change-Id: I4f28294d41c43f4585916ec0e4786a9392f74813 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584369 Reviewed-by: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Commit-Queue: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Cr-Commit-Position: refs/heads/master@{#654163} components/crash/core/common/BUILD.gn | 6 ++++++ components/crash/core/common/DEPS | 2 ++ components/crash/core/common/objc_zombie.mm | 15 +++++++++++++-- components/gwp_asan/client/sampling_allocator_shims.cc | 5 +++++ components/gwp_asan/client/sampling_allocator_shims.h | 6 ++++++ 5 files changed, 32 insertions(+), 2 deletions(-) commit 6ae342a3f37127542b2009d9168b21b4e3d2099f Author: Alexandre Frechette <frechette@chromium.org> Date: Thu Apr 25 20:04:37 2019 +0000 Adding field trial testing config for manual translate trigger. Change-Id: I1ca332c0a4557039609345ca13e2c40ee759ca8b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572585 Reviewed-by: Nik Bhagat <nikunjb@chromium.org> Commit-Queue: Alexandre Frechette <frechette@chromium.org> Cr-Commit-Position: refs/heads/master@{#654162} testing/variations/fieldtrial_testing_config.json | 15 +++++++++++++++ 1 file changed, 15 insertions(+) commit 83cbc0f44ae5b5316c404fce43c0a93527cd81ed Author: Sunny Sachanandani <sunnyps@chromium.org> Date: Thu Apr 25 20:03:37 2019 +0000 viz: Safe downcast from OutputSurface to SkiaOutputSurface We need to instantiate Display with both OutputSurface and an equivalent pointer to SkiaOutputSurface. This CL provides a way to safely downcast from an OutputSurface to SkiaOutputSurface to decouple the creation of OutputSurface and Display in preparation for upcoming CLs. Bug: 953970 Change-Id: I7d25d980e208ccb39aa3f4e0eee3b9a662ccd502 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1583049 Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Reviewed-by: Bo <boliu@chromium.org> Reviewed-by: kylechar <kylechar@chromium.org> Cr-Commit-Position: refs/heads/master@{#654161} android_webview/browser/gfx/surfaces_instance.cc | 5 +---- components/viz/service/display/display.cc | 5 ++--- components/viz/service/display/display.h | 5 ++--- components/viz/service/display/output_surface.cc | 4 ++++ components/viz/service/display/output_surface.h | 4 ++++ components/viz/service/display/skia_output_surface.cc | 4 ++++ components/viz/service/display/skia_output_surface.h | 2 ++ components/viz/service/display_embedder/gpu_display_provider.cc | 5 +---- components/viz/test/test_layer_tree_frame_sink.cc | 4 +--- 9 files changed, 21 insertions(+), 17 deletions(-) commit ad14b36f443aa4e4f1661ecf7d4c1d20dfc3a5dd Author: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Date: Thu Apr 25 19:58:12 2019 +0000 Roll src/third_party/swiftshader 3c203531c7d7..244660437f4b (3 commits) https://swiftshader.googlesource.com/SwiftShader.git/+log/3c203531c7d7..244660437f4b git log 3c203531c7d7..244660437f4b --date=short --no-merges --format='%ad %ae %s' 2019-04-25 chrisforbes@google.com Partial support for input attachments 2019-04-25 chrisforbes@google.com Fix weird sampled image format query 2019-04-25 chrisforbes@google.com Produce correct `one` value for integer textures Created with: gclient setdep -r src/third_party/swiftshader@244660437f4b The AutoRoll server is located here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:b/131171141,chromium:b/117974925,chromium:b/129523279 TBR=swiftshader-team+autoroll@chromium.org Change-Id: Ic0e63b9ed522df9ef0b07b03346bb2dc88a91601 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584761 Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#654160} DEPS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit 7291c4a2de9765287abf4e65a65b06cab2f8eeef Author: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Date: Thu Apr 25 19:57:19 2019 +0000 Roll src-internal 34471d8b8ea8..0b6a22a105de (2 commits) https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/34471d8b8ea8..0b6a22a105de Created with: gclient setdep -r src-internal@0b6a22a105de The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=cmumford@chromium.org,gab@chromium.org,droger@chromium.org Change-Id: I3e0c7fb1f2eb2f16287d4d33ba5ea988ce96a318 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584762 Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#654159} DEPS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit f123e2c7a60d1390077cf4f9bdbe7a98581da504 Author: Fabrice de Gans-Riberi <fdegans@chromium.org> Date: Thu Apr 25 19:57:08 2019 +0000 [fuchsia] Remove stale references to webrunner. The //fuchsia directory should no longer use "webrunner". Change some of the remaining references to "web_engine" or "WebEngine". Bug: 922635 Change-Id: I0fd8beb838c76a0a39b54c4aa78f0d73194ae02a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1579779 Reviewed-by: Sergey Ulanov <sergeyu@chromium.org> Commit-Queue: Fabrice de Gans-Riberi <fdegans@chromium.org> Cr-Commit-Position: refs/heads/master@{#654158} fuchsia/BUILD.gn | 2 +- fuchsia/engine/BUILD.gn | 4 ++-- fuchsia/engine/browser/web_engine_net_log.cc | 2 +- fuchsia/engine/web_engine_main_delegate.cc | 2 +- 4 files changed, 5 insertions(+), 5 deletions(-) commit 0e602ef4055141ec70efa920cc5b9c7b9a62fc16 Author: Chris Hamilton <chrisha@chromium.org> Date: Thu Apr 25 19:55:26 2019 +0000 [SR] Fix a use-after-free in session restore logic. The callback being invoked can cause the retaining object to delete itself and the SessionRestorePolicy object, at which point setting the |notification_state_| property is invalid. BUG=946863 Change-Id: I54b9b3890eb729c541472c829476458511bb8d45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584370 Reviewed-by: Sébastien Marchand <sebmarchand@chromium.org> Commit-Queue: Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#654157} chrome/browser/resource_coordinator/session_restore_policy.cc | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) commit 0c4647477f02931733608d680eedeaf5c51fff4a Author: Sergiy Belozorov <sergiyb@chromium.org> Date: Thu Apr 25 19:53:19 2019 +0000 Implement a feature allowing to add all missing URLs to the WPR archive R=perezju@chromium.org Bug: 895891 Change-Id: Ifdcc5876e170a43166050a4d743a5d89e7a04962 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1511426 Commit-Queue: Sergiy Belozorov <sergiyb@chromium.org> Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org> Cr-Commit-Position: refs/heads/master@{#654156} tools/perf/cli_tools/update_wpr/update_wpr.py | 135 ++++++++++++++------- .../cli_tools/update_wpr/update_wpr_unittest.py | 66 +++++++--- tools/perf/core/path_util.py | 8 +- tools/perf/core/services/request.py | 6 +- 4 files changed, 150 insertions(+), 65 deletions(-) commit 26d47ce236b7f979e858a56cefe88747355c6ed6 Author: Alex Cooper <alcooper@chromium.org> Date: Thu Apr 25 19:51:44 2019 +0000 Update presubmit script to allow modifications to webxr_test_pages The webxr_test_pages started out as a sample pages from the immersive web working group, and such have to live in third_party; however these pages are now forked from that so that they can serve as manual test pages in-sync with the chrome implementation of the webxr spec for developers working on the spec. Because these changes are specific to chrome, they don't fit under web_platform_tests, and because they were originally third_party sample pages, they have to live in third_party. Change-Id: If5e09667bff8124b645e020f6e88824e7b848c0b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1554129 Reviewed-by: Nico Weber <thakis@chromium.org> Reviewed-by: Bill Orr <billorr@chromium.org> Commit-Queue: Alexander Cooper <alcooper@chromium.org> Cr-Commit-Position: refs/heads/master@{#654155} third_party/PRESUBMIT.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) commit 2c4df45d5c8376634ec9d599929e09673a4f5745 Author: Ken Rockot <rockot@google.com> Date: Thu Apr 25 19:50:36 2019 +0000 Convert remaining audio service to new Mojo types Update services/audio and closely related code to use new Mojo bindings types instead of old ones. This is a follow-up to https://chromium-review.googlesource.com/c/chromium/src/+/1574452 which started the conversion process. Bug: 955171 Change-Id: I61907cb5eb7a39cad121dada6c17570fdf3f7948 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1578019 Reviewed-by: Alex Moshchuk <alexmos@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Reviewed-by: Olga Sharonova <olka@chromium.org> Commit-Queue: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#654154} .../browser/media/audio_loopback_stream_broker.cc | 5 +- .../media/audio_loopback_stream_broker_unittest.cc | 19 ++--- content/browser/media/audio_muting_session.cc | 4 +- content/browser/media/audio_muting_session.h | 3 +- .../forwarding_audio_stream_factory_unittest.cc | 25 ++++--- ...er_frame_audio_input_stream_factory_unittest.cc | 6 +- content/browser/speech/speech_recognizer_impl.cc | 7 +- content/public/test/audio_service_test_helper.cc | 20 +++--- content/public/test/audio_service_test_helper.h | 16 +++-- content/public/utility/content_utility_client.h | 4 +- .../shell/utility/shell_content_utility_client.cc | 4 +- .../shell/utility/shell_content_utility_client.h | 2 +- content/utility/utility_service_factory.cc | 10 +-- content/utility/utility_service_factory.h | 5 +- services/audio/local_muter.cc | 2 +- services/audio/local_muter.h | 2 +- services/audio/local_muter_unittest.cc | 4 +- services/audio/log_factory_manager_unittest.cc | 19 ++--- services/audio/loopback_stream.cc | 16 ++--- services/audio/loopback_stream.h | 34 +++++---- services/audio/loopback_stream_unittest.cc | 50 +++++++------ services/audio/public/cpp/device_factory.cc | 4 +- services/audio/public/cpp/device_factory.h | 3 +- services/audio/public/cpp/fake_stream_factory.h | 8 +-- services/audio/public/cpp/fake_system_info.cc | 2 +- services/audio/public/cpp/fake_system_info.h | 4 +- services/audio/public/cpp/input_ipc.cc | 38 ++++------ services/audio/public/cpp/input_ipc.h | 19 ++--- services/audio/public/mojom/stream_factory.mojom | 15 ++-- services/audio/service.cc | 70 +++++++++--------- services/audio/service.h | 26 ++++--- services/audio/service_factory.cc | 10 +-- services/audio/service_factory.h | 13 ++-- services/audio/service_main.cc | 6 +- services/audio/stream_factory.cc | 18 ++--- services/audio/stream_factory.h | 10 +-- services/audio/stream_factory_unittest.cc | 8 +-- services/audio/system_info.cc | 4 +- services/audio/system_info.h | 10 +-- .../test/audio_system_to_service_adapter_test.cc | 84 +++++++++++----------- .../audio/test/debug_recording_session_unittest.cc | 14 ++-- services/audio/test/in_process_service_test.cc | 3 +- .../audio/test/service_lifetime_connector_test.cc | 6 +- .../audio/test/service_lifetime_test_template.h | 8 ++- services/audio/test/service_observer_mock.cc | 7 +- services/audio/test/service_observer_mock.h | 8 ++- 46 files changed, 346 insertions(+), 309 deletions(-) commit 1e19262f95a23b28e5b4564a8cadc67028e96e8d Author: Bruno Santos <brunoad@chromium.org> Date: Thu Apr 25 19:49:37 2019 +0000 Use internal manifest for Kiosk Next Home. Also, protects parts of code that should only run if Kiosk Next is enabled. Bug: 927809 Change-Id: I4ad58d9ac5c75c64440714021b1ffb86d7e609e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1580079 Reviewed-by: Tommy Martino <tmartino@chromium.org> Reviewed-by: Devlin <rdevlin.cronin@chromium.org> Reviewed-by: Michael Giuffrida <michaelpg@chromium.org> Commit-Queue: Bruno Santos <brunoad@chromium.org> Cr-Commit-Position: refs/heads/master@{#654153} chrome/browser/BUILD.gn | 8 ++++++++ chrome/browser/browser_resources.grd | 11 ++++++++++- chrome/browser/chromeos/kiosk_next_home/kiosk_next.gni | 4 ++++ chrome/browser/extensions/BUILD.gn | 8 ++++++++ .../chrome_component_extension_resource_manager.cc | 2 +- .../extensions/chrome_extensions_interface_registration.cc | 14 +++++++++++--- .../extensions/component_extensions_whitelist/whitelist.cc | 4 +++- chrome/browser/extensions/component_loader.cc | 12 +++++++----- chrome/browser/resources/BUILD.gn | 8 ++++++++ chrome/browser/resources/component_extension_resources.grd | 6 ++++-- chrome/browser/ui/BUILD.gn | 8 ++++++++ chrome/browser/ui/startup/startup_browser_creator.cc | 2 +- chrome/browser/ui/webui/settings/settings_ui.cc | 3 ++- 13 files changed, 75 insertions(+), 15 deletions(-) commit 36d4868dc0316b9f1fbef6b0a56c0edf5f46c27c Author: Sergiy Belozorov <sergiyb@chromium.org> Date: Thu Apr 25 19:48:12 2019 +0000 Default to reuse option when branch name looks like one created by the script Also offer to do cleanup on exception. This way if script crashes, users can still do cleanup. R=perezju@chromium.org Bug: 895891 Change-Id: Ibd137b46a60a10753bd2be8de5afc4976a03bd58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1520600 Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: Sergiy Belozorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#654152} tools/perf/cli_tools/update_wpr/update_wpr.py | 56 ++++++++++++---------- .../cli_tools/update_wpr/update_wpr_unittest.py | 5 +- 2 files changed, 34 insertions(+), 27 deletions(-) commit fa25101faf091614edb81cc34ffba810e10f3a7b Author: Xianzhu Wang <wangxianzhu@chromium.org> Date: Thu Apr 25 19:46:50 2019 +0000 Move NG Physical/Logical Offset/Size/Rect out of ng - from layout/ng/geometry to layout/geometry - remove NG prefix - rename NGPhysicalOffsetRect to PhysicalRect. Change-Id: Ib04d8d65ea6c7b95444708ef5abc397c60c8f4ab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1582872 Reviewed-by: Bruce Dawson <brucedawson@chromium.org> Reviewed-by: Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#654151} third_party/blink/renderer/core/BUILD.gn | 8 +- .../renderer/core/editing/visible_units_line.cc | 4 +- third_party/blink/renderer/core/layout/BUILD.gn | 24 +- .../core/layout/geometry/logical_offset.cc | 58 ++++ .../logical_offset.h} | 52 ++-- .../logical_offset_test.cc} | 16 +- .../logical_rect.cc} | 14 +- .../renderer/core/layout/geometry/logical_rect.h | 56 ++++ .../logical_rect_test.cc} | 18 +- .../logical_size.cc} | 6 +- .../renderer/core/layout/geometry/logical_size.h | 86 ++++++ .../core/layout/geometry/physical_offset.cc | 52 ++++ .../physical_offset.h} | 48 +-- .../physical_offset_test.cc} | 16 +- .../physical_rect.cc} | 22 +- .../physical_rect.h} | 44 ++- .../physical_rect_test.cc} | 18 +- .../physical_size.cc} | 6 +- .../renderer/core/layout/geometry/physical_size.h | 65 ++++ .../renderer/core/layout/layout_block_flow_line.cc | 2 +- .../blink/renderer/core/layout/layout_inline.cc | 9 +- .../blink/renderer/core/layout/layout_replaced.cc | 14 +- .../blink/renderer/core/layout/layout_text.cc | 6 +- .../renderer/core/layout/layout_tree_as_text.cc | 2 +- .../layout/ng/exclusions/ng_exclusion_space.cc | 2 +- .../core/layout/ng/exclusions/ng_exclusion_space.h | 6 +- .../core/layout/ng/geometry/ng_bfc_offset.h | 2 +- .../renderer/core/layout/ng/geometry/ng_bfc_rect.h | 2 +- .../core/layout/ng/geometry/ng_box_strut.h | 4 +- .../core/layout/ng/geometry/ng_logical_offset.cc | 63 ---- .../core/layout/ng/geometry/ng_logical_rect.h | 56 ---- .../core/layout/ng/geometry/ng_logical_size.h | 87 ------ .../core/layout/ng/geometry/ng_physical_offset.cc | 59 ---- .../core/layout/ng/geometry/ng_physical_size.h | 68 ---- .../core/layout/ng/geometry/ng_static_position.cc | 2 +- .../core/layout/ng/geometry/ng_static_position.h | 6 +- .../core/layout/ng/inline/ng_caret_rect.cc | 39 ++- .../core/layout/ng/inline/ng_inline_box_state.cc | 10 +- .../core/layout/ng/inline/ng_inline_box_state.h | 8 +- .../ng/inline/ng_inline_fragment_traversal.cc | 4 +- .../layout/ng/inline/ng_inline_layout_algorithm.cc | 4 +- .../ng/inline/ng_inline_layout_algorithm_test.cc | 16 +- .../core/layout/ng/inline/ng_inline_node.cc | 2 +- .../ng/inline/ng_line_box_fragment_builder.h | 12 +- .../core/layout/ng/inline/ng_line_breaker.cc | 6 +- .../core/layout/ng/inline/ng_line_breaker.h | 2 +- .../core/layout/ng/inline/ng_line_breaker_test.cc | 8 +- .../core/layout/ng/inline/ng_line_truncator.cc | 2 +- .../ng/inline/ng_physical_line_box_fragment.cc | 20 +- .../ng/inline/ng_physical_line_box_fragment.h | 11 +- .../layout/ng/inline/ng_physical_text_fragment.cc | 42 ++- .../layout/ng/inline/ng_physical_text_fragment.h | 13 +- .../ng/inline/ng_physical_text_fragment_test.cc | 16 +- .../layout/ng/inline/ng_text_fragment_builder.h | 2 +- .../renderer/core/layout/ng/layout_box_utils.cc | 6 +- .../core/layout/ng/layout_ng_block_flow.cc | 8 +- .../renderer/core/layout/ng/layout_ng_mixin.cc | 6 +- .../core/layout/ng/layout_ng_table_caption.cc | 4 +- .../layout/ng/list/ng_unpositioned_list_marker.cc | 13 +- .../layout/ng/list/ng_unpositioned_list_marker.h | 4 +- .../renderer/core/layout/ng/ng_absolute_utils.cc | 16 +- .../renderer/core/layout/ng/ng_absolute_utils.h | 12 +- .../core/layout/ng/ng_absolute_utils_test.cc | 4 +- .../layout/ng/ng_base_layout_algorithm_test.cc | 6 +- .../core/layout/ng/ng_base_layout_algorithm_test.h | 8 +- .../core/layout/ng/ng_block_layout_algorithm.cc | 46 +-- .../core/layout/ng/ng_block_layout_algorithm.h | 18 +- .../layout/ng/ng_block_layout_algorithm_test.cc | 342 ++++++++++----------- .../blink/renderer/core/layout/ng/ng_block_node.cc | 24 +- .../blink/renderer/core/layout/ng/ng_block_node.h | 16 +- .../core/layout/ng/ng_box_fragment_builder.cc | 32 +- .../core/layout/ng/ng_box_fragment_builder.h | 6 +- .../core/layout/ng/ng_column_layout_algorithm.cc | 30 +- .../core/layout/ng/ng_column_layout_algorithm.h | 10 +- .../layout/ng/ng_column_layout_algorithm_test.cc | 38 +-- .../renderer/core/layout/ng/ng_constraint_space.cc | 8 +- .../renderer/core/layout/ng/ng_constraint_space.h | 34 +- .../core/layout/ng/ng_constraint_space_builder.cc | 6 +- .../core/layout/ng/ng_constraint_space_builder.h | 18 +- .../layout/ng/ng_constraint_space_builder_test.cc | 12 +- .../layout/ng/ng_container_fragment_builder.cc | 57 ++-- .../core/layout/ng/ng_container_fragment_builder.h | 18 +- .../core/layout/ng/ng_fieldset_layout_algorithm.cc | 20 +- .../core/layout/ng/ng_fieldset_layout_algorithm.h | 6 +- .../layout/ng/ng_fieldset_layout_algorithm_test.cc | 4 +- .../core/layout/ng/ng_flex_layout_algorithm.cc | 6 +- .../core/layout/ng/ng_flex_layout_algorithm.h | 4 +- .../renderer/core/layout/ng/ng_floats_utils.cc | 36 +-- .../renderer/core/layout/ng/ng_floats_utils.h | 8 +- .../blink/renderer/core/layout/ng/ng_fragment.h | 4 +- .../renderer/core/layout/ng/ng_fragment_builder.h | 6 +- .../core/layout/ng/ng_inline_layout_test.cc | 11 +- .../core/layout/ng/ng_layout_input_node.cc | 16 +- .../renderer/core/layout/ng/ng_layout_input_node.h | 10 +- .../renderer/core/layout/ng/ng_layout_result.h | 6 +- .../renderer/core/layout/ng/ng_length_utils.cc | 76 ++--- .../renderer/core/layout/ng/ng_length_utils.h | 27 +- .../core/layout/ng/ng_length_utils_test.cc | 8 +- .../blink/renderer/core/layout/ng/ng_link.h | 12 +- .../core/layout/ng/ng_out_of_flow_layout_part.cc | 53 ++-- .../core/layout/ng/ng_out_of_flow_layout_part.h | 16 +- .../core/layout/ng/ng_page_layout_algorithm.cc | 12 +- .../core/layout/ng/ng_page_layout_algorithm.h | 4 +- .../core/layout/ng/ng_physical_box_fragment.cc | 19 +- .../core/layout/ng/ng_physical_box_fragment.h | 4 +- .../layout/ng/ng_physical_container_fragment.h | 2 +- .../core/layout/ng/ng_physical_fragment.cc | 31 +- .../renderer/core/layout/ng/ng_physical_fragment.h | 24 +- .../renderer/core/layout/ng/ng_relative_utils.cc | 14 +- .../renderer/core/layout/ng/ng_relative_utils.h | 8 +- .../core/layout/ng/ng_relative_utils_test.cc | 14 +- .../renderer/core/layout/ng/ng_space_utils.cc | 4 +- .../core/paint/ng/ng_box_fragment_painter.cc | 18 +- .../renderer/core/paint/ng/ng_paint_fragment.cc | 137 ++++----- .../renderer/core/paint/ng/ng_paint_fragment.h | 46 +-- .../core/paint/ng/ng_paint_fragment_test.cc | 8 +- .../core/paint/ng/ng_paint_fragment_traversal.h | 2 +- .../core/paint/ng/ng_text_fragment_painter.cc | 20 +- tools/win/DebugVisualizers/webkit.natvis | 24 +- 119 files changed, 1351 insertions(+), 1391 deletions(-) commit 99320c4f1a79d84985cdf9f3c9767a27191d55d4 Author: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Date: Thu Apr 25 19:46:36 2019 +0000 Roll src/third_party/pdfium abc8777d6cf8..64b9a05218cd (4 commits) https://pdfium.googlesource.com/pdfium.git/+log/abc8777d6cf8..64b9a05218cd git log abc8777d6cf8..64b9a05218cd --date=short --no-merges --format='%ad %ae %s' 2019-04-25 tsepez@chromium.org Revert "Remove potentially tree-corrupting methods from CXFA_LayoutItem." 2019-04-25 thestig@chromium.org Roll third_party/skia/ b5c685991..06be07979 (241 commits; 42 trivial rolls) 2019-04-25 thestig@chromium.org Roll third_party/skia/ 2655ede7c..b5c685991 (1 commit) 2019-04-25 thestig@chromium.org Roll third_party/skia/ 442973874..2655ede7c (9 commits) Created with: gclient setdep -r src/third_party/pdfium@64b9a05218cd The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: Ieae020730ebc4c163298e715c96d02ecaef160bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584742 Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#654150} DEPS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit af4feec96e13e1046509568743c3c8859a40f3d0 Author: Carlos Caballero <carlscab@google.com> Date: Thu Apr 25 19:43:43 2019 +0000 Rename ScopedTaskEnvironment::ExecutionMode ScopedTaskEnvironment::ExecutionMode is becoming ScopedTaskEnvironment::ThreadPoolExecutionMode This CL was uploaded by git cl split. R=yuweih@chromium.org Bug: 708584 Change-Id: Ib721ce0db3f4897daf8208c15c2eff99d042360a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1583790 Auto-Submit: Carlos Caballero <carlscab@google.com> Reviewed-by: Yuwei Huang <yuweih@chromium.org> Commit-Queue: Yuwei Huang <yuweih@chromium.org> Cr-Commit-Position: refs/heads/master@{#654149} remoting/host/file_transfer/file_transfer_message_handler_unittest.cc | 2 +- remoting/host/file_transfer/ipc_file_operations_unittest.cc | 2 +- remoting/host/file_transfer/local_file_operations_unittest.cc | 2 +- remoting/host/gcd_state_updater_unittest.cc | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) commit 097af94950890d9c83ce8f7e503e7f45224ed77a Author: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Date: Thu Apr 25 19:42:00 2019 +0000 Roll src/third_party/depot_tools b2b66999ea37..97654081c73d (1 commits) https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/b2b66999ea37..97654081c73d git log b2b66999ea37..97654081c73d --date=short --no-merges --format='%ad %ae %s' 2019-04-25 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@97654081c73d The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=agable@chromium.org Change-Id: If4e22183e80ce4fc0dc9d58dcefb0c60a563321c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584763 Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#654148} DEPS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit b44c869ab26ed9112e0094fc311081cef1d2b689 Author: Ryan Sturm <ryansturm@chromium.org> Date: Thu Apr 25 19:40:49 2019 +0000 Preconnect at regular intervals in Navigation Predictor For users that stay foregrounded, we should preconnect every 60 (finch param) seconds. Bug: 956229 Change-Id: Ica092ce7d6b53de54e0ed16fbe8a6639cac171d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1577975 Commit-Queue: Ryan Sturm <ryansturm@chromium.org> Reviewed-by: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#654147} .../navigation_predictor/navigation_predictor.cc | 21 ++++++ .../navigation_predictor/navigation_predictor.h | 7 +- .../navigation_predictor_browsertest.cc | 86 +++++++++++++++++++++- tools/metrics/histograms/enums.xml | 3 + 4 files changed, 112 insertions(+), 5 deletions(-) commit bb18ab263004dc1a7ae01c50fb1302ccf7e6b476 Author: Ian Kilpatrick <ikilpatrick@chromium.org> Date: Thu Apr 25 19:39:58 2019 +0000 [LayoutNG] Rename old_layout -> legacy_layout. As above! There should be no behaviour change. Change-Id: I21ebb773d1051e4fcb8defef97c2a4f82a586bc6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584566 Reviewed-by: Christian Biesinger <cbiesinger@chromium.org> Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Auto-Submit: Ian Kilpatrick <ikilpatrick@chromium.org> Cr-Commit-Position: refs/heads/master@{#654146} .../renderer/core/layout/ng/layout_ng_block_flow.cc | 4 ++-- .../renderer/core/layout/ng/layout_ng_flexible_box.cc | 2 +- .../renderer/core/layout/ng/layout_ng_table_caption.cc | 2 +- .../renderer/core/layout/ng/layout_ng_table_cell.cc | 2 +- .../blink/renderer/core/layout/ng/ng_block_node.cc | 16 ++++++++-------- .../blink/renderer/core/layout/ng/ng_block_node.h | 12 ++++++------ .../renderer/core/layout/ng/ng_box_fragment_builder.h | 4 ++-- .../core/layout/ng/ng_container_fragment_builder.h | 2 +- .../blink/renderer/core/layout/ng/ng_layout_result.cc | 2 +- .../core/layout/ng/ng_out_of_flow_layout_part.cc | 2 +- .../renderer/core/layout/ng/ng_physical_box_fragment.cc | 2 +- .../core/layout/ng/ng_physical_box_fragment_test.cc | 17 +++++++++-------- .../renderer/core/layout/ng/ng_physical_fragment.cc | 8 ++++---- .../renderer/core/layout/ng/ng_physical_fragment.h | 8 ++++---- .../blink/renderer/core/paint/ng/ng_paint_fragment.cc | 2 +- 15 files changed, 43 insertions(+), 42 deletions(-) commit 1e25bf4e9db2d5d2c8dd6e67ab062600e590246c Author: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Date: Thu Apr 25 19:39:45 2019 +0000 Roll src/third_party/catapult 58d22d158bab..dfc385d7c610 (2 commits) https://chromium.googlesource.com/catapult.git/+log/58d22d158bab..dfc385d7c610 git log 58d22d158bab..dfc385d7c610 --date=short --no-merges --format='%ad %ae %s' 2019-04-25 benjhayden@chromium.org Reland Finish migrating spa to es6 modules. 2019-04-25 eroman@chromium.org Fix eslint errors in source_entry.js Created with: gclient setdep -r src/third_party/catapult@dfc385d7c610 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:950011 TBR=crouleau@chromium.org Change-Id: I0dd023519f0f076a35ccc0038dc2c1afe419b422 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584760 Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#654145} DEPS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) commit 0ead651a5b21f5d7e3472d543735998ce70bb65f Author: Bruce Dawson <brucedawson@chromium.org> Date: Thu Apr 25 19:38:52 2019 +0000 Use range-based for loop and more references While investigating a UAF bug I noticed that a for loop was copying a scoped_refptr, causing a wasteful increment/decrement of the reference count. Fixing that was just a matter of making the range declaration a reference, but while I was in the area I converted a couple of loops from iterators to range-based loops, for modest readability and efficiency gains. Change-Id: I04534f7838c05afa6d38f3064c837bed46c60450 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584565 Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Auto-Submit: Bruce Dawson <brucedawson@chromium.org> Reviewed-by: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#654144} base/metrics/field_trial.cc | 16 +++++++--------- base/test/scoped_feature_list.cc | 2 +- 2 files changed, 8 insertions(+), 10 deletions(-) commit 3c9b1bea4974fda872c250145d136dd74b382982 Author: Kevin Ellis <kevers@chromium.org> Date: Thu Apr 25 19:38:23 2019 +0000 Fix DCHECK failure in keyframe_effect.cc. A DCHECK failure was exposed by fixing the spec compliance of animation progress. The current iteration of a web animation may be unresolved. Bug: 954558 Change-Id: Ie4b464e4e173fc726774f07c072af21100a6836d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1577927 Reviewed-by: Stephen McGruer <smcgruer@chromium.org> Commit-Queue: Kevin Ellis <kevers@chromium.org> Cr-Commit-Position: refs/heads/master@{#654143} .../blink/renderer/core/animation/timing_calculations.h | 8 +++++--- .../renderer/core/animation/timing_calculations_test.cc | 15 +++++++++++++++ 2 files changed, 20 insertions(+), 3 deletions(-) commit 32fbe21d3aa5e4650d947968f28d759988211dc4 Author: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Date: Thu Apr 25 19:36:07 2019 +0000 GWP-ASan: Report crash handler errors When we encounter an error in the crash handler and we know that we are processing a GWP-ASan exception, send up a protobuf with an error string instead of just sending up the type of failure via UMA histogram. Bug: 950958 Change-Id: I09911ede55141248a993ab625e93ac8db6a8cad5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1578360 Commit-Queue: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Reviewed-by: Vitaly Buka <vitalybuka@chromium.org> Cr-Commit-Position: refs/heads/master@{#654142} .../gwp_asan/client/guarded_page_allocator.h | 1 + components/gwp_asan/common/allocator_state.cc | 22 +++++++--- components/gwp_asan/common/allocator_state.h | 9 ++-- .../gwp_asan/common/allocator_state_unittest.cc | 11 ++--- components/gwp_asan/crash_handler/crash.proto | 4 ++ .../gwp_asan/crash_handler/crash_analyzer.cc | 49 ++++++++++------------ components/gwp_asan/crash_handler/crash_analyzer.h | 1 + .../crash_handler/crash_analyzer_unittest.cc | 25 +++++++++++ components/gwp_asan/crash_handler/crash_handler.cc | 7 +++- 9 files changed, 89 insertions(+), 40 deletions(-) commit 4961cfc60dbce5d59dfcb6be93ca8abd44bca684 Author: Josh Nohle <nohle@google.com> Date: Thu Apr 25 19:34:12 2019 +0000 [DeviceSync v2] Handle GCM messages from DeviceSync v2 GCM messages sent by the CryptAuth v2 DeviceSync backend may contain some or all of the following key-value pairs: "S": TargetService specification, e.g., "2" for DeviceSync "I": session ID string to be included in ClientMetadata "F": feature type string, e.g., "BetterTogetherHostEnabled" "K": DeviceSync group name, e.g., "DeviceSync:BetterTogether" Previously, for CryptAuth v1 and CryptAuth v2 Enrollment, the only relevant key-value pair was "registrationTickleType" which specified the target service, with values "1" or "2" for Enrollment and "3" for DeviceSync. We update the CryptAuth GCM manager to handle these new key-value pairs. Bug: 951969 Change-Id: Ie5e7fbc709613b61530d641c66072ed34e28f2b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1579474 Commit-Queue: Josh Nohle <nohle@chromium.org> Reviewed-by: Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#654141} .../device_sync/cryptauth_device_manager_impl.cc | 8 +- .../device_sync/cryptauth_device_manager_impl.h | 5 +- .../cryptauth_device_manager_impl_unittest.cc | 6 +- .../cryptauth_enrollment_manager_impl.cc | 4 +- .../cryptauth_enrollment_manager_impl.h | 5 +- .../cryptauth_enrollment_manager_impl_unittest.cc | 3 +- .../services/device_sync/cryptauth_gcm_manager.cc | 8 +- .../services/device_sync/cryptauth_gcm_manager.h | 21 +- .../device_sync/cryptauth_gcm_manager_impl.cc | 168 +++++++++++++-- .../cryptauth_gcm_manager_impl_unittest.cc | 239 ++++++++++++++++++++- .../cryptauth_v2_enrollment_manager_impl.cc | 5 +- .../cryptauth_v2_enrollment_manager_impl.h | 5 +- ...ryptauth_v2_enrollment_manager_impl_unittest.cc | 3 +- .../device_sync/fake_cryptauth_gcm_manager.cc | 12 +- .../device_sync/fake_cryptauth_gcm_manager.h | 12 +- 15 files changed, 457 insertions(+), 47 deletions(-) commit ef6832c65db01a5143580a363553ea56c2e30d36 Author: Tarun Bansal <tbansal@chromium.org> Date: Thu Apr 25 19:33:14 2019 +0000 Cleanup of some of the unused code from DRP params (i) Secure proxy holdback field trial is not being used anymore. Effectively, IsIncludedInSecureProxyHoldbackFieldTrial() now always returns false. (ii) Enabling/disabling quic for non-core data saver proxies is not used anymore. Effectively, IsQuicEnabledForNonCoreProxies() now always returns true. Change-Id: I356e0d9fc8409d9458119462fac33a43a8550e56 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1584780 Reviewed-by: Robert Ogden <robertogden@chromium.org> Reviewed-by: rajendrant <rajendrant@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#654140} .../core/browser/data_reduction_proxy_configurator.cc | 5 ----- .../browser/data_reduction_proxy_configurator_unittest.cc | 11 ----------- .../core/common/data_reduction_proxy_bypass_protocol.cc | 7 +++---- .../core/common/data_reduction_proxy_params.cc | 14 -------------- .../core/common/data_reduction_proxy_params.h | 7 ------- .../core/common/data_reduction_proxy_params_unittest.cc | 13 ++----------- 6 files changed, 5 insertions(+), 52 deletions(-) Updated tag 'tree_truth' (was 7026e85f64bd)