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

0 views

Skip to first unread message

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

unread,

Jun 7, 2018, 2:22:58 PM6/7/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,generate_build_files,steps" on "Browser Side Navigation Linux"

https://ci.chromium.org/buildbot/chromium.fyi/Browser%20Side%20Navigation%20Linux/50999

Revision: d6eb0c60433edd4f0bc0b6435a57a9143a6ff265, c7c852a5282c651364eed69e2a951ce7163e8018, 48a689e2976d5b507bf6544a784a9228c27c4a14, 75d0099408d0710b1c3d9b1b2603d0d36518b7c6, 884aeb2e2d562191db63c99640b0c171a71f7775
Blame list: cth...@chromium.org,dgo...@chromium.org,elly...@chromium.org,ijam...@chromium.org,morl...@chromium.org

Build 50999

running steps via annotated script
preamble
stdio

update_scripts
stdio
gclient_json
build: 08817a42bd36df40b3742ff1b71c010873dd7c83
depot_tools: e89dcf7dee37d535a9180823f2c8fe33cde1bfcc

LUCI Migration
stdio

LogDog Bootstrap
stdio

setup_build<br>running recipe: "chromium"
stdout
run_recipe

makedirs checkout path
stdout

makedirs checkout path (2)
stdout

remove .cipd
stdout

bot_update<br>[45GB/245GB used (18%)]
stdout
json.output

ensure_goma
stdio

ensure_goma.ensure_installed
stdout
json.output

swarming.py --version<br>0.12
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

Failure reason
stdio
reason

Changed by: ijam...@chromium.org
Changed at: Thu 07 Jun 2018 11:10:20
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: d6eb0c60433edd4f0bc0b6435a57a9143a6ff265
Changed files:

chrome/browser/android/vr/arcore_device/arcore_device.cc
chrome/browser/android/vr/arcore_device/arcore_device.h
chrome/browser/android/vr/arcore_device/arcore_gl.cc
chrome/browser/android/vr/arcore_device/arcore_gl.h
chrome/browser/android/vr/arcore_device/arcore_gl_thread.cc
chrome/browser/android/vr/arcore_device/arcore_gl_thread.h
chrome/browser/android/vr/arcore_device/arcore_impl.h
chrome/browser/geolocation/geolocation_permission_context_android.cc
chrome/browser/media/webrtc/media_stream_devices_controller.cc
chrome/browser/permissions/permission_update_infobar_delegate_android.cc
chrome/browser/permissions/permission_update_infobar_delegate_android.h
chrome/browser/vr/service/vr_display_host.cc
chrome/browser/vr/service/vr_display_host.h
device/vr/public/mojom/vr_service.mojom
device/vr/test/mock_vr_display_impl.cc
device/vr/test/mock_vr_display_impl.h
device/vr/vr_device.h
device/vr/vr_device_base.cc
device/vr/vr_device_base.h
device/vr/vr_device_base_unittest.cc
device/vr/vr_display_impl.cc
device/vr/vr_display_impl.h
device/vr/vr_display_impl_unittest.cc
third_party/blink/renderer/modules/xr/xr_device.cc

Comments:

Request camera permission when AR features are requested.

Adds a new XRSessionRequest structure to be passed when the
RequestSession call is made to be able to pass information like
user gesture. The RequestSession now requests camera permission
in ARCoreDevice. The code to ask for camera permission has been
copied and adapted from
src/chrome/browser/media/webrtc/media_stream_devices_controller.cc
and a re-check for the Android OS level camera permission is still
needed.

Bug: 835037, 845160, 775372

...skip...
ts_rel;master.tryserver.chromium.linux:linux_vr
Change-Id: Iecabc3927beb6106a1ada5289f3af3ecffd48ed8
Reviewed-on: https://chromium-review.googlesource.com/1055677
Commit-Queue: Iker Jamardo <ijam...@chromium.org>
Reviewed-by: David Dorwin <ddo...@chromium.org>
Reviewed-by: Daniel Cheng <dch...@chromium.org>
Reviewed-by: Bill Orr <bil...@chromium.org>
Reviewed-by: Raymes Khoury <ray...@chromium.org>
Reviewed-by: Michael Thiessen <mthi...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565347}

Properties:

git_revision: d6eb0c60433edd4f0bc0b6435a57a9143a6ff265

Changed by: morl...@chromium.org
Changed at: Thu 07 Jun 2018 11:13:24
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: c7c852a5282c651364eed69e2a951ce7163e8018
Changed files:

