Builder android-cronet-arm64-rel Build 5257

Results:

Success

Trigger Info:

Project	chromium
Revision	0edf161ea086a970d14c9946ded919743e693731
Got Revision	0edf161ea086a970d14c9946ded919743e693731

Execution:

Steps and Logfiles:

Show: Expanded Collapsed Non-Green

( 308 ms ) setup_build
running recipe: "chromium"
- stdout
- run_recipe
( 218 ms ) report builders

running builder/tester 'android-cronet-arm64-rel' on master 'chromium.android'
- stdout
( 43 secs ) bot_update
[60GB/295GB used (20%)]
- stdout
- json.output
( 124 ms ) ensure_goma
- - no logs -
1. ( 124 ms ) ensure_installed
  - stdout
  - json.output
( 736 ms ) swarming.py --version
0.13
- stdout
( 1 secs ) clobber
- stdout
( 10 secs ) gclient runhooks
- stdout
( 2 secs ) get compile targets for scripts
- stdout
- json.output
( 1 secs ) read test spec (chromium.android.json)
path: /b/swarming/w/ir/cache/builder/src/testing/buildbot/chromium.android.json
- stdout
- json.output
( 3 secs ) clean local files
- stdout
- python.inline
( 158 ms ) tree truth steps
- stdout
( 422 ms ) clean isolated files
- stdout
( 483 ms ) lookup GN args

Count of GN args (22) exceeds limit (15), presented in logs instead
- stdout
- gn_args
( 38 secs ) generate_build_files
- stdout
- swarming-targets-file.txt
( 3 secs ) preprocess_for_goma
- - no logs -
1. ( 429 ms ) goma cache directory
  - stdout
2. ( 1 secs ) start_goma
  - stdout
  - cloudtail
3. ( 1 secs ) start cloudtail
  - stdout
( 10 mins 17 secs ) compile
- stdout
( 2 secs ) compile confirm no-op
- stdout
( 10 secs ) postprocess_for_goma
- - no logs -
1. ( 495 ms ) goma_jsonstatus
  - stdout
  - json.output
2. ( 486 ms ) goma_stat
  - stdout
3. ( 517 ms ) stop_goma
  - stdout
4. ( 7 secs ) upload_log
5. ( 1 secs ) stop cloudtail
  - stdout
( 3 secs ) isolate tests
- stdout
- json.output
( 319 ms ) trigger
- stdout
( 0 ) recipe result
- result

Build Properties:

