Home | Search | chromium - Builders
Login

Builder android-cronet-arm-rel Build 5675 Canonical Ubuntu

Results:

Success

Trigger Info:

Projectchromium
Revision1d2509fc2ebc7c9848726c1b208483e1e1efc485
Got Revision1d2509fc2ebc7c9848726c1b208483e1e1efc485

Execution:

Steps and Logfiles:

Show:
  1. ( 304 ms ) setup_build
    running recipe: "chromium"
  2. ( 215 ms ) report builders
     
    running builder/tester 'android-cronet-arm-rel' on master 'chromium.android'
  3. ( 42 secs ) bot_update
    [59GB/295GB used (20%)]
  4. ( 0 ) ensure_goma
  5. ( 725 ms ) swarming.py --version
    0.13
  6. ( 1 secs ) clobber
  7. ( 10 secs ) gclient runhooks
  8. ( 1 secs ) read test spec (chromium.android.json)
    path: /b/swarming/w/ir/cache/builder/src/testing/buildbot/chromium.android.json
  9. ( 3 secs ) clean local files
  10. ( 45 ms ) tree truth steps
  11. ( 472 ms ) lookup GN args
     
    Count of GN args (23) exceeds limit (15), presented in logs instead
  12. ( 10 secs ) generate_build_files
  13. ( 13 mins 8 secs ) compile
  14. ( 3 secs ) isolate tests
  15. ( 358 ms ) trigger

Build Properties:

NameValueSource
$build/goma {"jobs": 150} setup_build
$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 "swarm2471-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": 1539337829621707, "id": "8932881011926062736", "project": "chromium", "tags": ["builder:android-cronet-arm-rel", "buildset:commit/git/1d2509fc2ebc7c9848726c1b208483e1e1efc485", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/1d2509fc2ebc7c9848726c1b208483e1e1efc485", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9098311073901803408", "scheduler_job_id:chromium/android-cronet-arm-rel", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "android-cronet-arm-rel" setup_build
buildnumber 5675 setup_build
mastername "chromium.android" setup_build
path_config "generic" setup_build
recipe "chromium" setup_build
repository "https://chromium.googlesource.com/chromium/src.git" setup_build
revision "1d2509fc2ebc7c9848726c1b208483e1e1efc485" setup_build
got_angle_revision "671809e62e807eb0f890a5d53eee1f8cb5fb825e" bot_update
got_buildtools_revision "2dff9c9c74e9d732e6fe57c84ef7fd044cc45d96" bot_update
got_nacl_revision "2ebbdab4bfb1ed9dfe52ce756b34829207dbe25f" bot_update
got_revision "1d2509fc2ebc7c9848726c1b208483e1e1efc485" bot_update
got_revision_cp "refs/heads/master@{#599154}" bot_update
got_swarming_client_revision "486c9b53c4d54dd4b95bb6ce0e31160e600dfc11" bot_update
got_v8_revision "f70aaa8ab2e8815505a6145c745e50d8328cd28c" bot_update
got_v8_revision_cp "refs/heads/7.1.302@{#1}" bot_update
got_webrtc_revision "28d200c24624f0f2ce3013e33effc4ffc123cc2c" bot_update
got_webrtc_revision_cp "refs/heads/master@{#25136}" bot_update
swarm_hashes_refs/heads/master(at){#599154}_without_patch {"cronet_sample_test_apk": "fa733ad40a7c889adf621bfbffece72f8df07159", "cronet_smoketests_missing_native_library_instrumentation_apk": "7db8ea775cc787ac7be62ca76920551f691d3bda", "cronet_smoketests_platform_only_instrumentation_apk": "b3d2e289ab4aaed433f812f425b0eb7ba72ad964", "cronet_test_instrumentation_apk": "cf46c1f6c6dad01dd21a2ba5c508f6c6694f2277", "cronet_tests_android": "fa217a3f74a8592fd1d592d2f43c2e4f5548b5e7", "cronet_unittests_android": "e32c2e3ab922d4545ecaa6cbb8903e3217f36c2f", "net_unittests": "68bfa5590879408dca10c750e51eb7377a589299"} isolate tests

Blamelist:

  1. Yuki Yamada (yukiyohnoyoudont@google.com)
  2. Gauthier Ambard (gambardohnoyoudont@chromium.org)

Timing:

Create Friday, 12-Oct-18 09:50:29 UTC
Start Friday, 12-Oct-18 09:50:33 UTC
End Friday, 12-Oct-18 10:05:31 UTC
Pending 4 secs
Execution 14 mins 57 secs

All Changes:

  1. Added security check for cross origin

    Changed by Yuki Yamada - yukiyohnoyoudont@google.com
    Changed at Friday, 12-Oct-18 09:45:43 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 1d2509fc2ebc7c9848726c1b208483e1e1efc485

    Comments

    Added security check for cross origin
    
    This CL adds a security check for cross origin with
    BindingSecurity::ShouldAllowAccessToCreationContext().
    |js_event|, a V8 wrapper object for event object, must be created in the
    relevant realm of the event target, but it is possible that listener's
    relevant context cannnot access the relevant realm of event target
    (ex. when Document.origin is changed).
    We have to check this before invoking event listener.
    
    Bug: 872138, 884516
    Change-Id: Ic5d0c8e6cda4db57a2097ce230e75cc59905b350
    Reviewed-on: https://chromium-review.googlesource.com/c/1270300
    Commit-Queue: Yuki Yamada <yukiy@google.com>
    Reviewed-by: Kentaro Hara <haraken@chromium.org>
    Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
    Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599154}

    Changed files

    • third_party/blink/renderer/bindings/core/v8/js_based_event_listener.cc
  2. [iOS] Support Dynamic Type for ReadingList empty string

    Changed by Gauthier Ambard - gambardohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 09:41:43 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision fe100b299e7bc889ccafc03cf8854efe64020679

    Comments

    [iOS] Support Dynamic Type for ReadingList empty string
    
    This CL changes the string displayed when the ReadingList is empty so
    it doesn't overflow below the bottom and top toolbar when the font is
    too big. It does that by limiting the font size.
    
    Bug: 893535
    Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
    Change-Id: I03335f2c1fee192a765b6e85011627c0f5ae2019
    Reviewed-on: https://chromium-review.googlesource.com/c/1276925
    Commit-Queue: Gauthier Ambard <gambard@chromium.org>
    Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599153}

    Changed files

    • ios/chrome/browser/ui/reading_list/empty_reading_list_message_util.mm
    • ios/chrome/browser/ui/reading_list/reading_list_table_view_controller.mm