Home | Search | chromium - Builders
Login

Builder Win ASan Release Media Build 24284 Microsoft Windows

Results:

Success

Trigger Info:

Projectchromium
Revisiondcf09c7b1347f35a690e11fc7f592b678d5532db
Got Revisiondcf09c7b1347f35a690e11fc7f592b678d5532db

Execution:

Steps and Logfiles:

Show:
  1. ( 534 ms ) setup_build
    running recipe: "chromium"
  2. ( 5 secs ) taskkill
  3. ( 399 ms ) remove .cipd
  4. ( 3 mins 2 secs ) bot_update
    [61GB/299GB used (20%)]
  5. ( 0 ) ensure_goma
  6. ( 624 ms ) swarming.py --version
    0.13
  7. ( 13 secs ) clobber
  8. ( 1 mins 23 secs ) gclient runhooks
  9. ( 1 mins 38 secs ) generate_build_files
  10. ( 546 ms ) clang_revision
  11. ( 8 mins 20 secs ) compile
  12. ( 401 ms ) create staging_dir
  13. ( 1 secs ) Copy sancov
  14. ( 1 secs ) filter build_dir
  15. ( 1 mins 9 secs ) zipping
  16. ( 14 secs ) gsutil upload

Build Properties:

NameValueSource
$recipe_engine/path {"cache_dir": "C:\\b\\swarming\\w\\ir\\cache", "temp_dir": "C:\\b\\swarming\\w\\ir\\tmp\\rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "swarm2201-c4" setup_build
branch "refs/heads/master" setup_build
buildbucket {"build": {"bucket": "luci.chromium.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1531429573682700, "id": "8941173419305567072", "project": "chromium", "tags": ["builder:Win ASan Release Media", "buildset:commit/git/dcf09c7b1347f35a690e11fc7f592b678d5532db", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/dcf09c7b1347f35a690e11fc7f592b678d5532db", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9106603481570629856", "scheduler_job_id:chromium/Win ASan Release Media", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "Win ASan Release Media" setup_build
buildnumber 24284 setup_build
mastername "chromium.lkgr" setup_build
path_config "generic" setup_build
recipe "chromium" setup_build
repository "https://chromium.googlesource.com/chromium/src.git" setup_build
revision "dcf09c7b1347f35a690e11fc7f592b678d5532db" setup_build
got_angle_revision "5598148b761380773c4e650b5b67f47553b0f090" bot_update
got_buildtools_revision "0dd5c6f980d22be96b728155249df2da355989d9" bot_update
got_nacl_revision "e6ce828ef60c4c1438867b535efbbb5d9a177c0e" bot_update
got_revision "dcf09c7b1347f35a690e11fc7f592b678d5532db" bot_update
got_revision_cp "refs/heads/master@{#574717}" bot_update
got_swarming_client_revision "9a518d097dca20b7b00ce3bdfc5d418ccc79893a" bot_update
got_v8_revision "8b17db68f75a95d69debfa85b70aea3ec0766aaf" bot_update
got_v8_revision_cp "refs/heads/6.9.352@{#1}" bot_update
got_webrtc_revision "895617fa9bc36c240b37d47402cdb341ecadf994" bot_update
got_webrtc_revision_cp "refs/heads/master@{#23955}" bot_update
got_clang_revision "336424-1" clang_revision

Blamelist:

  1. liberato@chromium.org (liberatoohnoyoudont@chromium.org)
  2. Henrique Grandinetti (hgrandinettiohnoyoudont@chromium.org)

Timing:

Create Thursday, 12-Jul-18 21:06:13 UTC
Start Thursday, 12-Jul-18 21:06:18 UTC
End Thursday, 12-Jul-18 21:25:53 UTC
Pending 4 secs
Execution 19 mins 35 secs

All Changes:

  1. Handle MCVD cleanup properly if texture has been destroyed.

    Changed by liberato@chromium.org - liberatoohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 21:05:18 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision dcf09c7b1347f35a690e11fc7f592b678d5532db

    Comments

    Handle MCVD cleanup properly if texture has been destroyed.
    
    If AbstractTexture drops its reference to the underlying texture,
    then that texture might have been freed.  This happens when the
    gl stub is lost.
    
    Previously, MCVD assumed that the CodecImage attached to the texture
    was valid.  However, if the AbstractTexture has dropped its ref
    to the texture, and there are no other refs, then this assumption
    isn't right.
    
    This CL checks if the AbstractTexture still has a TextureBase before
    referencing the CodecImage.
    
    An alternate approach of holding a scoped_refptr to the CodecImage
    in the callback would also work, but might keep the CodecImage
    around longer than it should when the stub is destroyed.  This can
    hold the MediaCodec open longer if the CodecImage has an unrendered
    codec buffer.
    
    Bug: 863012
    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
    Change-Id: If83a9eb0c27d6eb8e995424bdf71f7f7bc93590d
    Reviewed-on: https://chromium-review.googlesource.com/1135697
    Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
    Commit-Queue: Frank Liberato <liberato@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574717}

    Changed files

    • media/gpu/android/texture_pool.cc
    • media/gpu/android/video_frame_factory_impl.cc
  2. Fix wrong calculation of time on UsageTimeLimitProcessor.

    Changed by Henrique Grandinetti - hgrandinettiohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 21:04:17 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 4e070c6b548e584a4e7934e88bcfb98a045518b1

    Comments

    Fix wrong calculation of time on UsageTimeLimitProcessor.
    
    The function that calculates wheter the time window limit of the current day is
    active, had a wrong logic.
    
    Bug: 860679
    Change-Id: Iafaaa54e36424033adaf384cdbc9c15d7aa2328e
    Reviewed-on: https://chromium-review.googlesource.com/1133459
    Reviewed-by: Jacob Dufault <jdufault@chromium.org>
    Reviewed-by: Alexander Alekseev <alemate@chromium.org>
    Commit-Queue: Henrique Grandinetti <hgrandinetti@google.com>
    Cr-Commit-Position: refs/heads/master@{#574716}

    Changed files

    • chrome/browser/chromeos/child_accounts/usage_time_limit_processor.cc
    • chrome/browser/chromeos/child_accounts/usage_time_limit_processor_unittest.cc