Name	Value	Source
$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	"swarm2477-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": 1539354733778114, "id": "8932863286631965792", "project": "chromium", "tags": ["builder:android-cronet-arm64-rel", "buildset:commit/git/0edf161ea086a970d14c9946ded919743e693731", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/0edf161ea086a970d14c9946ded919743e693731", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9098293348796175056", "scheduler_job_id:chromium/android-cronet-arm64-rel", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"}	setup_build
buildername	"android-cronet-arm64-rel"	setup_build
buildnumber	5257	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	"0edf161ea086a970d14c9946ded919743e693731"	setup_build
got_angle_revision	"671809e62e807eb0f890a5d53eee1f8cb5fb825e"	bot_update
got_buildtools_revision	"2dff9c9c74e9d732e6fe57c84ef7fd044cc45d96"	bot_update
got_nacl_revision	"2ebbdab4bfb1ed9dfe52ce756b34829207dbe25f"	bot_update
got_revision	"0edf161ea086a970d14c9946ded919743e693731"	bot_update
got_revision_cp	"refs/heads/master@{#599203}"	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	"c34cf71d601759149a79cd9d59293397009f7b0e"	bot_update
got_webrtc_revision_cp	"refs/heads/master@{#25140}"	bot_update
swarm_hashes_refs/heads/master(at){#599203}_without_patch	{"cronet_sample_test_apk": "0c9ed8f79aed707a476b75626eb37d0c70436892", "cronet_smoketests_missing_native_library_instrumentation_apk": "21cec4b4df6ec08398bb60d53a664d1b17cb6adc", "cronet_smoketests_platform_only_instrumentation_apk": "865549e1e4c7e8b4c690670b7d7c63287dcef55d", "cronet_test_instrumentation_apk": "7953887336a4f0584ee4d9141bb49b33fc99d63e", "cronet_tests_android": "bdf1d1fefe72ce48efb8c7a131b0d3b5b9ada76a", "cronet_unittests_android": "43e35b8c13b0aac7cd6fd83faabdf27b8e746d87", "net_unittests": "d7b7bb5b57d1169eaf4e43bd04700dd1b793bacb"}	isolate tests

Blamelist:

Sergey Kuznetsov (kuznetsovsohnoyoudont@yandex-team.ru)
Mihai Sardarescu (msardaohnoyoudont@chromium.org)
Matthew Cary (mattcaryohnoyoudont@chromium.org)

Timing:

Create	Friday, 12-Oct-18 14:32:13 UTC
Start	Friday, 12-Oct-18 14:32:15 UTC
End	Friday, 12-Oct-18 14:44:52 UTC
Pending	1 secs
Execution	12 mins 36 secs

All Changes:

Fix possible crash when |change_dispatcher_| gone.

Changed by	Sergey Kuznetsov - kuznetsovsohnoyoudont@yandex-team.ru
Changed at	Friday, 12-Oct-18 14:29:05 UTC
Repository	https://chromium.googlesource.com/chromium/src
Branch
Revision	0edf161ea086a970d14c9946ded919743e693731

Comments

Fix possible crash when |change_dispatcher_| gone.

Nothing prevents CookieMonsterChangeDispatcher::Subscription to
outlive CookieMonsterChangeDispatcher.

Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs
Change-Id: Ifa2cd0c8def9ae1c63a0a3d37bd469bbbc78e5ca
Reviewed-on: https://chromium-review.googlesource.com/c/1242460
Commit-Queue: Maks Orlovich <morlovich@chromium.org>
Reviewed-by: David Roger <droger@chromium.org>
Reviewed-by: Maks Orlovich <morlovich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599203}

Changed files

ios/net/cookies/cookie_store_ios.h
ios/net/cookies/cookie_store_ios.mm
net/cookies/cookie_monster_change_dispatcher.cc
net/cookies/cookie_store_change_unittest.h
net/cookies/cookie_store_unittest.h

Rename method in token service diagnostic observer for clarity

Changed by	Mihai Sardarescu - msardaohnoyoudont@chromium.org
Changed at	Friday, 12-Oct-18 14:27:41 UTC
Repository	https://chromium.googlesource.com/chromium/src
Branch
Revision	466b727ac3cd8f967fed05a86562c01616824627

Comments

Rename method in token service diagnostic observer for clarity

Bug: NONE
Change-Id: I18d05d59ba0be38d7598396fab2cef8b44647fcf
Reviewed-on: https://chromium-review.googlesource.com/c/1278754
Reviewed-by: Thomas Tangl <tangltom@chromium.org>
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Commit-Queue: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599202}

Changed files

components/signin/core/browser/about_signin_internals.cc
components/signin/core/browser/about_signin_internals.h
google_apis/gaia/oauth2_token_service.cc
google_apis/gaia/oauth2_token_service.h

Orderfile: skip madvise if ordering has not been done.

Changed by	Matthew Cary - mattcaryohnoyoudont@chromium.org
Changed at	Friday, 12-Oct-18 14:19:58 UTC
Repository	https://chromium.googlesource.com/chromium/src
Branch
Revision	ea3132cbbc836b398e523438627cc27349505196

Comments

Orderfile: skip madvise if ordering has not been done.

Change a CHECK to an logged warning if madvise is attempted when code
ordering has not been performed. This happens on upstream builds
including arm64 builds.

Bug: 894748
Change-Id: Icdb7ae0c94cfad5beec0215c6186e29135c1079a
Reviewed-on: https://chromium-review.googlesource.com/c/1278746
Reviewed-by: Egor Pasko <pasko@chromium.org>
Reviewed-by: Matthew Cary <mattcary@chromium.org>
Reviewed-by: Benoit L <lizeb@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Commit-Queue: Matthew Cary <mattcary@chromium.org>
Cr-Commit-Position: refs/heads/master@{#599201}

Changed files

base/android/library_loader/library_prefetcher.cc