Home | Search | chromium - Builders
Login

Builder ASan Release Media (32-bit x86 with V8-ARM) Build 54647 Canonical Ubuntu

Results:

Success

Trigger Info:

Projectchromium
Revision7e806513b1b91da6e96a56bfd3ca5f464c4666b9
Got Revision7e806513b1b91da6e96a56bfd3ca5f464c4666b9

Execution:

Steps and Logfiles:

Show:
  1. ( 542 ms ) setup_build
    running recipe: "chromium"
  2. ( 226 ms ) report builders
     
    running builder 'ASan Release Media (32-bit x86 with V8-ARM)' on master 'chromium.lkgr'
  3. ( 1 mins 49 secs ) bot_update
    [91GB/295GB used (30%)]
  4. ( 1 secs ) swarming.py --version
    0.13
  5. ( 4 secs ) clobber
  6. ( 29 secs ) gclient runhooks
  7. ( 491 ms ) lookup GN args
     
    enable_ipc_fuzzer = true
    enable_nacl = false
    ffmpeg_branding = "ChromeOS"
    is_asan = true
    is_component_build = false
    is_debug = false
    proprietary_codecs = true
    sanitizer_coverage_flags = "trace-pc-guard"
    strip_absolute_paths_from_debug_symbols = true
    target_cpu = "x86"
    use_goma = true
    v8_enable_verify_heap = true
    v8_target_cpu = "arm"
    goma_dir = "/b/swarming/w/ir/cache/goma/client"
  8. ( 31 secs ) generate_build_files
  9. ( 735 ms ) clang_revision
  10. ( 16 mins 24 secs ) compile
  11. ( 450 ms ) create staging_dir
  12. ( 7 secs ) Copy sancov
  13. ( 730 ms ) filter build_dir
  14. ( 4 mins 49 secs ) zipping
  15. ( 59 secs ) gsutil upload

Build Properties:

NameValueSource
$recipe_engine/path {"cache_dir": "/b/swarming/w/ir/cache", "temp_dir": "/b/swarming/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "swarm1971-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": 1541785111818594, "id": "8930314850548616304", "project": "chromium", "tags": ["builder:ASan Release Media (32-bit x86 with V8-ARM)", "buildset:commit/git/7e806513b1b91da6e96a56bfd3ca5f464c4666b9", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/7e806513b1b91da6e96a56bfd3ca5f464c4666b9", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9095744912753546496", "scheduler_job_id:chromium/ASan Release Media (32-bit x86 with V8-ARM)", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "ASan Release Media (32-bit x86 with V8-ARM)" setup_build
buildnumber 54647 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 "7e806513b1b91da6e96a56bfd3ca5f464c4666b9" setup_build
got_angle_revision "5d2dfa46c05c8c661304922ce1e1e7d7b045a342" bot_update
got_buildtools_revision "13a00f110ef910a25763346d6538b60f12845656" bot_update
got_dawn_revision "2960ec33667a11bb4445dfe8154aef133c741afe" bot_update
got_nacl_revision "99673869a3cd8731d924bd32fa486feebfdc6c4f" bot_update
got_revision "7e806513b1b91da6e96a56bfd3ca5f464c4666b9" bot_update
got_revision_cp "refs/heads/master@{#606888}" bot_update
got_swarming_client_revision "f78187ab77127de42555afe0ad410bebde6ac6a5" bot_update
got_v8_revision "f6ed3c5d7eb73112326476d681f0f79d40fc1e0e" bot_update
got_v8_revision_cp "refs/heads/7.2.287@{#1}" bot_update
got_webrtc_revision "e769ed90c3599a131166d5a61625c1b64318fb91" bot_update
got_webrtc_revision_cp "refs/heads/master@{#25544}" bot_update
got_clang_revision "346388-1" clang_revision

Blamelist:

  1. Fady Samuel (fsamuelohnoyoudont@chromium.org)
  2. Joe DeBlasio (jdeblasioohnoyoudont@chromium.org)
  3. Sky Malice (skymohnoyoudont@google.com)
  4. chromium-autoroll (chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com)
  5. Hayato Ito (hayatoohnoyoudont@chromium.org)
  6. Mathias Carlen (mcarlenohnoyoudont@chromium.org)

Timing:

Create Friday, 09-Nov-18 17:38:31 UTC
Start Friday, 09-Nov-18 17:38:34 UTC
End Friday, 09-Nov-18 18:07:30 UTC
Pending 2 secs
Execution 28 mins 56 secs

All Changes:

  1. viz: Plumb LocalSurfaceIdAllocation to more places [part 5]

    Changed by Fady Samuel - fsamuelohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 17:36:34 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 7e806513b1b91da6e96a56bfd3ca5f464c4666b9

    Comments

    viz: Plumb LocalSurfaceIdAllocation to more places [part 5]
    
    viz::LocalSurfaceIdAllocation bundles a viz::LocalSurfaceId and an
    allocation time for that viz::LocalSurfaceId. The purpose of this allocation
    time is to determine how long it takes for a surface to get embedded once
    an ID has been allocated for it.
    
    There was some concern raised about drift between LocalSurfaceIds and allocation
    times and so a previous CL bundled the two together in a class. This CL plumbs
    LocalSurfaceIdAllocation to more places to ensure the two quantities remain
    in sync with one another.
    
    This CL plumbs removes various accessors on ChildLocalSurfaceIdAllocator and
    ParentLocalSurfaceIdAllocator that return an allocation time or LocalSurfaceId
    independent of the other. This is to encourage future plumbing of
    LocalSurfaceIdAllocation instead.
    
    Bug: 655231
    Change-Id: I3118cc06e0afa04914a0e2e7457ffdfd5ad5778d
    Reviewed-on: https://chromium-review.googlesource.com/c/1327573
    Reviewed-by: Jonathan Ross <jonross@chromium.org>
    Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
    Commit-Queue: Fady Samuel <fsamuel@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606888}

    Changed files

    • cc/test/layer_tree_test.cc
    • cc/test/layer_tree_test.h
    • cc/trees/layer_tree_host_impl.cc
    • cc/trees/layer_tree_host_impl.h
    • cc/trees/layer_tree_host_unittest.cc
    • components/viz/common/surfaces/child_local_surface_id_allocator.h
    • components/viz/common/surfaces/child_local_surface_id_allocator_unittest.cc
    • components/viz/common/surfaces/parent_local_surface_id_allocator.h
    • components/viz/common/surfaces/parent_local_surface_id_allocator_unittest.cc
    • content/browser/renderer_host/render_widget_host_view_mac_unittest.mm
    • third_party/blink/renderer/platform/graphics/surface_layer_bridge.cc
    • third_party/blink/renderer/platform/graphics/video_frame_submitter.cc
    • third_party/blink/renderer/platform/graphics/video_frame_submitter_test.cc
  2. Record Android revokes in Permissions.Action.Notification.

    Changed by Joe DeBlasio - jdeblasioohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 17:35:40 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f8c0f174bb015976222531ead359ade19b488833

    Comments

    Record Android revokes in Permissions.Action.Notification.
    
    This CL modifies fixes a bug in which metrics were not being recorded on
    Android when notification permissions were revoked. It adds a missing
    ScopedRevocationReporter for Chrome-visible changes, and adds detection
    code for when notification permissions are revoked in Android O+
    system channel settings.
    
    Bug: 782126
    Change-Id: I609e909936d09e6dd948f0601bbf73ecdb6b8b75
    Reviewed-on: https://chromium-review.googlesource.com/c/1324394
    Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org>
    Reviewed-by: Theresa <twellington@chromium.org>
    Reviewed-by: Dominick Ng <dominickn@chromium.org>
    Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
    Reviewed-by: Finnur Thorarinsson <finnur@chromium.org>
    Reviewed-by: Christopher Thompson <cthomp@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606887}

    Changed files

    • chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java
    • chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePreferenceBridge.java
    • chrome/browser/android/preferences/website_preference_bridge.cc
    • chrome/browser/permissions/permission_uma_util.h
    • tools/metrics/ukm/ukm.xml
  3. [Feed] Check loadUrlParams for null, fallback to opening online.

    Changed by Sky Malice - skymohnoyoudont@google.com
    Changed at Friday, 09-Nov-18 17:33:59 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 60b47f98f87e0f3689826e4342fab09b7a35ed5e

    Comments

    [Feed] Check loadUrlParams for null, fallback to opening online.
    
    Bug: 901179
    Change-Id: Ieb3a421a7930ddc0014f7ef57b6a8f6d89dc3615
    Reviewed-on: https://chromium-review.googlesource.com/c/1315358
    Commit-Queue: Sky Malice <skym@chromium.org>
    Reviewed-by: Filip Gorski <fgorski@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606886}

    Changed files

    • chrome/android/feed/core/java/src/org/chromium/chrome/browser/feed/FeedLoggingBridge.java
    • chrome/android/feed/core/java/src/org/chromium/chrome/browser/feed/action/FeedActionHandler.java
    • chrome/android/junit/src/org/chromium/chrome/browser/feed/action/FeedActionHandlerTest.java
    • chrome/browser/android/feed/feed_logging_bridge.cc
    • chrome/browser/android/feed/feed_logging_bridge.h
  4. Roll src/third_party/perfetto cf90797ef05a..5fc6c2c52e7b (1 commits)

    Changed by chromium-autoroll - chromium-autorollohnoyoudont@skia-public.iam.gserviceaccount.com
    Changed at Friday, 09-Nov-18 17:32:05 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 0475284ad58e36e9950059000bb7ec3556bb31bb

    Comments

    Roll src/third_party/perfetto cf90797ef05a..5fc6c2c52e7b (1 commits)
    
    https://android.googlesource.com/platform/external/perfetto.git/+log/cf90797ef05a..5fc6c2c52e7b
    
    
    git log cf90797ef05a..5fc6c2c52e7b --date=short --no-merges --format='%ad %ae %s'
    2018-11-09 lalitm@google.com Merge "trace_processor: allow integers to be compared against doubles"
    
    
    Created with:
      gclient setdep -r src/third_party/perfetto@5fc6c2c52e7b
    
    The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-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=perfetto-bugs@google.com
    
    Change-Id: Ifd53d09565e264f09c1deffb23f8adfce3a34958
    Reviewed-on: https://chromium-review.googlesource.com/c/1329505
    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@{#606885}

    Changed files

    • DEPS
  5. Fix a bug in https://crrev.com/c/1272937, where event.target can be still null

    Changed by Hayato Ito - hayatoohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 17:31:43 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 2ed9605b4c99774d8865dbd190e13c9fe5c2d03a

    Comments

    Fix a bug in https://crrev.com/c/1272937, where event.target can be still null
    
    Fix a bug in the previous CL, https://crrev.com/c/1272937, where event.target
    can be null when GetElement() returns nullptr.
    
    Bug: 892970, 893449, 902287
    Change-Id: I7a08227d39117c2dc90fe720f0d6ffd62d9b2ea6
    Reviewed-on: https://chromium-review.googlesource.com/c/1322177
    Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
    Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606884}

    Changed files

    • third_party/blink/renderer/modules/accessibility/ax_object.cc
  6. [Autofill Assistant] Handle payment request dismiss.

    Changed by Mathias Carlen - mcarlenohnoyoudont@chromium.org
    Changed at Friday, 09-Nov-18 17:30:12 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f3fe32fc147390a01e54fbd7c54e62ca3a3b8ad4

    Comments

    [Autofill Assistant] Handle payment request dismiss.
    
    Before this patch the payment request UI did not properly handle cancel
    and back button.
    
    This patch notifies the Autofill Assistant controller to shut down
    gracefully with a 'give up' message whenever the payment request UI has
    been canceled.
    
    Bug: 806868
    Change-Id: Ie3d83bf48a2004f0a6a72763c4b509b3db023ff9
    Reviewed-on: https://chromium-review.googlesource.com/c/1326499
    Commit-Queue: Mathias Carlen <mcarlen@chromium.org>
    Reviewed-by: Stephane Zermatten <szermatt@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#606883}

    Changed files

    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/AutofillAssistantPaymentRequest.java
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/AutofillAssistantUiController.java
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/AutofillAssistantUiDelegate.java
    • chrome/android/java/src/org/chromium/chrome/browser/autofill_assistant/ui/PaymentRequestUI.java
    • components/autofill_assistant_strings.grdp