buildbot warning in chromium.fyi on Browser Side Navigation Linux, revision d5f1046c9a939b00f2451af27690faf455c07ea1

1 view

Skip to first unread message

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

unread,

Feb 26, 2018, 6:59:31 AM2/26/18

to plzna...@chromium.org, gateke...@chromium-gatekeeper-sentry.appspotmail.com

https://build.chromium.org/p/chromium.fyi/waterfall

Automatically closing tree for "bot_update" on "Browser Side Navigation Linux"

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

Revision: 040325c16f8d7ee994c46657319e06cc21a6fc62, 9e25952fd786af7f983525980cdae2c5b90b0dde, 95a82a9b77511c2b9108741d8ff33721e09c5a1b, 17a5affd36ab79fdf00e2e4d283615c7ad3454d3, d5f1046c9a939b00f2451af27690faf455c07ea1, d49a775061c3d506d9cb869d8c087e0779a4d912, ad6c523b9de20fbed7bbcf980736c212526a2ae0
Blame list: angle-chrom...@skia-buildbots.google.com.iam.gserviceaccount.com,beaufort...@gmail.com,dull...@chromium.org,hajim...@chromium.org,ptw...@vewd.com,vas...@chromium.org

Build 39596

running steps via annotated script
preamble
stdio

update_scripts
stdio
gclient_json
build: f17e559a60a58b9c18dee167777d97caf2cfc14e..a741e6431dfe8e2abc0b02983af75ccfd931950c
depot_tools: f438ff743a4b157b9be8cecefa7a170f9695d185

LUCI Migration
stdio

LogDog Bootstrap
stdio

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

makedirs checkout path
stdout

bot_update
stdout

Changed by: angle-chrom...@skia-buildbots.google.com.iam.gserviceaccount.com
Changed at: Mon 26 Feb 2018 02:11:23
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 040325c16f8d7ee994c46657319e06cc21a6fc62
Changed files:

DEPS

Comments:

Roll src/third_party/angle/ 81aa15f10..3582c0e2c (1 commit)

https://chromium.googlesource.com/angle/angle.git/+log/81aa15f10682..3582c0e2c3c1

$ git log 81aa15f10..3582c0e2c --date=short --no-merges --format='%ad %ae %s'
2018-02-22 oetuaho Add a test for redefining an unavailable built-in

Created with:
  roll-dep src/third_party/angle


The AutoRoll server is located here: https://angle-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/
...skip...
m.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=geof...@chromium.org

Change-Id: Ic4ac0d13123092c525e0a56f34e1f890512f8a19
Reviewed-on: https://chromium-review.googlesource.com/936436
Reviewed-by: angle-chromium-autoroll <angle-chrom...@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: angle-chromium-autoroll <angle-chrom...@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#539100}

Properties:

git_revision: 040325c16f8d7ee994c46657319e06cc21a6fc62

Changed by: dull...@chromium.org
Changed at: Mon 26 Feb 2018 02:11:55
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 9e25952fd786af7f983525980cdae2c5b90b0dde
Changed files:

chrome/browser/browsing_data/navigation_entry_remover_browsertest.cc
components/sessions/core/tab_restore_service_helper.cc

Comments:

TabRestoreService: Fix window with one tab

When only a single Tab is left after deleting tabs from a Window in
TabRestoreService, the Window should be converted to a Tab.

Fix another case where tabstrip_index was used incorrectly as it might
not be valid.

Bug: 407074
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I86f49969ea83a0771634be554541aab92334a967
Reviewed-on: https://chromium-review.googlesource.com/931706
Reviewed-by: Scott Violet <s...@chromium.org>
Commit-Queue: Christian Dullweber <dull...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539101}

Properties:

git_revision: 9e25952fd786af7f983525980cdae2c5b90b0dde

Changed by: vas...@chromium.org
Changed at: Mon 26 Feb 2018 02:33:13
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 95a82a9b77511c2b9108741d8ff33721e09c5a1b
Changed files:

ios/chrome/browser/passwords/password_controller_unittest.mm
ios/chrome/browser/ui/settings/BUILD.gn
ios/chrome/browser/ui/settings/passwords_settings_egtest.mm

Comments:

Increase test coverage of "Show All" manual fallback for passwords.

Bug: 782287
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I534f2b892efe8d2af08bbea6ee724c38509a55c9
Reviewed-on: https://chromium-review.googlesource.com/926581
Reviewed-by: Sylvain Defresne <sdef...@chromium.org>
Commit-Queue: Vasilii Sukhanov <vas...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539102}

Properties:

git_revision: 95a82a9b77511c2b9108741d8ff33721e09c5a1b

Changed by: ptw...@vewd.com
Changed at: Mon 26 Feb 2018 02:34:43
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: 17a5affd36ab79fdf00e2e4d283615c7ad3454d3
Changed files:

third_party/tcmalloc/chromium/src/base/linux_syscall_support.h

Comments:

Avoid using inline cast in mips assembly code.

When building the code with clang the compilation fails due to the
following error:
  "invalid use of a cast in a inline asm context requiring an l-value:
   remove the cast or build with -fheinous-gnu-extensions"

Doing the cast outside of __asm__ block fixes the problem.

R=j...@chromium.org

