buildbot failure in chromium.fyi on Browser Side Navigation Linux, revision fee5eca36f4966df5e10f34c75aa9ca2b5c21ec2

2 views
Skip to first unread message

buil...@chromium-build.appspotmail.com

unread,
Mar 22, 2018, 5:08:27 PM3/22/18
to plzna...@chromium.org, gateke...@chromium-gatekeeper-sentry.appspotmail.com
https://build.chromium.org/p/chromium.fyi/waterfall

Automatically closing tree for "Failure reason,compile,steps" on "Browser Side Navigation Linux"

https://build.chromium.org/p/chromium.fyi/builders/Browser%20Side%20Navigation%20Linux/builds/41718

Revision: fee5eca36f4966df5e10f34c75aa9ca2b5c21ec2, a48f241bc00414b325b0b682ca6d38028c422299, 0571def5265a85b68aac825eec042930a38de02e, 88415c81bd2e440871e40161a12efe6aceeba83b, 6a9c9b66a1912fcf4900c815f10cadded65df3dc, 0ca55b814e692b9d6e53919a0764eb392f917239, b23ffdff9c21cfe220d739bd3ada55d3fd6a6924, af36705706f58893c1e3ec0da2c975ff1fb73d76, 23777d97c2cbf0946190d2c72d98389026ff456f, 2766cd5c984fe26c0b46bda9feea595aeb064e2b, feab23606d5a61eb822d40f5ab80707dc319d001, fb03e26f7e6f44d034671c2d3b931e062870403b
Blame list: a...@chromium.org,almas...@google.com,cliffo...@chromium.org,jbud...@chromium.org,micha...@chromium.org,mich...@chromium.org,morl...@chromium.org,sa...@chromium.org,shengh...@chromium.org,v8-au...@chromium.org,yi...@chromium.org

Build 41718
running steps via annotated script
preamble
stdio
update_scripts
stdio
gclient_json
build: 054b33ca03663b2882713ae68a267dc38e357412
depot_tools: c621b21ad4bfce59924ffc0116eff58bcd4b8bc0
LUCI Migration
stdio
LogDog Bootstrap
stdio
setup_build<br>running recipe: "chromium"
stdout
run_recipe
makedirs checkout path
stdout
bot_update<br>[44GB/245GB used (17%)]
stdout
json.output
ensure_goma
stdio
ensure_goma.ensure_installed
stdout
json.output
swarming.py --version<br>0.10.2
stdout
gclient runhooks
stdout
get compile targets for scripts
stdout
json.output
read test spec (chromium.fyi.json)<br>path: /mnt/data/b/c/builder/Browser_Side_Navigation_Linux/src/testing/buildbot/chromium.fyi.json
stdout
json.output
generate_build_files
stdout
calculate the number of recommended jobs
stdout
preprocess_for_goma
stdio
preprocess_for_goma.goma cache directory
stdout
preprocess_for_goma.start_goma
stdout
cloudtail
preprocess_for_goma.start cloudtail
stdout
compile
stdout
json.output[ninja_info]
raw_io.output[failure_summary]
postprocess_for_goma
stdio
postprocess_for_goma.goma_jsonstatus
stdout
json.output
postprocess_for_goma.goma_stat
stdout
postprocess_for_goma.stop_goma
stdout
postprocess_for_goma.upload_log
stdout
json.output
ninja_log
compiler_proxy_log
postprocess_for_goma.stop cloudtail
stdout
Failure reason
stdio
reason

Changed by: shengh...@chromium.org
Changed at: Thu 22 Mar 2018 13:19:02
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: fee5eca36f4966df5e10f34c75aa9ca2b5c21ec2
Changed files:

  • ios/build/bots/chromium.fyi/ios-webview.json
Comments:
[ios] Support ios_deployment_target '9.0' to ios-webview.json

chromium.fyi builder ios-webview still runs tests targeting to simulator
os 9.3. Add the config to support the test target app be built for os
9.0.

