Builder Jumbo Linux x64 Build 30675
Results:
Success
Trigger Info:
Project | chromium |
Revision | 5bc177455c5cdf9c9550e39d243d5a2185d846d5 |
Got Revision | 5bc177455c5cdf9c9550e39d243d5a2185d846d5 |
Execution:
- Source: Task 4294de401e319410
- Bot: swarm1859-c4
- Recipe: chromium
Steps and Logfiles:
Show:-
- - no logs -
-
( 1 secs ) read test spec (chromium.fyi.json)path: /b/swarming/w/ir/cache/builder/src/testing/buildbot/chromium.fyi.json
-
( 429 ms ) lookup GN argsis_component_build = falseis_debug = falsejumbo_file_merge_limit = 50strip_absolute_paths_from_debug_symbols = truesymbol_level = 0use_goma = trueuse_jumbo_build = truegoma_dir = "/b/swarming/w/ir/cache/goma/client"
-
- - no logs -
-
- - no logs -
Build Properties:
Name | Value | Source |
---|---|---|
$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 | "swarm1859-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": 1548269431929121, "id": "8923515548104345840", "project": "chromium", "tags": ["builder:Jumbo Linux x64", "buildset:commit/git/5bc177455c5cdf9c9550e39d243d5a2185d846d5", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/5bc177455c5cdf9c9550e39d243d5a2185d846d5", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9088945610390703504", "scheduler_job_id:chromium/Jumbo Linux x64", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} | setup_build |
buildername | "Jumbo Linux x64" | setup_build |
buildnumber | 30675 | setup_build |
mastername | "chromium.fyi" | setup_build |
path_config | "generic" | setup_build |
recipe | "chromium" | setup_build |
repository | "https://chromium.googlesource.com/chromium/src.git" | setup_build |
revision | "5bc177455c5cdf9c9550e39d243d5a2185d846d5" | setup_build |
got_angle_revision | "47ca1b2fa3990e51a2ced20f7a7d3589a5477ad1" | bot_update |
got_buildtools_revision | "2f02e1f363b1af2715536f38e239853f04ec1497" | bot_update |
got_dawn_revision | "c0f31e73e481f32d308cb6232e75e6d860d97063" | bot_update |
got_nacl_revision | "ff15c51d20785fa51c0ef03ccb480b87299acbeb" | bot_update |
got_revision | "5bc177455c5cdf9c9550e39d243d5a2185d846d5" | bot_update |
got_revision_cp | "refs/heads/master@{#625283}" | bot_update |
got_swarming_client_revision | "0e3e1c4dc4e79f25a5b58fcbc135dc93183c0c54" | bot_update |
got_v8_revision | "21b1ee522c100e983d64754291774d68e7bdc31f" | bot_update |
got_v8_revision_cp | "refs/heads/7.3.492@{#1}" | bot_update |
got_webrtc_revision | "b0397d69a93a909fc0b29ffdb9236bab1e1b2db7" | bot_update |
got_webrtc_revision_cp | "refs/heads/master@{#26368}" | bot_update |
Blamelist:
- Moe Ahmadi (mahmadiohnoyoudont@chromium.org)
- Jun Mukai (mukaiohnoyoudont@chromium.org)
- Jon Kunkee (jkunkeeohnoyoudont@microsoft.com)
- Lei Zhang (thestigohnoyoudont@chromium.org)
- Leonardo Padilha (ldaguilarohnoyoudont@chromium.org)
- Stepan Khapugin (stkhapuginohnoyoudont@chromium.org)
Timing:
Create | Wednesday, 23-Jan-19 18:50:31 UTC |
Start | Wednesday, 23-Jan-19 18:50:38 UTC |
End | Wednesday, 23-Jan-19 18:59:14 UTC |
Pending | 7 secs |
Execution | 8 mins 35 secs |
All Changes:
-
Revert "Enable connectivity checker for fuchsia"
Changed by Moe Ahmadi - mahmadiohnoyoudont@chromium.org Changed at Wednesday, 23-Jan-19 18:48:59 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 5bc177455c5cdf9c9550e39d243d5a2185d846d5 Comments
Revert "Enable connectivity checker for fuchsia" This reverts commit 9df0aa65ec025ddc73b669acf79122b5b3a9825d. Reason for revert: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/fuchsia-x64-cast/11742 https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/fuchsia-arm64-cast/11675 Original change's description: > Enable connectivity checker for fuchsia > > We are using the default connectivity checker, and adding the switch > |--require-wlan| to allow developers to force connections to only WLAN > network interfaces. > > Fuchsia devices typically have an ethernet cable connected for > development, which causes the connectivity checker to always return connected. > For some tests and development, we need to force the usage of WLAN > interfaces, thus this switch was needed. > > Run cast with --require-wlan, observe disconnected > > Bug: 777973 > Test: net_unittests && Run cast with no flags, observe connected && > Change-Id: I359c57a83ce42eb81e5795c0c2c7b711b907471e > Reviewed-on: https://chromium-review.googlesource.com/c/1419903 > Reviewed-by: Paul Jensen <pauljensen@chromium.org> > Reviewed-by: Luke Halliwell <halliwell@chromium.org> > Reviewed-by: Wez <wez@chromium.org> > Commit-Queue: Aidan Wolter <awolter@chromium.org> > Cr-Commit-Position: refs/heads/master@{#625258} TBR=wez@chromium.org,pauljensen@chromium.org,halliwell@chromium.org,awolter@chromium.org Change-Id: I5ea25b3824c74669aa4180706da8d9ada7d244a6 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 777973 Reviewed-on: https://chromium-review.googlesource.com/c/1431015 Reviewed-by: Moe Ahmadi <mahmadi@chromium.org> Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#625283}
Changed files
- chromecast/base/chromecast_switches.cc
- chromecast/base/chromecast_switches.h
- chromecast/browser/cast_browser_main_parts.cc
- chromecast/net/BUILD.gn
- chromecast/net/network_change_notifier_factory_fuchsia.cc
- chromecast/net/network_change_notifier_factory_fuchsia.h
- net/BUILD.gn
- net/base/network_change_notifier.cc
- net/base/network_change_notifier_fuchsia.cc
- net/base/network_change_notifier_fuchsia.h
- net/base/network_change_notifier_fuchsia_unittest.cc
- net/base/network_interfaces_fuchsia.cc
- net/base/network_interfaces_fuchsia.h
-
Support of aspect ratio, max/min sizes for Mash
Changed by Jun Mukai - mukaiohnoyoudont@chromium.org Changed at Wednesday, 23-Jan-19 18:48:18 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 3a0c4c90e5d8456d7c7c975294d8f764deb4d2bd Comments
Support of aspect ratio, max/min sizes for Mash Right now maximum/minimum sizes are computed from views, it's hard to detect when it exactly changes. Because of that, individual client sets those properties for now to configure those metrics propertly. Bug: 921272 Test: manually Change-Id: I062da6e771f7f1c02d3ab07bebd98ceff410b3c8 Reviewed-on: https://chromium-review.googlesource.com/c/1422704 Reviewed-by: Mitsuru Oshima <oshima@chromium.org> Reviewed-by: Tom Sepez <tsepez@chromium.org> Reviewed-by: Scott Violet <sky@chromium.org> Commit-Queue: Jun Mukai <mukai@chromium.org> Cr-Commit-Position: refs/heads/master@{#625282}
Changed files
- ash/wm/non_client_frame_controller.cc
- chrome/browser/ui/views/overlay/overlay_window_views.cc
- services/ws/public/cpp/property_type_converters.cc
- services/ws/public/cpp/property_type_converters.h
- services/ws/public/mojom/window_manager.mojom
- ui/aura/client/aura_constants.cc
- ui/aura/client/aura_constants.h
- ui/aura/mus/property_converter.cc
- ui/aura/mus/property_converter.h
- ui/views/mus/desktop_window_tree_host_mus.cc
- ui/views/mus/desktop_window_tree_host_mus.h
- ui/views/widget/desktop_aura/desktop_window_tree_host_platform.cc
- ui/views/widget/desktop_aura/desktop_window_tree_host_platform.h
-
Roll src/third_party/ffmpeg/ 42bb040dd..4b75b8bab (2 commits)
Changed by Jon Kunkee - jkunkeeohnoyoudont@microsoft.com Changed at Wednesday, 23-Jan-19 18:47:20 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 682c04d8553abbf37a5b7c34ba66776f7c751b00 Comments
Roll src/third_party/ffmpeg/ 42bb040dd..4b75b8bab (2 commits) https://chromium.googlesource.com/chromium/third_party/ffmpeg.git/+log/42bb040dde0a..4b75b8bab993 $ git log 42bb040dd..4b75b8bab --date=short --no-merges --format='%ad %ae %s' 2019-01-10 chcunningham Add ARM64 Windows support to the Chromium FFMPEG repo 2019-01-10 liberato Handle autorename files automatically. Bug: 893460 Change-Id: Iccb2d42f701e5a45e04d1cd47b15f4fb4e00682c Reviewed-on: https://chromium-review.googlesource.com/c/1429385 Reviewed-by: Chrome Cunningham <chcunningham@chromium.org> Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#625281}
Changed files
- DEPS
-
Remove Android-specific printing IPCs.
Changed by Lei Zhang - thestigohnoyoudont@chromium.org Changed at Wednesday, 23-Jan-19 18:46:31 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 03f4a53bc296dd3a65f95c7161b7305587ac392d Comments
Remove Android-specific printing IPCs. Instead, print more like as other platforms. On the renderer side, remove most Android-specific code, so it behaves like Linux. As a result, the printed document data is transferred via PrintHostMsg_DidPrintDocument, instead of the Android-specific IPC messages. To handle this in WebView, add AwPrintManager::OnDidPrintDocument() to receive the printed data, and write it out to its file descriptor. To handle this in Chromium, reuse the existing printing architecture and add the missing Android-specific bits. Hold the file descriptor used for printing in PrintingContextAndroid, and implement PrintingContextAndroid::PrintDocument() to write to the file descriptor. BUG=922657 Change-Id: I1c0123cade448de5a3ace00d671ecae61a04ffd3 Reviewed-on: https://chromium-review.googlesource.com/c/1420297 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Tom Sepez <tsepez@chromium.org> Reviewed-by: Bo <boliu@chromium.org> Reviewed-by: Shimi Zhang <ctzsm@chromium.org> Cr-Commit-Position: refs/heads/master@{#625280}
Changed files
- android_webview/BUILD.gn
- android_webview/browser/aw_content_browser_client.cc
- android_webview/browser/aw_print_manager.cc
- android_webview/browser/aw_print_manager.h
- android_webview/browser/aw_printing_message_filter.cc
- android_webview/browser/aw_printing_message_filter.h
- chrome/browser/printing/print_view_manager_base.cc
- chrome/browser/printing/print_view_manager_basic.cc
- chrome/browser/printing/print_view_manager_basic.h
- chrome/browser/printing/printing_message_filter.cc
- chrome/browser/printing/printing_message_filter.h
- components/printing/common/print_messages.h
- components/printing/renderer/print_render_frame_helper.cc
- components/printing/renderer/print_render_frame_helper_linux.cc
- printing/BUILD.gn
- printing/printed_document.cc
- printing/printed_document_android.cc
- printing/printing_context_android.cc
- printing/printing_context_android.h
-
Request status report after a suspend mode has been completed.
Changed by Leonardo Padilha - ldaguilarohnoyoudont@chromium.org Changed at Wednesday, 23-Jan-19 18:43:48 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision b9650e0492c23bab18d5eb218cb402add39bfa53 Comments
Request status report after a suspend mode has been completed. Add return from suspend mode as trigger to status report. Bug: 919132 Change-Id: I9a3533de21bc8d83761889d26e81066c14e03428 Reviewed-on: https://chromium-review.googlesource.com/c/1427839 Reviewed-by: Aga Wronska <agawronska@chromium.org> Commit-Queue: Leonardo Padilha <ldaguilar@chromium.org> Cr-Commit-Position: refs/heads/master@{#625279}
Changed files
- chrome/browser/chromeos/child_accounts/event_based_status_reporting_service.cc
- chrome/browser/chromeos/child_accounts/event_based_status_reporting_service.h
- chrome/browser/chromeos/child_accounts/event_based_status_reporting_service_unittest.cc
-
Revert "Add a DCHECK in omnibox_view_ios."
Changed by Stepan Khapugin - stkhapuginohnoyoudont@chromium.org Changed at Wednesday, 23-Jan-19 18:43:33 UTC Repository https://chromium.googlesource.com/chromium/src Branch Revision 095f090c96f6131c77d4533353ea6b37e0fd36f7 Comments
Revert "Add a DCHECK in omnibox_view_ios." This reverts commit 2cd80cbbd1ec496fbc335e2d17cfc816ba56b55b. Reason for revert: DCHECK fires on real users' devices. Unclear why. Original change's description: > Add a DCHECK in omnibox_view_ios. > > I suspect that processing_user_event is always true. > If this is the case, the textfield delegate can be simplified. In > order to verify this, add a DCHECK and see if it fires on the bots. > > Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet > Change-Id: Ic45526950bf1f4b05a4c11a9cde551d0eef6dc0e > Reviewed-on: https://chromium-review.googlesource.com/1120816 > Reviewed-by: Rohit Rao <rohitrao@chromium.org> > Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#573710} TBR=rohitrao@chromium.org,justincohen@chromium.org,stkhapugin@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I2729f16524f3523098c09543705f007fb5e14515 Reviewed-on: https://chromium-review.googlesource.com/c/1430088 Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by: Stepan Khapugin <stkhapugin@chromium.org> Cr-Commit-Position: refs/heads/master@{#625278}
Changed files
- ios/chrome/browser/ui/omnibox/omnibox_view_ios.mm