Builder android-cronet-x86-rel Build 12879

Results:

Success

Trigger Info:

Project	chromium
Revision	911282f872704c7ee55683249c13fe231fb5498e
Got Revision	911282f872704c7ee55683249c13fe231fb5498e

Execution:

• Source: Task 4292d04199339a10
• Bot: swarm2479-c4
• Recipe: chromium

Steps and Logfiles:

Show: Expanded Default Non-Green

1. ( 315 ms ) setup_build

   running recipe: "chromium"
   • stdout
   • run_recipe
2. ( 216 ms ) report builders

    

   running builder/tester 'android-cronet-x86-rel' on master 'chromium.android'
   • stdout
   • bots.json
3. ( 48 secs ) bot_update

   [64GB/295GB used (21%)]
   • stdout
   • json.output
4. ( 67 ms ) ensure_goma
   • - no logs -
   1. ( 67 ms ) ensure_installed
      • stdout
      • json.output
5. ( 702 ms ) swarming.py --version

   0.14
   • stdout
6. ( 1 secs ) clobber
   • stdout
7. ( 11 secs ) gclient runhooks
   • stdout
8. ( 2 secs ) get compile targets for scripts
   • stdout
   • json.output
9. ( 1 secs ) read test spec (chromium.android.json)

   path: /b/swarming/w/ir/cache/builder/src/testing/buildbot/chromium.android.json
   • stdout
   • json.output
10. ( 3 secs ) clean local files
    • stdout
    • python.inline
11. ( 77 ms ) tree truth steps
    • stdout
12. ( 466 ms ) lookup GN args

     

    Count of GN args (23) exceeds limit (15), presented in logs instead
    • stdout
    • gn_args
13. ( 11 secs ) generate_build_files
    • stdout
14. ( 3 secs ) preprocess_for_goma
    • - no logs -
    1. ( 430 ms ) goma cache directory
       • stdout
    2. ( 1 secs ) start_goma
       • stdout
       • cloudtail
    3. ( 1 secs ) start cloudtail
       • stdout
15. ( 11 mins 54 secs ) compile
    • stdout
16. ( 2 secs ) compile confirm no-op
    • stdout
17. ( 10 secs ) postprocess_for_goma
    • - no logs -
    1. ( 488 ms ) goma_jsonstatus
       • stdout
       • json.output
    2. ( 485 ms ) goma_stat
       • stdout
    3. ( 487 ms ) stop_goma
       • stdout
    4. ( 7 secs ) upload_log
       • stdout
       • json.output
       • compiler_proxy_log
       • ninja_log
    5. ( 1 secs ) stop cloudtail
       • stdout
18. ( 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	"swarm2479-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": 1548234962440302, "id": "8923551691982647744", "project": "chromium", "tags": ["builder:android-cronet-x86-rel", "buildset:commit/git/911282f872704c7ee55683249c13fe231fb5498e", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/911282f872704c7ee55683249c13fe231fb5498e", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9088981754282521808", "scheduler_job_id:chromium/android-cronet-x86-rel", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"}	setup_build
buildername	"android-cronet-x86-rel"	setup_build
buildnumber	12879	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	"911282f872704c7ee55683249c13fe231fb5498e"	setup_build
got_angle_revision	"5313c8a89964f536bff2c7f7943499f9c52e5664"	bot_update
got_buildtools_revision	"2f02e1f363b1af2715536f38e239853f04ec1497"	bot_update
got_dawn_revision	"c0f31e73e481f32d308cb6232e75e6d860d97063"	bot_update
got_nacl_revision	"ff15c51d20785fa51c0ef03ccb480b87299acbeb"	bot_update
got_revision	"911282f872704c7ee55683249c13fe231fb5498e"	bot_update
got_revision_cp	"refs/heads/master@{#625142}"	bot_update
got_swarming_client_revision	"0e3e1c4dc4e79f25a5b58fcbc135dc93183c0c54"	bot_update
got_v8_revision	"f6d33d43027bc31ea10b1856356e7ac6d829bbad"	bot_update
got_v8_revision_cp	"refs/heads/7.3.487@{#1}"	bot_update
got_webrtc_revision	"71b5a7df7794bbc4103296fcd8bd740acebdc901"	bot_update
got_webrtc_revision_cp	"refs/heads/master@{#26351}"	bot_update

Blamelist:

1. Matt Falkenhagen (falkenohnoyoudont@chromium.org)

Timing:

Create	Wednesday, 23-Jan-19 09:16:02 UTC
Start	Wednesday, 23-Jan-19 09:16:06 UTC
End	Wednesday, 23-Jan-19 09:29:52 UTC
Pending	3 secs
Execution	13 mins 46 secs

All Changes:

1. Make blink::SpaceSplitString thread-safe.

   Changed by	Matt Falkenhagen - falkenohnoyoudont@chromium.org
   Changed at	Wednesday, 23-Jan-19 09:11:26 UTC
   Repository	https://chromium.googlesource.com/chromium/src
   Branch
   Revision	911282f872704c7ee55683249c13fe231fb5498e

   Comments

   Make blink::SpaceSplitString thread-safe.
   
   This fixes a race because both main thread and worker threads parse CSP
   which uses blink::CSPDirectiveList which uses blink::SpaceSplitString.
   
   Bug: 807838
   Change-Id: Id1eafe17daf3d37f9c1b6fc64aaa18ea964c57bc
   Reviewed-on: https://chromium-review.googlesource.com/c/1429819
   Reviewed-by: Kent Tamura <tkent@chromium.org>
   Commit-Queue: Matt Falkenhagen <falken@chromium.org>
   Cr-Commit-Position: refs/heads/master@{#625142}

   Changed files

   • third_party/blink/renderer/core/dom/space_split_string.cc
   • third_party/blink/web_tests/TestExpectations