Bug: 794147
Change-Id: I49bcdbd3d4177cbdafa37ffc2d12983db1fe7621
Reviewed-on: https://chromium-review.googlesource.com/822411
Reviewed-by: Will Harris <w...@chromium.org>
Commit-Queue: Hugo Holgersson <hu...@vewd.com>
Cr-Commit-Position: refs/heads/master@{#539103}

Properties:

git_revision: 17a5affd36ab79fdf00e2e4d283615c7ad3454d3

Changed by: ptw...@vewd.com
Changed at: Mon 26 Feb 2018 02:38:54
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: d5f1046c9a939b00f2451af27690faf455c07ea1
Changed files:

base/test/launcher/test_launcher_nacl_nonsfi.cc
base/test/launcher/unit_test_launcher.cc
base/test/launcher/unit_test_launcher.h

Comments:

Remove tmp files used to pass command line to tests after they finish

Right now the file used to pass command line parameters to test-
processes is not removed after the test finished. In cases where tests
are run in repeat mode those flag files just keep accumulating until
the system runs out of space in it's $TMPDIR.

To solve this, the patch makes sure the path to the file containing the
flags is stored in UnitTestProcessLifetimeObserver and removed once
the OnCompleted method is called.

Bug: 814687
Change-Id: Ifb0ba6dba860d9297fbc2c939e0cc10f620907fa
Reviewed-on: https://chromium-review.googlesource.com/931341
Commit-Queue: Nico Weber <tha...@chromium.org>
Reviewed-by: Nico Weber <tha...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539104}

Properties:

git_revision: d5f1046c9a939b00f2451af27690faf455c07ea1

Changed by: hajim...@chromium.org
Changed at: Mon 26 Feb 2018 02:41:13
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: d49a775061c3d506d9cb869d8c087e0779a4d912
Changed files:

components/autofill/content/renderer/autofill_agent.cc

Comments:

Reland: Fix a leak of a document that is held by AutofillAgent

This CL is just relanding https://chromium-review.googlesource.com/c/chromium/src/+/597482,
after confirming that there are no statistical significant regressions
in the real world by Finch experiments (crbug.com/755489).

Note that this might still cause Telemetry regressions reported at
crbug.com/753071 before, but this is expected.

A document is leaked because AutofillAgent hold a lastly used element
even after navigation happens, and
...skip...
l a new (input) element is focused. This CL fixes this issue by
resetting the element in AutofillAgent when navigation happens.

Bug: 734427, 755489
Change-Id: Ie3ee7f202abf56e22102e3d7f68cf2778f85ca87
Reviewed-on: https://chromium-review.googlesource.com/908371
Commit-Queue: Hajime Hoshi <hajim...@chromium.org>
Reviewed-by: Kouhei Ueno <kou...@chromium.org>
Reviewed-by: Kentaro Hara <har...@chromium.org>
Reviewed-by: Evan Stade <est...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539105}

Properties:

git_revision: d49a775061c3d506d9cb869d8c087e0779a4d912

Changed by: beaufort...@gmail.com
Changed at: Mon 26 Feb 2018 03:00:44
Repository: https://chromium.googlesource.com/chromium/src
Branch: master
Revision: ad6c523b9de20fbed7bbcf980736c212526a2ae0
Changed files:

chrome/android/java/src/org/chromium/chrome/browser/media/PictureInPictureController.java
content/browser/DEPS
content/browser/media/media_web_contents_observer.cc
content/browser/media/media_web_contents_observer.h
content/browser/web_contents/web_contents_android.cc
content/browser/web_contents/web_contents_android.h
content/browser/web_contents/web_contents_impl.cc
content/browser/web_contents/web_contents_impl.h
content/common/DEPS
content/common/media/media_player_delegate_messages.h
content/public/android/java/src/org/chromium/content/browser/webcontents/WebContentsImpl.java
content/public/android/java/src/org/chromium/content_public/browser/WebContents.java
content/public/test/android/javatests/src/org/chromium/content/browser/test/mock/MockWebContents.java
content/renderer/media/renderer_webmediaplayer_delegate.cc
content/renderer/media/renderer_webmediaplayer_delegate.h
content/renderer/media/stream/webmediaplayer_ms_unittest.cc
media/blink/webmediaplayer_delegate.h
media/blink/webmediaplayer_impl.cc
media/blink/webmediaplayer_impl.h
media/blink/webmediaplayer_impl_unittest.cc
third_party/WebKit/Source/core/html/media/HTMLMediaElementEventListenersTest.cpp
third_party/WebKit/Source/core/html/media/MediaCustomControlsFullscreenDetector.cpp
third_party/WebKit/public/platform/WebFullscreenVideoStatus.h
third_party/WebKit/public/platform/WebMediaPlayer.h

Comments:

Prevent auto-PiP if disablePictureInPicture video attribute is set.

This CLs makes sure fullscreen videos don't enter automatically
Picture-in-Picture if the disablePictureInPicture attribute is set on
the video element.

Test page: https://beaufortfrancois.github.io/sandbox/media/disable-pip.html

Bug: 806249
Change-Id: I3621ef58c8f1778d54865eca6cc30a736994d493
Reviewed-on: https://chromium-review.googlesource.com/918421
Commit-Queue: François Beaufort <beaufort...@gmail.com>
Reviewed-by: Mounir Lamouri <mlam...@chromium.org>
Reviewed-by: Dale Curtis <dalec...@chromium.org>
Reviewed-by: Kinuko Yasuda <kin...@chromium.org>
Cr-Commit-Position: refs/heads/master@{#539106}

Properties:

git_revision: ad6c523b9de20fbed7bbcf980736c212526a2ae0

Reply all

Reply to author

Forward

0 new messages