- 13 Nov, 2019 40 commits
-
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/359bc80a7ee4..b5407bfb585e git log 359bc80a7ee4..b5407bfb585e --date=short --no-merges --format='%ad %ae %s' 2019-11-13 swiftshader.regress@gmail.com Regres: Update test lists @ 88632cac Created with: gclient setdep -r src/third_party/swiftshader@b5407bfb585e If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_chromium_msan_rel_ng;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=swiftshader-team+autoroll@google.com Bug: None Change-Id: I0def1413313352681965f9b2ad0247aa7ac2c826 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913633Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#714979}
-
Matt Menke authored
Make HostResolverMojo take a NetworkIsolationKey. It now both passes the NetworkIsolationKey to the HostResolverMojo::Impl it wraps, and uses the NetworkIsolationKey as part of the key into its local HostCache. Bug: 1021661 Change-Id: Ib9fc433635933c0e0a587c762d4ce6e581bf7317 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904487 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#714978}
-
Elly Fong-Jones authored
This change: 1) Fixes two errors in flag-metadata.json, both instances of: "owners": [ "foo,bar" ] where "owners": [ "foo", "bar" ] was meant. 2) Adds a new unit test, AboutFlagsTest.OwnersLookValid, that detects this mistake and another couple of common syntax mistakes in owner metadata. Bug: None Change-Id: I944efd9c95fcb6a2ccdce2b174b542ddf3e451d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912803 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#714977}
-
Ioana Pandele authored
The overhead of creating a proper ChromePasswordManagerClient instance is too big for unit tests and may lead to unexpected side-effects. Bug: 1007218 Change-Id: I98f02d64a513befa1cf22346ca94847fa7b1f75c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912717Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Ioana Pandele <ioanap@chromium.org> Cr-Commit-Position: refs/heads/master@{#714976}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/a4e6d1590f29..57d0093ac901 Created with: gclient setdep -r src-internal@57d0093ac901 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: Ib7e463cb7cb298c3d8037a377590bcc1060a9bd8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913654Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#714975}
-
Azeem Arshad authored
This CL adds aria labels for the 4 custom name server input fields. The fields will be announced as "Custom name server 1", "Custom name server 2" etc. Bug: 1005944 Change-Id: I79eb275d5ad3ca60b1f29ea04c455fd8258d7939 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913880Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Azeem Arshad <azeemarshad@chromium.org> Cr-Commit-Position: refs/heads/master@{#714974}
-
Jesse Schettler authored
This reverts commit 3af7932b. Reason for revert: ARC++ printing can pass CTS Original change's description: > Revert "arc: Enable print-spooler-experiment feature flag by default" > > This reverts commit c79e6751. > > Reason for revert: Fixing CTS for ToT > > Original change's description: > > arc: Enable print-spooler-experiment feature flag by default > > > > Also, enable the flag for all channels except stable. > > > > Bug: b:130798103 > > Test: android-sh -c getprop | grep ro.boot.arc_print_spooler > > Change-Id: I4ca45c248737bbc0e5c90ed679bfe2b77ec9ed12 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1784006 > > Reviewed-by: Elijah Taylor <elijahtaylor@chromium.org> > > Commit-Queue: Jesse Schettler <jschettler@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#693860} > > TBR=yusukes@chromium.org,elijahtaylor@chromium.org,jschettler@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: b:130798103 > Change-Id: I400c7050bbf4c82ffba230b1900d14daa309c628 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1831273 > Reviewed-by: Yusuke Sato <yusukes@chromium.org> > Commit-Queue: Jesse Schettler <jschettler@chromium.org> > Cr-Commit-Position: refs/heads/master@{#701184} TBR=yusukes@chromium.org,elijahtaylor@chromium.org,jschettler@chromium.org Bug: b:130798103 Change-Id: I378ae1dee61231462f8588bd1303400b47158bd1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1914479Reviewed-by:
Jesse Schettler <jschettler@chromium.org> Commit-Queue: Jesse Schettler <jschettler@chromium.org> Cr-Commit-Position: refs/heads/master@{#714973}
-
Stephen Roe authored
This is a reland of 8db01995 after fixing https://b.corp.google.com/issues/143701391 It adds the parentheses changes from https://chromium-review.googlesource.com/c/chromium/src/+/1893190 Original change's description: > Add third_party/qemu-linux-arm64 to DEPS runhooks. > > qemu-linux-arm64 is from the /fuchsia/qemu/linux-arm64 CIPD package. > > Bug: 1015177 > Change-Id: Ie193d18e4b52059ad853cee07075979b1faea038 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1865751 > Reviewed-by: Scott Violet <sky@chromium.org> > Reviewed-by: Kevin Marshall <kmarshall@chromium.org> > Commit-Queue: Stephen Roe <steveroe@google.com> > Cr-Commit-Position: refs/heads/master@{#708478} Bug: 1015177 Change-Id: I27c848336a3466ef2aa2801d29b7b850fd0411d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913570 Auto-Submit: Stephen Roe <steveroe@google.com> Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#714972}
-
Koji Ishii authored
Following bot results are included. 265 266 269 270 271 272 273 275 30 lines were removed and 32 lines were deflaked by consecutive results since 253. TBR=yosin@chromium.org NOTRY=true Bug: 982194 Change-Id: I4c914f77da1efc37c6305d97136d48893b8546d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912399Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#714971}
-
Anastasia Helfinstein authored
All calls to connectChildren were immediately followed by a call to setChildren, so I combined them, along with several other readability changes. This change is a pure refactor. Bug: None Change-Id: If340c812127d95412939977a12f247203da07d86 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902516 Commit-Queue: Anastasia Helfinstein <anastasi@google.com> Reviewed-by:
Chris Hall <chrishall@chromium.org> Cr-Commit-Position: refs/heads/master@{#714970}
-
Ken Rockot authored
This changes the GPU process to expose browser-facing interfaces through the Content ChildProcess interface. ChildThreadImpl now allows subclasses to provide a BinderMap to handle incoming interface binding requests from the browser. This is a fairly simple drop-in replacement for what used to be done through the Service Manager with a ConnectionFilter. There should be no behavioral changes here. In particular for GPU interfaces, this CL is careful to preserve the behavior which avoids binding incoming interface receivers until |OnGpuServiceConnection()| is invoked. Bug: 977637 Change-Id: Id00a1f0b1178f9c3fed2816eea9808f162829798 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1905050Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Zhenyao Mo <zmo@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Commit-Queue: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#714969}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/cfdbaaa37664..9e7107eb96af git log cfdbaaa37664..9e7107eb96af --date=short --no-merges --format='%ad %ae %s' 2019-11-13 rharrison@chromium.org Migrate Dawn fuzzers to using new SPVC split API Created with: gclient setdep -r src/third_party/dawn@9e7107eb96af If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC cwallez@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel TBR=cwallez@google.com Bug: None Change-Id: I096cc1e4cc76941e6f391813770fd4754c8f64a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913658Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#714968}
-
Daniel Vogelheim authored
Bug: 1020589 Change-Id: Ieefc31f5a16f9d374577cd498a493d86ba95d4f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913256Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org> Cr-Commit-Position: refs/heads/master@{#714967}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 222d1e64. With Chromium commits locally applied on WPT: 1d5b803f "Add WebVTT support for inline styling - Web Platform Tests" 88d98fda "Add more scroll to text WPTs" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: foolip@chromium.org, lpz@chromium.org, robertma@chromium.org: external/wpt/tools rego@igalia.com: external/wpt/css/css-contain external/wpt/css/css-grid NOAUTOREVERT=true TBR=foolip No-Export: true Change-Id: Id5e8f88d03784476d8d6fb40b1ed8100c0b9ea7b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1914190Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#714966}
-
Saman Sami authored
Video frames that are capable of being put in an overlay should be marked as such. This bug only affects the ChromeosVideoDecoder experiment. Bug: 984482,756454 Change-Id: I1b6b31adb8525e8081ae8cd404b00bcd62b3f305 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913210 Commit-Queue: Saman Sami <samans@chromium.org> Reviewed-by:
Chih-Yu Huang <akahuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#714965}
-
Jinsuk Kim authored
Tab's 2 public methods updateWindowAndroid and notifyActivityAttachmentChanged are related, one calling the other. This CL consolidates them into one to make the API surface smaller. Bug: 995903 Change-Id: I0ad30525451c633a18d49c4b419be91bbab83754 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906847 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#714964}
-
Yue Zhang authored
Bug: 1024089 Change-Id: Ie8e0a95b6dd2151cbf88f9e42c0791cc6f264c85 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913713Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: Yue Zhang <yuezhanggg@chromium.org> Cr-Commit-Position: refs/heads/master@{#714963}
-
Manuel Rego Casasnovas authored
This patch separates PaintLayer::NeedsRepaint() into two flags: * PaintLayer::SelfNeedsRepaint() * PaintLayer::DescendantNeedsRepaint() This also introduces a method SelfOrDescendantNeedsRepaint() that is used in the places where we need to check for any of the flags (somehow the equivalent to the previous NeedsRepaint()). This is the initial step to allow optimizations in future patches. Ideally we should be able to avoid repainting a PaintLayer if it only has DescendantNeedsRepaint(). BUG=917911 Change-Id: Ie9d509328515492905d8da3b11d2cbf95da5f7b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911792Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Manuel Rego <rego@igalia.com> Cr-Commit-Position: refs/heads/master@{#714962}
-
Livvie Lin authored
Call to the method was fixed in https://chrome-internal.googlesource.com/chrome/ios_internal.git/+/1dc0ccf4574deb1d4a5ea95b2397ba44a5d48e14 Bug: 1020670 Change-Id: I25a2b9768cfcab42e6dd86b25b72cafa1fc76296 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1907077Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Livvie Lin <livvielin@chromium.org> Cr-Commit-Position: refs/heads/master@{#714961}
-
Xiaoqian Dai authored
If hotseat was hidden before window drag starts, the drag will first bring up the hotseat and then after hotseat is fully brought up, the window can be dragged up. In this case we should calculate the initial window drag scale at the initial event location, otherwise we'll see a jump of window scaling down effect. For the extended hotseat case, since the drag can start below the mini shelf, we should continue use window bottom location to calculate the initial window drag scale, otherwise the dragged window might be above the finger, not under the finger. Bug: 997885 Change-Id: I81c100379899d032442522460b5085f97aadb4a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913482Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#714960}
-
Vincent Boisselle authored
To stay consistent with the UmaAndUkmDemographics Finch config, the tested group UMA_enabled should disable UkmReportNoisedUserBirthYearAndGender feature. Bug: 1023825 Change-Id: I8c8bbd341b576ea67cd6bd88a28a87c34b29977d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1914193Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#714959}
-
Henrique Nakashima authored
Chromium's only exists for historic reasons, currently serving documentation purposes only. AndroidX's version is supported by Android lint, though the check at build time is not currently working (crbug.com/1022939). This check is still useful for Android Studio. Bug: 889467 Change-Id: I6fefc44c8cda5d744bd722ca20a7583f8e5e19cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901635 Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#714958}
-
Andrew Grieve authored
* Ignore library when License File: NOT_SHIPPED * Expand regex to match a few more variation of existing licenses. Bug: 693079 Change-Id: I87cba0fc358f155a3c8fd3f590fc5ec5210ce896 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911603Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Auto-Submit: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#714957}
-
Chris Lu authored
This change implements the model logic for InfobarTranslateTableViewController. This change also adds a infobarButtonText method that returns the desired action button text depending on the current TranslateStep. Screenshot: https://drive.google.com/open?id=1UFfeJc56Hh1e0boGRfxPAOLNPFRle0v4 Bug: 1014959 Change-Id: I04a1021fbf6024527050691bc7704cda0668e478 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899201 Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#714956}
-
Tibor Goldschwendt authored
Change-Id: Id5315c46499b56e49e6acebe1aff67d84ab6bc7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903931 Auto-Submit: Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Christopher Grant <cjgrant@chromium.org> Cr-Commit-Position: refs/heads/master@{#714955}
-
Aaron Krajeski authored
By the old method both the paint canvas and the recorder canvas would hold the transform, thus it would be applied twice. With this change OffscreenCanvas::TransferToImageBitmap restores the matrix onto the recorder directly if it is deferred. In general, canvas hosts should not reach directly into resource provider's paint canvases. Bug: 1021321 Change-Id: I1f499218c6e58168d13b6fc88ad88d69b4435191 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1904266Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Reviewed-by:
Yi Xu <yiyix@chromium.org> Commit-Queue: Aaron Krajeski <aaronhk@chromium.org> Cr-Commit-Position: refs/heads/master@{#714954}
-
Garrett Beaty authored
Bug: 1011908 Change-Id: I431823141a8d2affe9d9ac24b5e4964626166c1b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1908924 Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org> Cr-Commit-Position: refs/heads/master@{#714953}
-
Lukasz Anforowicz authored
There are multiple calls to GetOriginForURLLoaderFactory made from RenderFrameHostImpl::CommitNavigation. Instead of making these calls, we can make a single call early during CommitNavigation and store the result in a variable that gets reused later on. This refactoring has been suggested when code reviewing another CL here: https://chromium-review.googlesource.com/c/chromium/src/+/1875273/15/content/browser/frame_host/render_frame_host_impl.cc#5420 It turns out that moving the GetOriginForURLLoaderFactory call earlier means that it also gets called for navigations to about:srcdoc. In this case the URLLoaderFactory should be inherited from the parent, so the origin calculated by GetOriginForURLLoaderFactory is not technically needed. OTOH, GetOriginForURLLoaderFactory should in the future calculate the origin to commit, and therefore we should make sure it correctly handles all the cases. In particular, without special-casing about:srcdoc, NavigationBrowserTest.BlockedSrcDocRendererInitiated* tests would hit a CHECK(...CanAccessDataForOrigin...) in GetOriginForURLLoaderFactory. This CL fixes this. Bug: 998247, 920634, 936310 Change-Id: I1ca851c8e574d24f05b4943f6bee186ebd50aa61 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910865Reviewed-by:
Nasko Oskov <nasko@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#714952}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/c5ec54e51b91..06a394e036ab git log c5ec54e51b91..06a394e036ab --date=short --no-merges --format='%ad %ae %s' 2019-11-13 mbonadei@webrtc.org Revert "Reland "Define WEBRTC_ENABLE_SYMBOL_EXPORT if is_component_build=true."" 2019-11-13 mbonadei@webrtc.org Reland "Define WEBRTC_ENABLE_SYMBOL_EXPORT if is_component_build=true." 2019-11-13 mbonadei@webrtc.org Use explicit template decl/def for RTCNonStandardStatsMember. Created with: gclient setdep -r src/third_party/webrtc@06a394e036ab If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=webrtc-chromium-sheriffs-robots@google.com Bug: None Change-Id: I3c7b20e7733c19a86155b6d455484d508771f6c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913653Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#714951}
-
Elly Fong-Jones authored
TBR=wjmaclean@chromium.org Bug: 1024028 Change-Id: I22a4a2bf37a28243f926b53c03e6ff4741494e9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913864Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#714950}
-
Peter Kasting authored
Screenshots: https://bugs.chromium.org/p/chromium/issues/detail?id=1024091#c2 Bug: 1024091 Change-Id: I2c318452bafbefc16da3e9c2687911162b26c3f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913509 Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#714949}
-
Weiliang Chen authored
In Android pre-SurfaceControl, we might encounter cases where the video resource is not ready when we first re-open chromium. Match the behavior of GLRenderer, and early return without crashing in SkiaRenderer. Bug: 1016528 Change-Id: I28aae1c46e724c4c97cea4313c395a66f2af13b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913168Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Commit-Queue: weiliangc <weiliangc@chromium.org> Cr-Commit-Position: refs/heads/master@{#714948}
-
Mirko Bonadei authored
While testing the WebRTC component build [1], lld-link started to complain about undefined symbols from the MS SDK. This is caused by the fact that different parts of Chromium were relying on WebRTC to pass some libraries to the linker but since WebRTC is going to support the component build soon, these libraries will not reach the linker since they will be linked only in the WebRTC shared library. This CL addresses the errors detected here [2]. [1] - https://chromium-review.googlesource.com/c/chromium/src/+/1874722 [2] - https://ci.chromium.org/p/chromium/builders/try/win_chromium_compile_dbg_ng/436771 Bug: webrtc:9419 Change-Id: I7af507a13285f976b5545025e2181ae4e47eded2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913251 Commit-Queue: Mirko Bonadei <mbonadei@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#714947}
-
Garrett Beaty authored
These changes simplify migrating to starlark by removing spurious diffs from the semantic diff. The "Memory Infra tester" builder is actually named "Memory Infra Tester". The cronet notifier had an entry for the removed android-cronet-marshmallow-arm-rel builder. The release-tarballs notifier only notified on builders from a non-existent bucket. Bug: 1011908 Change-Id: I753df31a5496e0a7d73e738776f84ea088668e30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910346Reviewed-by:
Aaron Gable <agable@chromium.org> Commit-Queue: Garrett Beaty <gbeaty@chromium.org> Cr-Commit-Position: refs/heads/master@{#714946}
-
liberato@chromium.org authored
This will mirror the finch config, once presubmit for the gcl change passes. Right now, it's mad about the field trial testing config. Change-Id: I58ecc99ae5901e0cd6aee407d73aefcfebe36f04 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913485Reviewed-by:
Ilya Sherman <isherman@chromium.org> Commit-Queue: Frank Liberato <liberato@chromium.org> Cr-Commit-Position: refs/heads/master@{#714945}
-
Anatoliy Potapchuk authored
Also, bring back IsLoggedInAsWebKioskApp() for permissions handling. Bug: 1015383,1006230 Change-Id: I66dfda221fd828443f580746c68faf47757d4137 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911797 Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#714944}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/8b73d27ddd61..99e73fa8c1c5 git log 8b73d27ddd61..99e73fa8c1c5 --date=short --no-merges --format='%ad %ae %s' 2019-11-13 robertphillips@google.com Remove GL 4-bit stencil option Created with: gclient setdep -r src/third_party/skia@99e73fa8c1c5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC halcanary@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=halcanary@google.com Bug: None Change-Id: I95f3084c4912efd832b6818f51b2ddf91dbe0940 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913652Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#714943}
-
Francois Doray authored
Change-Id: I543ef610bdd8b6ff75033fc6014a37b871b07086 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1909386 Auto-Submit: François Doray <fdoray@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#714942}
-
danakj authored
This removal was done in 238f6ccc which claims it was to remove the not-run tests from the shard bot's output. The list of TestExpectations no longer appear to affect the output, as only the list of tests to be run on the shard appear in the output.json file. After building the model of 80000 TestExpectations, we then go through them all removing tests that are not going to run on the bot. To do so we do O(TestExpectations * TestsNotRunningOnBot) which is like O(100k * 100k) so it is very slow. Locally, not doing this saves 31 of the 46 seconds spent during startup between: 16:14:54.439 117357 Collecting tests ... ... 16:15:40.689 117357 Found 3626 tests (total 86597); running 3388, skipping 238. On windows bots we see this time go from 65 seconds to 28 seconds, saving 37 seconds, or another 60% of the test collection startup time. R=robertma@chromium.org Bug: 982208 Change-Id: I0570cc5610f296b3f5850973f02fb85946ebaf38 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913165 Commit-Queue: danakj <danakj@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#714941}
-
Yifan Luo authored
See https://wpt.fyi/results/fetch/metadata?label=experimental&label=master&aligned. 1. `xslt.tentative.https.sub.html`: delete tests for same-site and cross-site, because these requests are not allowed and has been blocked by browser behavior. 2. `redirect/multiple-redirect-https=downgrade-upgrade.tentative.sub.html` and `redirect/redirect-http-upgrade.tentative.sub.html`: removed from `NeverFixTests` and change the expectation for mode from `nasted-navigate` to `navigate`. 3. `redirect/redirect-https-downgrade.tentative.sub.html`: removed from `NeverFixTests` and change the expectation for Https downgrade script. Bug: 1024198 Change-Id: Ia0cf0ccc2bef38c4525bebab0a050e7b1de9a5e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912712Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Yifan Luo <lyf@google.com> Cr-Commit-Position: refs/heads/master@{#714940}
-