Bug: 817060
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I44773540bf11a1506da159b2b779c3675ac648d0
Reviewed-on: https://chromium-review.googlesource.com/976376
Reviewed-by: Sergey Berezin <sergey...@chromium.org>
Commit-Queue: Shenghua Zhang <shengh...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545210}
Properties:
  • git_revision: fee5eca36f4966df5e10f34c75aa9ca2b5c21ec2

Changed by: v8-au...@chromium.org
Changed at: Thu 22 Mar 2018 13:19:11
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: a48f241bc00414b325b0b682ca6d38028c422299
Changed files:

  • DEPS
Comments:
Update V8 to version 6.7.142.

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/61c909d6..171721c4

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel

TBR=hab...@chromium.org,mache...@chromium.org,kozyat...@chromium.org,ser...@chromium.org

Change-Id: I67f634b16cecb7044958ab4124a283e21e258de3
Reviewed-on: https://chromium-review.googlesource.com/976141
Reviewed-by: v8 autoroll <v8-au...@chromium.org>
Commit-Queue: v8 autoroll <v8-au...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545211}
Properties:
  • git_revision: a48f241bc00414b325b0b682ca6d38028c422299

Changed by: micha...@chromium.org
Changed at: Thu 22 Mar 2018 13:20:10
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 0571def5265a85b68aac825eec042930a38de02e
Changed files:

  • android_webview/java/src/org/chromium/android_webview/AwAutofillProvider.java
  • android_webview/javatests/src/org/chromium/android_webview/test/AwAutofillTest.java
Comments:
[WebView Autofill] Not call cancel on Android P

Bug: 823424
Change-Id: I6976635c26c2a8eff2a95ae3ecc5f5fad930e52b
Reviewed-on: https://chromium-review.googlesource.com/969142
Reviewed-by: Changwan Ryu <chan...@chromium.org>
Commit-Queue: Tao Bai <micha...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545212}
Properties:
  • git_revision: 0571def5265a85b68aac825eec042930a38de02e

Changed by: a...@chromium.org
Changed at: Thu 22 Mar 2018 13:20:22
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 88415c81bd2e440871e40161a12efe6aceeba83b
Changed files:

  • content/browser/webauth/authenticator_impl.cc
  • content/browser/webauth/authenticator_impl_unittest.cc
  • device/fido/attestation_object.cc
  • device/fido/attestation_object.h
  • device/fido/attestation_statement.cc
  • device/fido/attestation_statement.h
  • device/fido/fido_attestation_statement.cc
  • device/fido/fido_attestation_statement.h
  • device/fido/register_response_data.cc
  • device/fido/register_response_data.h
Comments:
webauthn: suppress inappropriately identifying attestation certificates.

Some tokens return uniquely identifying attestation certificates, even
when that bit isn't set in the registration request. In order to prevent
these being trackable, this change identifies and suppresses such
certificates by replacing them with a “none” attestation.

An exception is made for RPs listed in the enterprise policy. In that
case we set the bit to request individual attestation already, therefore
there's no issue with returning individual certificates.

Bug: 793985
Change-Id: I8538d749ff9e2baa6d4ab566fa314673ff6e5b0e
Reviewed-on: https://chromium-review.googlesource.com/952541
Commit-Queue: Adam Langley <a...@chromium.org>
Reviewed-by: Balazs Engedy <eng...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545213}
Properties:
  • git_revision: 88415c81bd2e440871e40161a12efe6aceeba83b

Changed by: almas...@google.com
Changed at: Thu 22 Mar 2018 13:21:15
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 6a9c9b66a1912fcf4900c815f10cadded65df3dc
Changed files:

  • chromecast/media/cma/backend/video/av_sync_video.cc
Comments:
[Chromecast] Fix user builds

Since GatherPlaybackStatistics invocation is guarded by IS_DCHECK_ON,
the declaration of kPlaybackStatisticsCheckInterval needs to be so too,
otherwise we get an unused variable build error.

BUG=None
TEST=I can now build user builds