chrome/browser/android/feed/feed_host_service_factory.cc
chrome/browser/favicon/large_icon_service_factory.cc
chrome/browser/ntp_snippets/content_suggestions_service_factory.cc
chrome/browser/ntp_snippets/contextual_content_suggestions_service_factory.cc
chrome/browser/ntp_tiles/chrome_most_visited_sites_factory.cc
chrome/browser/search/suggestions/image_fetcher_impl_browsertest.cc
chrome/browser/search/suggestions/suggestions_service_factory.cc
chrome/browser/search/thumbnail_source.cc
chrome/browser/signin/chrome_signin_client.cc
chrome/browser/signin/chrome_signin_client.h
components/favicon/ios/DEPS
components/favicon/ios/web_favicon_driver.mm
components/feed/core/feed_image_manager_unittest.cc
components/image_fetcher/DEPS
components/image_fetcher/core/BUILD.gn
components/image_fetcher/core/image_data_fetcher.cc
components/image_fetcher/core/image_data_fetcher.h
components/image_fetcher/core/image_data_fetcher_unittest.cc
components/image_fetcher/core/image_fetcher.h
components/image_fetcher/core/image_fetcher_impl.cc
components/image_fetcher/core/image_fetcher_impl.h
components/image_fetcher/core/image_fetcher_impl_unittest.cc
components/image_fetcher/ios/BUILD.gn
components/image_fetcher/ios/ios_image_data_fetcher_wrapper.h
components/image_fetcher/ios/ios_image_data_fetcher_wrapper.mm
components/image_fetcher/ios/ios_image_data_fetcher_wrapper_unittest.mm
components/ntp_snippets/remote/cached_image_fetcher_unittest.cc
components/signin/core/browser/BUILD.gn
components/signin/core/browser/DEPS
components/signin/core/browser/account_fetcher_service.cc
components/signin/core/browser/account_fetcher_service.h
components/signin/core/browser/account_tracker_service_unittest.cc
components/signin/core/browser/signin_client.h
components/signin/core/browser/test_signin_client.cc
components/signin/core/browser/test_signin_client.h
ios/chrome/browser/favicon/ios_chrome_large_icon_service_factory.cc
ios/chrome/browser/ntp_snippets/ios_chrome_content_suggestions_service_factory_util.cc
ios/chrome/browser/ntp_tiles/ios_most_visited_sites_factory.cc
ios/chrome/browser/passwords/notify_auto_signin_view_controller.h
ios/chrome/browser/passwords/notify_auto_signin_view_controller.mm
ios/chrome/browser/passwords/password_controller.mm
ios/chrome/browser/payments/BUILD.gn
ios/chrome/browser/payments/ios_payment_instrument_finder.h
ios/chrome/browser/payments/ios_payment_instrument_finder.mm
ios/chrome/browser/payments/ios_payment_instrument_finder_unittest.mm
ios/chrome/browser/payments/payment_request.mm
ios/chrome/browser/signin/ios_chrome_signin_client.h
ios/chrome/browser/signin/ios_chrome_signin_client.mm
ios/chrome/browser/suggestions/suggestions_service_factory.mm
ios/chrome/browser/ui/browser_view_controller.mm
ios/chrome/browser/ui/omnibox/popup/omnibox_popup_coordinator.mm
ios/web_view/internal/DEPS
ios/web_view/internal/signin/ios_web_view_signin_client.h
ios/web_view/internal/signin/ios_web_view_signin_client.mm
ios/web_view/internal/signin/web_view_signin_client_factory.mm

Comments:

Revert "S13n: Port components/image_fetcher to SimpleURLLoader."

This reverts commit d6f27e51c78500e2530a255ea3f822fda7cbbd6d.

Reason for revert: image_fetcher is used from more than UI thread,
and SharedURLLoaderFactory isn't cross-thread.

Note that this isn't a 100% mechanical revert since it needed a merge
with some ImageDataFetcher cleanups; I also kept in some iOS glue bits
that are used in a pending CL and aren't directly related to the substance
of the revert.

Bug: 850089

Original change's
...skip...


# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 843205
Change-Id: Ifeb0c24a9e83bf271be3f10cb32f6d81ea0d0832
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.linux:linux_mojo;master.tryserver.chromium.mac:ios-simulator-cronet
Reviewed-on: https://chromium-review.googlesource.com/1090930
Commit-Queue: Maks Orlovich <morl...@chromium.org>
Reviewed-by: Maks Orlovich <morl...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565348}

