Builder WebKit Linux Trusty Leak Build #25260
Results:
build successful
Trigger Info:
| Got Revision | bb5802e5087f37012c6ce4c14733167a700592f7 |
Execution:
Steps and Logfiles:
Show:-
( 3 ms ) report buildersreport buildersrunning builder/tester 'WebKit Linux Trusty Leak' on master 'chromium.webkit'
-
( 1 ms ) read test spec (chromium.webkit.json)read test spec (chromium.webkit.json)path: /b/c/b/WebKit_Linux_Trusty_Leak/src/testing/buildbot/chromium.webkit.json
-
( 1 ms ) lookup GN argslookup GN argsis_component_build = falseis_debug = falsestrip_absolute_paths_from_debug_symbols = trueuse_goma = truegoma_dir = "/b/c/goma_cache/client"
-
( 1 ms ) test_pre_run.[trigger] webkit_layout_teststest_pre_run.[trigger] webkit_layout_testsRun on OS: 'Ubuntu-14.04'
-
( 19 mins 13 secs ) webkit_layout_testswebkit_layout_testsRun on OS: 'Ubuntu-14.04'Max shard duration: 0:17:25.167124 (shard #2)Min shard duration: 0:13:58.450300 (shard #0)Total tests: 87454* Passed: 74565 (74519 expected, 46 unexpected)* Skipped: 10353 (10353 expected, 0 unexpected)* Failed: 2491 (2491 expected, 0 unexpected)* Flaky: 45 (45 expected, 0 unexpected)
- stdout
- json.output
- swarming.summary
- Merge script log
- outdir_json
- step_metadata
- shard #0 (781.3 sec)
- shard #0 isolated out
- shard #1 (939.7 sec)
- shard #1 isolated out
- shard #2 (990.9 sec)
- shard #2 isolated out
- shard #3 (815.5 sec)
- shard #3 isolated out
- shard #4 (766.7 sec)
- shard #4 isolated out
- shard #5 (894.4 sec)
- shard #5 isolated out
- shard #6 (949.1 sec)
- shard #6 isolated out
- shard #7 (916.5 sec)
- shard #7 isolated out
- shard #8 (903.4 sec)
- shard #8 isolated out
- shard #9 (799.0 sec)
- shard #9 isolated out
-
( 1 mins 2 secs ) Upload to test-results [webkit_layout_tests]Upload to test-results [webkit_layout_tests]
Build Properties:
| Name | Value | Source |
|---|---|---|
| $recipe_engine/runtime | {"is_experimental":false,"is_luci":false} | Annotation(LUCI Migration) |
| luci_migration | {"status":"ok"} | Annotation(LUCI Migration) |
| log_location | "logdog://logs.chromium.org/chromium/bb/chromium.webkit/WebKit_Linux_Trusty_Leak/25260/+/recipes/annotations" | Annotation(LogDog Bootstrap) |
| logdog_prefix | "bb/chromium.webkit/WebKit_Linux_Trusty_Leak/25260" | Annotation(LogDog Bootstrap) |
| logdog_project | "chromium" | Annotation(LogDog Bootstrap) |
| got_angle_revision | "671809e62e807eb0f890a5d53eee1f8cb5fb825e" | Annotation(bot_update) |
| got_buildtools_revision | "2dff9c9c74e9d732e6fe57c84ef7fd044cc45d96" | Annotation(bot_update) |
| got_nacl_revision | "2ebbdab4bfb1ed9dfe52ce756b34829207dbe25f" | Annotation(bot_update) |
| got_revision | "bb5802e5087f37012c6ce4c14733167a700592f7" | Annotation(bot_update) |
| got_revision_cp | "refs/heads/master@{#599199}" | Annotation(bot_update) |
| got_swarming_client_revision | "486c9b53c4d54dd4b95bb6ce0e31160e600dfc11" | Annotation(bot_update) |
| got_v8_revision | "f70aaa8ab2e8815505a6145c745e50d8328cd28c" | Annotation(bot_update) |
| got_v8_revision_cp | "refs/heads/7.1.302@{#1}" | Annotation(bot_update) |
| got_webrtc_revision | "c34cf71d601759149a79cd9d59293397009f7b0e" | Annotation(bot_update) |
| got_webrtc_revision_cp | "refs/heads/master@{#25140}" | Annotation(bot_update) |
| swarm_hashes | {"webkit_layout_tests_exparchive":"92c48e37288317d00e5360ca1d206e672ebec39d"} | Annotation(find isolated tests) |
| swarm_hashes_refs/heads/master(at){#599199}_without_patch | {"webkit_layout_tests_exparchive":"92c48e37288317d00e5360ca1d206e672ebec39d"} | Annotation(isolate webkit_layout_tests_exparchive) |
| build_data_dir | "/b/rr/tmpEkx8Yg" | Annotation(steps) |
| build_id | "buildbot/chromium.webkit/WebKit Linux Trusty Leak/25260" | Annotation(steps) |
| builder_id | "master.chromium.webkit:WebKit Linux Trusty Leak" | Annotation(steps) |
| blamelist | ["chromium-autoroll@skia-public.iam.gserviceaccount.com","dgozman@chromium.org","penghuang@chromium.org","robertma@chromium.org","smaier@chromium.org","szermatt@chromium.org","treib@chromium.org","wanderview@chromium.org"] | Build |
| branch | "master" | Build |
| buildnumber | 25260 | Build |
| project | "chromium" | Build |
| repository | "https://chromium.googlesource.com/chromium/src" | Build |
| requestedAt | 1539351451 | Build |
| revision | "bb5802e5087f37012c6ce4c14733167a700592f7" | Build |
| slavename | "slave160-c1" | BuildSlave |
| buildername | "WebKit Linux Trusty Leak" | Builder |
| git_revision | "bb5802e5087f37012c6ce4c14733167a700592f7" | Change |
| scheduler | "global_scheduler" | Scheduler |
| buildbotURL | "http://build.chromium.org/p/chromium.webkit/" | master.cfg |
| mastername | "chromium.webkit" | master.cfg |
| workdir | "/b/build/slave/WebKit_Linux_Trusty_Leak" | slave |
Blamelist:
- (wanderviewohnoyoudont@chromium.org)
- (penghuangohnoyoudont@chromium.org)
- (szermattohnoyoudont@chromium.org)
- (treibohnoyoudont@chromium.org)
- (dgozmanohnoyoudont@chromium.org)
- (robertmaohnoyoudont@chromium.org)
- (smaierohnoyoudont@chromium.org)
- (chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com)
Timing:
| Create | N/A |
| Start | Friday, 12-Oct-18 14:07:46 UTC |
| End | Friday, 12-Oct-18 14:38:29 UTC |
| Pending | N/A |
| Execution | 30 mins 43 secs |
All Changes:
-
Make DataPipeBytesConsumer support ReadableStream loading better.
Changed by wanderviewohnoyoudont@chromium.org Changed at Friday, 12-Oct-18 13:36:24 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision c43eba9311573672d0773beed07f117d08eed09f Comments
Make DataPipeBytesConsumer support ReadableStream loading better. This CL fixes some edge condition interactions when being loaded as a ReadableStream. In particular, it: 1. Properly distinguishes between the end of the DataPipe and a closed DataPipe with bytes left to be read. A ReadableStream that is not actively draining the pipe could get closed too early. 2. Responses must support explicit completion in order to handle error conditions properly. This CL makes DataPipeBytesConsumer w ...skip... explicit signal before closing. 3. Service worker navigation preload is updated to provide the explicit completion signals. Bug: 894815 Change-Id: I8cff3de94aa2dcbc8deb4a9601a95c13b8ab94d9 Reviewed-on: https://chromium-review.googlesource.com/c/1272715 Commit-Queue: Ben Kelly <wanderview@chromium.org> Reviewed-by: Yutaka Hirano <yhirano@chromium.org> Reviewed-by: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#599192}
Changed files
- content/browser/service_worker/service_worker_browsertest.cc
- third_party/blink/renderer/core/fetch/bytes_consumer.h
- third_party/blink/renderer/core/fetch/data_pipe_bytes_consumer.cc
- third_party/blink/renderer/core/fetch/data_pipe_bytes_consumer.h
- third_party/blink/renderer/core/fetch/data_pipe_bytes_consumer_test.cc
- third_party/blink/renderer/core/fetch/fetch_data_loader.cc
- third_party/blink/renderer/core/fetch/fetch_data_loader.h
- third_party/blink/renderer/modules/service_worker/fetch_event.cc
- third_party/blink/renderer/modules/service_worker/fetch_event.h
- third_party/blink/renderer/modules/service_worker/fetch_respond_with_observer.cc
-
Hook up skia makeYUVAPromiseTexture API
Changed by penghuangohnoyoudont@chromium.org Changed at Friday, 12-Oct-18 13:39:05 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 33ca177f048a1d593c5486b46f09ac9ea91071a9 Comments
Hook up skia makeYUVAPromiseTexture API Know issues: * Skia only draws Y plane. * makeYUVAPromiseTexture() will release context, even if it failed. * Skia doesn't accept GL_RGBA8 format for a yuva plane Bug: 824382 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: Ic8da2b3de574ef2a0d527c79f34d383a676a3bc0 Reviewed-on: https://chromium-review.googlesource.com/c/1262421 Reviewed-by: Antoine Labour <piman@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#599193}
Changed files
- components/viz/service/display/skia_output_surface.h
- components/viz/service/display/skia_renderer.cc
- components/viz/service/display_embedder/skia_output_surface_impl.cc
- components/viz/service/display_embedder/skia_output_surface_impl.h
- components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.cc
- components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.h
-
[Autofill Assistant] Add Mathias and I as owners
Changed by szermattohnoyoudont@chromium.org Changed at Friday, 12-Oct-18 13:39:47 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 1b71114eba7ab4e982eaf03186b12503c8ca49c4 Comments
[Autofill Assistant] Add Mathias and I as owners Change-Id: I71ed01f39d1c81c62828d8e4900091e7941e7ab1 Reviewed-on: https://chromium-review.googlesource.com/c/1278399 Reviewed-by: Ganggui Tang <gogerald@chromium.org> Commit-Queue: Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#599194}
Changed files
- components/autofill_assistant/OWNERS
-
ProfileSyncService cleanup: Remove platform_sync_allowed_provider_
Changed by treibohnoyoudont@chromium.org Changed at Friday, 12-Oct-18 13:42:49 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision f0d63432c51741e6dc0bf48f0b84e4afb63f887f Comments
ProfileSyncService cleanup: Remove platform_sync_allowed_provider_ ProfileSyncService::platform_sync_allowed_provider_ was a callback that returned whether Android's "Master Sync" toggle is enabled, and so Sync shouldn't start up. However, PSS didn't know when the value changed, so the callback was kinda useless, and PSS could easily get into an inconsistent state (where there is a disable reason, but Sync is still running). Things were working out in practice only because the Android UI code also called RequestStart/Stop as necessary. So, since PSS needs change notifications anyway, let's make that explicit: This CL replaces the callback by a bool, which can be set explicitly. Bug: 867901 Change-Id: I5b7e9f4657d1f565d9643984eb52ec1df26b5c90 Reviewed-on: https://chromium-review.googlesource.com/c/1269870 Reviewed-by: Mikel Astiz <mastiz@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#599195}
Changed files
- chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
- chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java
- chrome/browser/sync/profile_sync_service_android.cc
- chrome/browser/sync/profile_sync_service_android.h
- components/browser_sync/profile_sync_service.cc
- components/browser_sync/profile_sync_service.h
-
Restore FrameLoadRequest::HrefTranslate
Changed by dgozmanohnoyoudont@chromium.org Changed at Friday, 12-Oct-18 13:42:52 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 703e2b8a5b805ecd4fba4bffb76095125330cec4 Comments
Restore FrameLoadRequest::HrefTranslate This was accidentaly removed as part of FrameLoadRequest cleanup. Bug: none Change-Id: I58bf324729d397a2f0340831fd116ab749f7a213 Reviewed-on: https://chromium-review.googlesource.com/c/1278290 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#599196}
Changed files
- third_party/blink/renderer/core/html/html_anchor_element.cc
- third_party/blink/renderer/core/loader/frame_load_request.h
-
[WPT import] Add a flag to upload but not submit CL
Changed by robertmaohnoyoudont@chromium.org Changed at Friday, 12-Oct-18 13:49:45 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 167121ea09987452572723a78b2d8987b770b751 Comments
[WPT import] Add a flag to upload but not submit CL We used to have only one flag, --auto-update, which controls both uploading and landing the import CL. This makes testing new setup of the bot (i.e. LUCI in this case) hard. This change adds one more flag, --auto-upload, to exercise more steps (namely uploading and rebaselining), which will be used on the experimental LUCI builder. Bug: 790503 Change-Id: I4ce07fc9ddf15a34c9cb25ba4723d250cad26977 Reviewed-on: https://chromium-review.googlesource.com/c/1277587 Commit-Queue: Robert Ma <robertma@chromium.org> Reviewed-by: Quinten Yearsley <qyearsley@chromium.org> Cr-Commit-Position: refs/heads/master@{#599197}
Changed files
- third_party/blink/tools/blinkpy/w3c/test_importer.py
-
Android: Adding min_api for r8 dexing.
Changed by smaierohnoyoudont@chromium.org Changed at Friday, 12-Oct-18 14:04:10 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 531c057ed48682ab7b275a932b872bcef1a932d9 Comments
Android: Adding min_api for r8 dexing. Bug: 872904 Change-Id: I44e020b7d7d121650f4c292b71b4d2dbc45fa9f3 Reviewed-on: https://chromium-review.googlesource.com/c/1277765 Reviewed-by: agrieve <agrieve@chromium.org> Commit-Queue: Sam Maier <smaier@chromium.org> Cr-Commit-Position: refs/heads/master@{#599198}
Changed files
- build/android/gyp/proguard.py
- build/config/android/internal_rules.gni
-
Roll src/third_party/webrtc 55d1af14751a..c34cf71d6017 (2 commits)
Changed by chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com Changed at Friday, 12-Oct-18 14:06:01 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision bb5802e5087f37012c6ce4c14733167a700592f7 Comments
Roll src/third_party/webrtc 55d1af14751a..c34cf71d6017 (2 commits) https://webrtc.googlesource.com/src.git/+log/55d1af14751a..c34cf71d6017 git log 55d1af14751a..c34cf71d6017 --date=short --no-merges --format='%ad %ae %s' 2018-10-12 oprypin@webrtc.org Revert "Remove old video_bitrate_allocator.h" 2018-10-12 mbonadei@webrtc.org Move SdpType from/to string definition close to declaration. Created with: gclient setdep -r src/third_party/webrtc@c34cf71d6017 The AutoRoll server is located here: http ...skip... ary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I7014c484f5486bdc51a05adc2b602509c6c86207 Reviewed-on: https://chromium-review.googlesource.com/c/1278713 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@{#599199}
Changed files
- DEPS