Change-Id: I8f8bbeae0640a4d89d5fcde0fc9e8733996f50b6
Reviewed-on: https://chromium-review.googlesource.com/976523
Reviewed-by: Kenneth MacKay <kma...@chromium.org>
Commit-Queue: Mina Almasry <almas...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545214}
Properties:
  • git_revision: 6a9c9b66a1912fcf4900c815f10cadded65df3dc

Changed by: yi...@chromium.org
Changed at: Thu 22 Mar 2018 13:21:30
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 0ca55b814e692b9d6e53919a0764eb392f917239
Changed files:

  • components/viz/common/quads/shared_quad_state.h
  • components/viz/service/display/surface_aggregator.cc
  • components/viz/service/display/surface_aggregator.h
  • components/viz/service/display/surface_aggregator_unittest.cc
Comments:
Fix regression: fix the unwanted black screen during resize

In DrawOcclusion, I made the assumption that the |quad_layer_rect|
represents the union of the |rect| of DrawQuads in this SharedQuadState.
With this assumption, I can use |quad_layer_rect| to occlude DrawQuads
behind it. However this assumption was not well documented, so that
some code was written after the draw occlusion algorithm did not
follow it, ex: Gutter is using the size of the surfaceDrawQuad as
|quad_layer_rect| and the actual |rect| size is very small.

Bug:824037, 824035, 824043, 824054

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I4cc02559926c02a3a769050250f1753c37f65307
Reviewed-on: https://chromium-review.googlesource.com/975821
Reviewed-by: Fady Samuel <fsa...@chromium.org>
Commit-Queue: Fady Samuel <fsa...@chromium.org>
Commit-Queue: Yi Xu <yi...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545215}
Properties:
  • git_revision: 0ca55b814e692b9d6e53919a0764eb392f917239

Changed by: sa...@chromium.org
Changed at: Thu 22 Mar 2018 13:22:26
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: b23ffdff9c21cfe220d739bd3ada55d3fd6a6924
Changed files:

  • content/browser/site_per_process_hit_test_browsertest.cc
  • testing/buildbot/filters/viz.content_browsertests.filter
Comments:
Moved feature_list initialization to test SetUp.

Since I didn't want to disable wheel scroll latching in
InputEventRouterWheelTargetTest I moved MainframeWheelEventsOnMainThread
and SubframeWheelEventsOnMainThread tests to a new subclass that
disables WheelScrollLatching in the SetUp function.
The change is for avoiding data race due to initialization of the
feature_list in the tests.

Bug: 822764
Change-Id: I76d9fee1474e8e7a50eca8d311941bcd969dafe3
Reviewed-on: https://chromium-review.googlesource.com/974366
Reviewed-by: Ken Buchanan <ke...@chromium.org>
Commit-Queue: Sahel Sharifymoghaddam <sa...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545216}
Properties:
  • git_revision: b23ffdff9c21cfe220d739bd3ada55d3fd6a6924

Changed by: mich...@chromium.org
Changed at: Thu 22 Mar 2018 13:22:34
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: af36705706f58893c1e3ec0da2c975ff1fb73d76
Changed files:

  • docs/code_reviews.md
Comments:
Minor doc changes for Gerrit

Remove Rietveld references. Refer to Gerrit's "Status" field rather than
the "Name" field.

Bug: None
Change-Id: I4308e2b9cfa8c140d50e24f47b70342b4aa169b4
Reviewed-on: https://chromium-review.googlesource.com/974914
Reviewed-by: Aaron Gable <aga...@chromium.org>
Commit-Queue: Michael Giuffrida <mich...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545217}
Properties:
  • git_revision: af36705706f58893c1e3ec0da2c975ff1fb73d76

Changed by: cliffo...@chromium.org
Changed at: Thu 22 Mar 2018 13:23:32
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 23777d97c2cbf0946190d2c72d98389026ff456f
Changed files:

  • testing/buildbot/gn_isolate_map.pyl
  • testing/buildbot/manage.py
Comments:
Update the target from media_router_tests to media_router_e2e_tests.