Properties:

git_revision: c7c852a5282c651364eed69e2a951ce7163e8018

Changed by: elly...@chromium.org
Changed at: Thu 07 Jun 2018 11:14:35
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 48a689e2976d5b507bf6544a784a9228c27c4a14
Changed files:

chrome/browser/site_per_process_interactive_browsertest.cc
chrome/browser/ui/browser_command_controller_interactive_browsertest.cc

Comments:

macviews: disable a few fullscreen tests

These tests are hitting some corner case of MacViews fullscreen.

TBR=s...@chromium.org

Bug: 850594
Change-Id: I7ee54c3d1076ef29007c89ff573013a0514ace1a
Reviewed-on: https://chromium-review.googlesource.com/1091136
Reviewed-by: Elly Fong-Jones <elly...@chromium.org>
Commit-Queue: Elly Fong-Jones <elly...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565349}

Properties:

git_revision: 48a689e2976d5b507bf6544a784a9228c27c4a14

Changed by: cth...@chromium.org
Changed at: Thu 07 Jun 2018 11:14:40
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 75d0099408d0710b1c3d9b1b2603d0d36518b7c6
Changed files:

chrome/browser/ui/views/location_bar/location_bar_view.cc

Comments:

Wire lock-only HTTPS indicator to ExperimentalUi flag

This lets the #ExperimentalUi flag control showing only the lock icon for
SECURE (non-EV) sites.

Change-Id: I8e807ada60c2e8ccbd9fb560f2148e87f863d74c
Reviewed-on: https://chromium-review.googlesource.com/1080238
Commit-Queue: Christopher Thompson <cth...@chromium.org>
Reviewed-by: Evan Stade <est...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565350}

Properties:

git_revision: 75d0099408d0710b1c3d9b1b2603d0d36518b7c6

Changed by: dgo...@chromium.org
Changed at: Thu 07 Jun 2018 11:16:02
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 884aeb2e2d562191db63c99640b0c171a71f7775
Changed files:

chrome/browser/devtools/chrome_devtools_manager_delegate.cc
chrome/browser/devtools/chrome_devtools_manager_delegate.h
chrome/browser/devtools/devtools_sanity_browsertest.cc
chrome/browser/devtools/devtools_targets_ui.cc
chrome/browser/devtools/devtools_window.cc
chrome/browser/devtools/devtools_window.h
chrome/browser/extensions/api/developer_private/inspectable_views_finder.cc
chrome/browser/profiles/profile_manager.cc
chrome/browser/renderer_context_menu/render_view_context_menu.cc
chrome/browser/renderer_context_menu/render_view_context_menu.h
chrome/browser/ui/browser.cc
chrome/browser/ui/browser_command_controller.cc
chrome/browser/ui/browser_command_controller_unittest.cc
chrome/test/data/devtools/extensions/options.crx
content/browser/devtools/render_frame_devtools_agent_host.cc
content/public/browser/devtools_manager_delegate.cc
content/public/browser/devtools_manager_delegate.h

Comments:

[DevTools] Respect kDisallowedForForceInstalledExtensions policy value

Added a helper to ChromeDevToolsManagerDelegate and DevToolsWindow
to check whether inspection is allowed. This helper is used to:
- not open DevToolsWindow;
- not show "Inspect Element" context menu item;
- disable DevTools-related browser commands;
- not show inspectable views on chrome://extensions;
- not show extensions on chrome://inspect.

RenderFrameDevToolsAgentHost also disallows attaching a debugging session
when embedde
...skip...
otocol or remote debugging.

Bug: 838146
Test: browser_tests --gtest_filter=DevToolsSanity*Policy*

Change-Id: I59f255cc89919f2568170f4854955ba95eadae7f
Reviewed-on: https://chromium-review.googlesource.com/1069411
Reviewed-by: Avi Drissman <a...@chromium.org>
Reviewed-by: Devlin <rdevlin...@chromium.org>
Reviewed-by: Pavol Marko <pma...@chromium.org>
Reviewed-by: Pavel Feldman <pfel...@chromium.org>
Commit-Queue: Dmitry Gozman <dgo...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565351}

Properties:

git_revision: 884aeb2e2d562191db63c99640b0c171a71f7775

Reply all

Reply to author

Forward

0 new messages