Home | Search | chromium - Builders
Login

Builder Mac ASAN Release Build 13383 Apple OSX

Results:

Success

Trigger Info:

Projectchromium
Revisionab9d0ad4ed6a95ee7675fabb4d16cf63d286dba2
Got Revisionab9d0ad4ed6a95ee7675fabb4d16cf63d286dba2

Execution:

Steps and Logfiles:

Show:
  1. ( 345 ms ) setup_build
    running recipe: "chromium"
  2. ( 413 ms ) remove .cipd
  3. ( 1 mins 16 secs ) bot_update
    [85GB/232GB used (36%)]
  4. ( 0 ) ensure_goma
  5. ( 771 ms ) swarming.py --version
    0.13
  6. ( 7 secs ) clobber
  7. ( 198 ms ) ensure_installed
  8. ( 14 secs ) gclient runhooks
  9. ( 19 secs ) generate_build_files
  10. ( 643 ms ) clang_revision
  11. ( 47 mins 41 secs ) compile
  12. ( 439 ms ) create staging_dir
  13. ( 454 ms ) Copy sancov
  14. ( 922 ms ) filter build_dir
  15. ( 2 mins 44 secs ) zipping
  16. ( 55 secs ) gsutil upload

Build Properties:

NameValueSource
$recipe_engine/path {"cache_dir": "/b/s/w/ir/cache", "temp_dir": "/b/s/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "build155-m9" setup_build
branch "refs/heads/master" setup_build
buildbucket {"build": {"bucket": "luci.chromium.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1531397829711300, "id": "8941206705271316704", "project": "chromium", "tags": ["builder:Mac ASAN Release", "buildset:commit/git/ab9d0ad4ed6a95ee7675fabb4d16cf63d286dba2", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/ab9d0ad4ed6a95ee7675fabb4d16cf63d286dba2", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9106636767499037648", "scheduler_job_id:chromium/Mac ASAN Release", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "Mac ASAN Release" setup_build
buildnumber 13383 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 "ab9d0ad4ed6a95ee7675fabb4d16cf63d286dba2" setup_build
got_angle_revision "5598148b761380773c4e650b5b67f47553b0f090" bot_update
got_buildtools_revision "0dd5c6f980d22be96b728155249df2da355989d9" bot_update
got_nacl_revision "e6ce828ef60c4c1438867b535efbbb5d9a177c0e" bot_update
got_revision "ab9d0ad4ed6a95ee7675fabb4d16cf63d286dba2" bot_update
got_revision_cp "refs/heads/master@{#574531}" 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 "5795489e83adba65cb480d57e71f52bae1f5ee1b" bot_update
got_webrtc_revision_cp "refs/heads/master@{#23944}" bot_update
got_clang_revision "336424-1" clang_revision

Blamelist:

  1. yoshiki iguchi (yoshikiohnoyoudont@chromium.org)
  2. Shik Chen (shikohnoyoudont@chromium.org)
  3. Benoit Lize (lizebohnoyoudont@chromium.org)
  4. Peter Mayo (petermayoohnoyoudont@chromium.org)

Timing:

Create Thursday, 12-Jul-18 12:17:09 UTC
Start Thursday, 12-Jul-18 12:17:12 UTC
End Thursday, 12-Jul-18 13:11:59 UTC
Pending 3 secs
Execution 54 mins 46 secs

All Changes:

  1. Activate widget correctly when the inline reply opens on ARC notification

    Changed by yoshiki iguchi - yoshikiohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 12:13:51 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision ab9d0ad4ed6a95ee7675fabb4d16cf63d286dba2

    Comments

    Activate widget correctly when the inline reply opens on ARC notification
    
    When a non-ARC normal window (eg. Chrome browser window) was active, the
    inline textbox didn't work for the second time and later. This CL solves
    this issue.
    
    This was because the widget was not activated correctly (IsActive() was
    true but CanActivate() was false) in that case. This CL removes the
    IsActive() guard to make the widget active correctly whenever the inline
    reply opens.
    
    Bug: b/111100376
    Bug: 858938
    Test: manual (inline reply can be used multiple times)
    
    Change-Id: I65d7604be5d24ff1e6c56336a76a5951d6066b39
    Reviewed-on: https://chromium-review.googlesource.com/1133039
    Reviewed-by: Eliot Courtney <edcourtney@chromium.org>
    Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574531}

    Changed files

    • ash/system/message_center/arc/arc_notification_content_view.cc
  2. media: sync the camera metadata definitions

    Changed by Shik Chen - shikohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 12:13:48 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 38884d4717c2517cb832722d0fa4871a89d19799

    Comments

    media: sync the camera metadata definitions
    
    We need some new metadata tags for external camera, so regenerate and
    sync all of them in arc-camera, chromium, and ARC++.
    
    Some "arc" are replaced by "cros" manually.
    
    BUG=b:77833131
    TEST=Verify CCA works on Nautilus.
    
    Change-Id: I308e258d9b72acdd825591041a67ce7352dce6a8
    Reviewed-on: https://chromium-review.googlesource.com/1117970
    Reviewed-by: Ricky Liang <jcliang@chromium.org>
    Reviewed-by: Daniel Cheng <dcheng@chromium.org>
    Commit-Queue: Shik Chen <shik@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574530}

    Changed files

    • media/capture/video/chromeos/mojo/camera_metadata_tags.mojom
  3. blink/wtf: Really move MovableStrings when DCHECK_IS_ON.

    Changed by Benoit Lize - lizebohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 12:09:37 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d1fef2b1ad38fe1ef8c3914fb5dadc39bae80b9b

    Comments

    blink/wtf: Really move MovableStrings when DCHECK_IS_ON.
    
    When a MovableString is parked, actually park the content, and poison
    the previous allocation. This is intended to find cached raw pointer
    accesses, which are not allowed beyond the UI thread task per the
    contract of the class.
    Enabled only when DCHECK_IS_ON to prevent potential crashes.
    
    Changes with and without DCHECK()s:
    - Cache some String properties in MovableStringImpl
    - Add a parked_strings_ to MovableStringTable for accounting, and
      update tests accordingly.
    
    Changes with DCHECK()s:
    - Add a parked_string_ member to MovableStringImpl
    - Copy the string to the parked version in Park(), and poison the
      previous allocation.
    - Restore the string in Unpark().
    
    Bug: 837659
    Change-Id: Ib7631d5e12471826c9e71a424e30d26cf8f1f042
    Reviewed-on: https://chromium-review.googlesource.com/1133169
    Reviewed-by: Kentaro Hara <haraken@chromium.org>
    Commit-Queue: Benoit L <lizeb@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574529}

    Changed files

    • third_party/blink/renderer/platform/wtf/text/movable_string.cc
    • third_party/blink/renderer/platform/wtf/text/movable_string.h
    • third_party/blink/renderer/platform/wtf/text/movable_string_test.cc
  4. Re-enable opening-handshake/003-sets-origin.worker.html

    Changed by Peter Mayo - petermayoohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 11:58:23 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d18929b41e24487865c4ed9da46129e6b037beeb

    Comments

    Re-enable opening-handshake/003-sets-origin.worker.html
    
    These tests no longer timeout.  The flakiness dashboard is green
    for a long time.
    
    Bug: 803558
    Change-Id: I2924740d2c736419fdc5130e546f557c8d874151
    Reviewed-on: https://chromium-review.googlesource.com/1134094
    Reviewed-by: Robert Ma <robertma@chromium.org>
    Commit-Queue: Peter Mayo <petermayo@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574528}

    Changed files

    • third_party/WebKit/LayoutTests/TestExpectations