Home | Search
Login

Builder Android Cronet Marshmallow 64bit Perf Build #26686 Canonical Ubuntu

Results:

build successful

Trigger Info:

Execution:

Steps and Logfiles:

Show:
  1. ( 21 mins 57 secs ) steps
    running steps via annotated script
  2. ( 10 secs ) update_scripts
    update_scripts
  3. ( 3 ms ) LUCI Migration
    LUCI Migration
  4. ( 4 ms ) LogDog Bootstrap
    LogDog Bootstrap
  5. ( 2 ms ) setup_build
    setup_build
    running recipe: "cronet"
  6. ( 2 ms ) cleanup index.lock
    cleanup index.lock
  7. ( 7 mins 37 secs ) bot_update
    bot_update
    [52GB/442GB used (11%)]
  8. ( 4 ms ) clean local files
    clean local files
  9. ( 3 ms ) ensure_goma
    ensure_goma
  10. ( 7 secs ) ensure_goma.ensure_installed
    ensure_goma.ensure_installed
  11. ( 3 ms ) clobber
    clobber
  12. ( 1 mins 51 secs ) gclient runhooks
    gclient runhooks
  13. ( 4 ms ) lookup GN args
    lookup GN args
     
    Count of GN args (22) exceeds limit (15), presented in logs instead
  14. ( 21 secs ) generate_build_files
    generate_build_files
  15. ( 3 ms ) preprocess_for_goma
    preprocess_for_goma
  16. ( 3 ms ) preprocess_for_goma.goma cache directory
    preprocess_for_goma.goma cache directory
  17. ( 5 secs ) preprocess_for_goma.start_goma
    preprocess_for_goma.start_goma
  18. ( 3 ms ) preprocess_for_goma.start cloudtail
    preprocess_for_goma.start cloudtail
  19. ( 5 mins 59 secs ) compile
    compile
  20. ( 3 ms ) compile confirm no-op
    compile confirm no-op
  21. ( 3 ms ) postprocess_for_goma
    postprocess_for_goma
  22. ( 4 ms ) postprocess_for_goma.goma_jsonstatus
    postprocess_for_goma.goma_jsonstatus
  23. ( 3 ms ) postprocess_for_goma.goma_stat
    postprocess_for_goma.goma_stat
  24. ( 3 ms ) postprocess_for_goma.stop_goma
    postprocess_for_goma.stop_goma
  25. ( 8 secs ) postprocess_for_goma.upload_log
    postprocess_for_goma.upload_log
  26. ( 3 ms ) postprocess_for_goma.stop cloudtail
    postprocess_for_goma.stop cloudtail
  27. ( 4 ms ) clear landmines
    clear landmines
  28. ( 3 ms ) ensure_goma (2)
    ensure_goma (2)
  29. ( 4 ms ) ensure_goma.ensure_installed (2)
    ensure_goma.ensure_installed (2)
  30. ( 38 secs ) gclient runhooks (2)
    gclient runhooks (2)
  31. ( 4 ms ) lookup GN args (2)
    lookup GN args (2)
     
    ffmpeg_branding = "Chrome"
    is_component_build = false
    is_debug = false
    proprietary_codecs = true
    strip_absolute_paths_from_debug_symbols = true
    use_goma = true
    goma_dir = "/b/c/goma_cache/client"
  32. ( 7 secs ) generate_build_files (2)
    generate_build_files (2)
  33. ( 3 ms ) preprocess_for_goma (2)
    preprocess_for_goma (2)
  34. ( 1 ms ) preprocess_for_goma.goma cache directory (2)
    preprocess_for_goma.goma cache directory (2)
  35. ( 5 secs ) preprocess_for_goma.start_goma (2)
    preprocess_for_goma.start_goma (2)
  36. ( 4 ms ) preprocess_for_goma.start cloudtail (2)
    preprocess_for_goma.start cloudtail (2)
  37. ( 21 secs ) compile (2)
    compile (2)
  38. ( 3 ms ) compile confirm no-op (2)
    compile confirm no-op (2)
  39. ( 3 ms ) postprocess_for_goma (2)
    postprocess_for_goma (2)
  40. ( 4 ms ) postprocess_for_goma.goma_jsonstatus (2)
    postprocess_for_goma.goma_jsonstatus (2)
  41. ( 3 ms ) postprocess_for_goma.goma_stat (2)
    postprocess_for_goma.goma_stat (2)
  42. ( 3 ms ) postprocess_for_goma.stop_goma (2)
    postprocess_for_goma.stop_goma (2)
  43. ( 6 secs ) postprocess_for_goma.upload_log (2)
    postprocess_for_goma.upload_log (2)
  44. ( 3 ms ) postprocess_for_goma.stop cloudtail (2)
    postprocess_for_goma.stop cloudtail (2)
  45. ( 3 mins 1 secs ) performance test
    performance test
  46. ( 4 ms ) get access token for service-account-cronet-perf.json
    get access token for service-account-cronet-perf.json
  47. ( 4 ms ) Perf Dashboard Upload
    Perf Dashboard Upload
  48. ( 4 ms ) recipe result
    recipe result

Build Properties:

NameValueSource
$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.android/Android_Cronet_Marshmallow_64bit_Perf/26686/+/recipes/annotations" Annotation(LogDog Bootstrap)
logdog_prefix "bb/chromium.android/Android_Cronet_Marshmallow_64bit_Perf/26686" 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_src_revision "531c057ed48682ab7b275a932b872bcef1a932d9" Annotation(bot_update)
got_src_revision_cp "refs/heads/master@{#599198}" 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 "55d1af14751ad10e0cae741240f013ea67039f45" Annotation(bot_update)
got_webrtc_revision_cp "refs/heads/master@{#25138}" Annotation(bot_update)
build_data_dir "/b/rr/tmp9kWGAN" Annotation(steps)
build_id "buildbot/chromium.android/Android Cronet Marshmallow 64bit Perf/26686" Annotation(steps)
builder_id "master.chromium.android:Android Cronet Marshmallow 64bit Perf" Annotation(steps)
blamelist ["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 26686 Build
project "src" Build
repository "https://chromium.googlesource.com/chromium/src" Build
requestedAt 1539351477 Build
revision "531c057ed48682ab7b275a932b872bcef1a932d9" Build
slavename "build88-b1" BuildSlave
buildername "Android Cronet Marshmallow 64bit Perf" Builder
git_revision "531c057ed48682ab7b275a932b872bcef1a932d9" Change
scheduler "chromium_commits" Scheduler
buildbotURL "https://build.chromium.org/p/chromium.android/" master.cfg
mastername "chromium.android" master.cfg
workdir "/b/build/slave/Android_Cronet_Marshmallow_64bit_Perf" slave

Blamelist:

  1. (wanderviewohnoyoudont@chromium.org)
  2. (penghuangohnoyoudont@chromium.org)
  3. (szermattohnoyoudont@chromium.org)
  4. (treibohnoyoudont@chromium.org)
  5. (dgozmanohnoyoudont@chromium.org)
  6. (robertmaohnoyoudont@chromium.org)
  7. (smaierohnoyoudont@chromium.org)

Timing:

Create N/A
Start Friday, 12-Oct-18 14:06:34 UTC
End Friday, 12-Oct-18 14:28:32 UTC
Pending N/A
Execution 21 mins 57 secs

All Changes:

  1. 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
  2. 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
  3. [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
  4. 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
  5. 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
  6. [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
  7. 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