Bug: 823346
Change-Id: Iaf364a9d63eac82b3d469076ad3d303ed37fa8bb
Reviewed-on: https://chromium-review.googlesource.com/974116
Reviewed-by: Dirk Pranke <dpr...@chromium.org>
Reviewed-by: mark a. foltz <mfo...@chromium.org>
Commit-Queue: mark a. foltz <mfo...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545218}
Properties:
  • git_revision: 23777d97c2cbf0946190d2c72d98389026ff456f

Changed by: jbud...@chromium.org
Changed at: Thu 22 Mar 2018 13:23:41
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 2766cd5c984fe26c0b46bda9feea595aeb064e2b
Changed files:

  • build/fuchsia/sdk.sha1
  • build/fuchsia/update_sdk.py
  • docs/fuchsia_sdk_updates.md
Comments:
Track fuchsia SDK hash in its own file.

Making life easier for the upcoming autoroller.

Bug: 811425
Change-Id: Ida1c874d311dfb741de6a85631e35068d811ceab
Reviewed-on: https://chromium-review.googlesource.com/970726
Commit-Queue: John Budorick <jbud...@chromium.org>
Reviewed-by: Kevin Marshall <kmar...@chromium.org>
Reviewed-by: Michael Achenbach <mache...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545219}
Properties:
  • git_revision: 2766cd5c984fe26c0b46bda9feea595aeb064e2b

Changed by: mich...@chromium.org
Changed at: Thu 22 Mar 2018 13:24:40
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: feab23606d5a61eb822d40f5ab80707dc319d001
Changed files:

  • chrome/renderer/BUILD.gn
  • chrome/renderer/extensions/chrome_extensions_dispatcher_delegate.cc
  • chrome/renderer/resources/renderer_resources.grd
  • extensions/renderer/BUILD.gn
  • extensions/renderer/dispatcher.cc
  • extensions/renderer/resources/extensions_renderer_resources.grd
  • extensions/renderer/resources/feedback_private_custom_bindings.js
Comments:
Move feedbackPrivate's custom bindings to //extensions

Most of the chrome.feedbackPrivate implementation lives in //extensions,
but the custom bindings used to transform blobs into UUIDs still live in
//chrome.

This just moves them into //extensions so that extensions running in
non-Chrome embedders can attach a file and/or screenshot to the
FeedbackInfo sent via chrome.feedbackPrivate.sendFeedback.

Bug: 824563
Change-Id: Ief8a53905f1d9e9afec66374ea63b611636a8ed4
Reviewed-on: https://chromium-review.googlesource.com/974754
Reviewed-by: Istiaque Ahmed <laz...@chromium.org>
Commit-Queue: Michael Giuffrida <mich...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545220}
Properties:
  • git_revision: feab23606d5a61eb822d40f5ab80707dc319d001

Changed by: morl...@chromium.org
Changed at: Thu 22 Mar 2018 13:24:45
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: fb03e26f7e6f44d034671c2d3b931e062870403b
Changed files:

  • net/extras/sqlite/sqlite_persistent_cookie_store.cc
  • tools/metrics/histograms/enums.xml
Comments:
SQLitePersistentCookieStore: fix enum labels for Cookie.BackingStoreUpdateResults

As far as I can see, the code has been using the same values since 2010:
https://codereview.chromium.org/2718011/
...but the description in enums.xml simply doesn't match them.
Since this (plus its obsolete name) look like the only metric using it,
fixing up enums.xml felt like the right thing to do.

The code was updated to use a standard enum setup rather than numbers to
make this a bit easier to get right (especially since I am considering
adding new values)

Bug: 787679
Change-Id: I491ff543f57a96cdbba02f0ed01935b77960a06f
Reviewed-on: https://chromium-review.googlesource.com/975630
Reviewed-by: Victor Costan <pwn...@chromium.org>
Reviewed-by: Jesse Doherty <j...@chromium.org>
Commit-Queue: Maks Orlovich <morl...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545221}
Properties:
  • git_revision: fb03e26f7e6f44d034671c2d3b931e062870403b

Reply all
Reply to author
Forward
0 new messages