- 14 Nov, 2019 40 commits
-
-
Chromium WPT Sync authored
Using wpt-import in Chromium e7498a38. With Chromium commits locally applied on WPT: 1d5b803f "Add WebVTT support for inline styling - Web Platform Tests" 88d98fda "Add more scroll to text WPTs" 788a3004 "Fix failing Fetch Metadata WPT" b16db925 "[Trusted Types] Add test for duplicate names in non-enforcing mode." 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 NOAUTOREVERT=true TBR=foolip No-Export: true Change-Id: I8d04fb94f036de323034ca3859ac0f4712704505 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916640Reviewed-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@{#715243}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/dda10ea5191d..c49e3c72fe42 Created with: gclient setdep -r src-internal@c49e3c72fe42 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: Ic681dd18ae25c3dbd24f03049aceee61886baf34 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916624Reviewed-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@{#715242}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/00fc2231f697..f6fc149b0b16 git log 00fc2231f697..f6fc149b0b16 --date=short --no-merges --format='%ad %ae %s' 2019-11-14 khokhlov@google.com Reland "Remove had_failures from benchmark metadata" Created with: gclient setdep -r src/third_party/catapult@f6fc149b0b16 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC sadrul@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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=sadrul@google.com Bug: chromium:990304 Change-Id: Ib92d5b9838eafa38065661b6be9ab6138d96f7f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916621Reviewed-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@{#715241}
-
Mathias Carlen authored
Before this patch, empty strings would be filtered before sent to the UI. The result of that was some tell_actions with single space ' ' messages to clear the status bar. This patch removes that constraint and empty strings will just remove the status message shown. As I drive by there's now a unittest for the tell action, even if that's going the wrong way in the stack ... R=arbesser@google.com Bug: b/143757287 Change-Id: I843a0cf36e246c4480de4ffa8fe59709ef576bd7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911211 Commit-Queue: Mathias Carlen <mcarlen@chromium.org> Reviewed-by:
Clemens Arbesser <arbesser@google.com> Cr-Commit-Position: refs/heads/master@{#715240}
-
Robbie Gibson authored
When the animation is on, there is a bug where if the user swipes left-to-right to go to the next Voiceover view during the animation, the Voiceover cursor ends up on view from the previous tab grid page, instead of the next tab grid page. This happens because the previous tab grid page's views are still accessible during the animation. To prevent this, the animation can be turned off when Voiceover is enabled. This means that when the user selects a new tab, the views immediately update. Bug: 978673 Change-Id: Id4b64f391614b8719e3f5d5d2e924df7d8983123 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1906348 Commit-Queue: Robbie Gibson <rkgibson@google.com> Reviewed-by:
edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#715239}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org 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:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: I21a695d9ca8a5cf376d14e10165c29dcd50fc1f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916622Reviewed-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@{#715238}
-
Martin Robinson authored
Instead of returning the previous text selection extents for unfocused text inputs in the ATK API, return an empty selection which matches what the visual selection shows. Bug: 1015354 Change-Id: I767f44dc52df3e4de579f6c4e6a027d72630d60a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886811 Commit-Queue: Martin Robinson <mrobinson@igalia.com> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#715237}
-
Shivani Sharma authored
To experiment split cache/network stack using etld+1 this CL adds support to the Network Isolation Key to replace origins with scheme://registrable-domain. It also adds a new feature which controls whether etld+1 will be used or not. Added browser and unit tests. Bug: 1018718 Change-Id: I02adf6bbb679154c99cf4b7a1ae10ba46da00067 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913062Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Shivani Sharma <shivanisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#715236}
-
Mike West authored
This reverts commit f5946a0a. Reason for revert: `LocalNTPJavascriptTest.RealboxTests` has failed every ChromeOS ASAN run (https://ci.chromium.org/p/chromium/builders/ci/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29) since this patch landed, starting in https://ci.chromium.org/p/chromium/builders/ci/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/36112. Original change's description: > Local NTP, Realbox: fix navigation to chrome:// URLs > > Partially borrowed from https://crrev.com/c/1836021 by Archana Simha <archanasihma@chromium.org> > > Bug: 1020025 > Change-Id: I7de22a41d872ec624f25d1633782d27448988c58 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1893824 > Commit-Queue: David Bokan <bokan@chromium.org> > Reviewed-by: David Bokan <bokan@chromium.org> > Reviewed-by: Moe Ahmadi <mahmadi@chromium.org> > Reviewed-by: Nasko Oskov <nasko@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Auto-Submit: Dan Beam <dbeam@chromium.org> > Cr-Commit-Position: refs/heads/master@{#715106} TBR=dcheng@chromium.org,nasko@chromium.org,dbeam@chromium.org,bokan@chromium.org,mahmadi@chromium.org Change-Id: I7552853a29abc63c667e949f9147bd76ad4e9d69 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1020025 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916861Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#715235}
-
Rune Lillesveen authored
We would like to return empty styles for elements which are connected but not part of the flat tree. That is, shadow host children which are not slotted or any nodes in its flat subtree. Bug: 831568 Change-Id: Ifd6ab89196a83571044ca995198e3e31c904304e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916460 Commit-Queue: Rune Lillesveen <futhark@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#715234}
-
nancy authored
Use the unified time format function to show the time limit in the app pause dialog. For example, hour = 1 minute = 1 Show: You can use it for 1 hour and 1 minute tomorrow. hour = 11 minute = 59 Show: You can use it for 11 hours and 59 minutes tomorrow. hour = 10 minute = 0 Show: You can use it for 10 hours tomorrow. hour = 0 minute = 10 Show: You can use it for 10 minutes tomorrow. hour = 0 minute = 0 Show: You can use it for 0 seconds tomorrow. BUG=1011235 Change-Id: I27619b2b7f999c68eabff6a337e9d54eb661db86 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913360 Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Reviewed-by:
Aga Wronska <agawronska@chromium.org> Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#715233}
-
Trent Apted authored
This is a better fit for layering/encapsulation/ownership. The tradeoff is some "slippage" in resource_ids. Bug: 996088 Change-Id: I6b5f03460c01e979da018dec757f88452a42adbb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913284Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Ryo Hashimoto <hashimoto@chromium.org> Commit-Queue: Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#715232}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/0cbfc427a7df..06d9a50ca1e5 git log 0cbfc427a7df..06d9a50ca1e5 --date=short --no-merges --format='%ad %ae %s' 2019-11-14 lalitm@google.com trace_processor: fix sort setting flags on correct columns 2019-11-13 treehugger-gerrit@google.com Merge "processor: Ignore track events with absolute 0 timestamp" 2019-11-13 lalitm@google.com trace_processor: add benchmarks for sorted columns Created with: gclient setdep -r src/third_party/perfetto@06d9a50ca1e5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@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=perfetto-bugs@google.com Bug: chromium:135177627,chromium:135177627 Change-Id: I734f1e09c24e18070eb74246507e28290370a18d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916620Reviewed-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@{#715231}
-
Stefano Sanfilippo authored
Change-Id: I20ce9961645deb44bab857d69a1c641600646d65 Bug: 940927 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1911206 Commit-Queue: Stefano Sanfilippo <ssanfilippo@chromium.org> Reviewed-by:
Daniel Vogelheim <vogelheim@chromium.org> Auto-Submit: Stefano Sanfilippo <ssanfilippo@chromium.org> Cr-Commit-Position: refs/heads/master@{#715230}
-
v.paturi authored
Inline SVG images do not have an Image object associated with them, so they do not go through the regular dark mode image classification that non-inline SVG images would. With the current design, the shapes belonging to the same image might receive different classifications in dark mode. So until there is a way to properly classify an inline SVG image by considering all its individual shapes, do not invert any part of that image. Bug: 963998 Change-Id: Ib946c3c1ff7cdce08d2f5af9cca8542ccb9c91fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1893111Reviewed-by:
Stephen Chenney <schenney@chromium.org> Reviewed-by:
Prashant Nevase <prashant.n@samsung.com> Commit-Queue: Varun Chowdhary Paturi <v.paturi@samsung.com> Cr-Commit-Position: refs/heads/master@{#715229}
-
Patrik Höglund authored
This converts the old RESULT lines to the new perf test reporter. It would be hard to port the result reporter to WebRTC, so let's keep the WebRTC code ignorant of that. Bug: chromium:1874811 Change-Id: I524fd3f24fa2739043e4facfa93cb1300a22a5c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899865Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Patrik Höglund <phoglund@chromium.org> Cr-Commit-Position: refs/heads/master@{#715228}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/c2f0287a28a4..ff11d441ccdf git log c2f0287a28a4..ff11d441ccdf --date=short --no-merges --format='%ad %ae %s' 2019-11-14 kevinptt@chromium.org Drallion: sign factory install shim Created with: gclient setdep -r src/third_party/chromite@ff11d441ccdf If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@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:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I4594be3af1b2d64ca74ce0e5fd9a3f477c0a942f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915825Reviewed-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@{#715227}
-
Alexander Hendrich authored
This CL modifies the protected interface class check in WebUSB to make an exception for HID devices when the request is originating from the first-party Imprivata extensions (login screen and in-session extensions, both in dev/prod versions). BUG: 995294 Change-Id: If436bb1b8d19a74c47cb060cbda8e0e3a061531f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1908544Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Alexander Hendrich <hendrich@chromium.org> Cr-Commit-Position: refs/heads/master@{#715226}
-
Kuo Jen Wei authored
Fix width, height of video resolution option get swapped incorrectly by crrev.com/c/1874996. Bug: None Test: Verify the text of video resolution options. Change-Id: I575f26da394be89486c5af8db890623897c9f363 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916181 Commit-Queue: Kuo Jen Wei <inker@chromium.org> Commit-Queue: Shik Chen <shik@chromium.org> Auto-Submit: Kuo Jen Wei <inker@chromium.org> Reviewed-by:
Shik Chen <shik@chromium.org> Cr-Commit-Position: refs/heads/master@{#715225}
-
Cherie Cheung authored
https://chromium.googlesource.com/chromiumos/platform2/system_api.git/+log/ce25839e00e5..637fb0e5d071 $ git log ce25839e0..637fb0e5d --date=short --no-merges --format='%ad %ae %s' 2019-11-12 cherieccy system_api: Add |rootfs_writable| field to StartArcVmRequest. Created with: roll-dep src/third_party/cros_system_api TEST=Build is successful Bug: b:144154973 Change-Id: I609f60d321b334c9d82cdb7b8845d206f91ed92d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915652Reviewed-by:
Yusuke Sato <yusukes@chromium.org> Reviewed-by:
Chirantan Ekbote <chirantan@chromium.org> Commit-Queue: Cherie Cheung <cherieccy@chromium.org> Cr-Commit-Position: refs/heads/master@{#715224}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/cca19d9bd441..dda10ea5191d Created with: gclient setdep -r src-internal@dda10ea5191d 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: I8114099d83fef3d79dc1e33d1d169c1983464e08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915823Reviewed-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@{#715223}
-
Bret Sepulveda authored
Unlike ToolbarActionsBar, ExtensionsToolbarContainer assumes the ToolbarActionsModel always exists. When the feature flag was enabled, this would crash any test that creates a BrowserView. To fix this, TestWithBrowserView initializes ToolbarActionsModel. Bug: 984654 Change-Id: I69df818c72f898dbf2e2bb7c80d3ac5bcc096c66 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901076Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Commit-Queue: Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#715222}
-
Olivier Robin authored
Starting with iOS 13, sourceApplication is only set when openURL is called from an application with the same team identifier. Oddly, it is also nil when calling from extension. Bug: 1020228 Change-Id: I9876647c30350e2242d20da2ab045805a9864932 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903349Reviewed-by:
Mark Cogan <marq@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
Justin Cohen <justincohen@chromium.org> Commit-Queue: Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#715221}
-
Jan Scheffler authored
This patch adds a test to check that all loaded css files are shown in the coverage list view even when not (yet) covered at all. Fixed: chromium:949045 Change-Id: Idda4df73866590010d59f84cd4193c051fe1cc0c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1903359Reviewed-by:
Sigurd Schneider <sigurds@chromium.org> Commit-Queue: Jan Scheffler <janscheffler@chromium.org> Cr-Commit-Position: refs/heads/master@{#715220}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/1b52f05868c9..ae55a94ff52c git log 1b52f05868c9..ae55a94ff52c --date=short --no-merges --format='%ad %ae %s' 2019-11-14 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/SwiftShader 88632cac30e1..b2407dd746de (3 commits) Created with: gclient setdep -r src/third_party/angle@ae55a94ff52c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC cnorthrop@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_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=cnorthrop@google.com Bug: chromium:None Change-Id: I555fec9c9e5ad84397a05e245ea3fbdcc4c546d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916140Reviewed-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@{#715219}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/ab3cdcaef56e..3da910d55f20 git log ab3cdcaef56e..3da910d55f20 --date=short --no-merges --format='%ad %ae %s' 2019-11-13 dneto@google.com Avoid uninit warning in GCC (#3044) 2019-11-13 bclayton@google.com utils/vscode: Add SPIR-V language server support 2019-11-13 bclayton@google.com utils/vscode: Add go language server packages. Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@3da910d55f20 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,cnorthrop@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_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 TBR=radial-bots+chrome-roll@google.com,cnorthrop@google.com Bug: None Change-Id: Ie5023c33ec6f0df4407a634a29375fd7209bfc5d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916139Reviewed-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@{#715218}
-
Marc Treib authored
- Add treib@ as new main POC while melandory@ is out. - Remove (comment out) previous owner pavely@. - Reduce duplication: Have chrome/browser/invalidation/OWNERS and ios/chrome/browser/invalidation/OWNERS refer to the "main" owners file instead of explicitly listing the same people. Bug: none Change-Id: I80524388818c26884c07928422b74d70e026076f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913404 Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
Tatiana Gornak <melandory@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#715217}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/32f02045f037..c2f0287a28a4 git log 32f02045f037..c2f0287a28a4 --date=short --no-merges --format='%ad %ae %s' 2019-11-14 bhogeswararao@google.com gwifi GroupEntry line length alignment. 2019-11-14 cjmcdonald@chromium.org Delete parallel_emerge_old Created with: gclient setdep -r src/third_party/chromite@c2f0287a28a4 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@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:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I8e4bcb83c04637f2f1aea238a82a474c141c30f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915679Reviewed-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@{#715216}
-
Kuo Jen Wei authored
Bug: 1024402 Test: Verify switching icon is changed during fps switching. Change-Id: I53e04841094b5439634a3b3c497bbcb4209cbde7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915648 Auto-Submit: Kuo Jen Wei <inker@chromium.org> Reviewed-by:
Shik Chen <shik@chromium.org> Commit-Queue: Kuo Jen Wei <inker@chromium.org> Cr-Commit-Position: refs/heads/master@{#715215}
-
Mike West authored
This reverts commit b1ea8aae. Reason for revert: PdfCompositorImplTest.CompleteDocumentLackingPrepare and PdfCompositorImplTest.CompositeDocumentAfterPrepare are consistently failing since https://ci.chromium.org/p/chromium/builders/ci/linux-trusty-rel/4546. Original change's description: > PDF compositor generate full document from individual pages > > During print preview when the PDF compositor is used, individual PDFs > for each page are made for the preview. Prior art has the full > document PDF used for printing being generated by sending a separate > metafile blob for all the pages, which is overly large for long > documents. This alters that to have the full document PDF get > composited using the same individual page metafile objects as they are > sent to the utility process for the composition. > > This effectively halves the amount of IPC traffic for Skia metafile > data and eliminates potential huge metafile messages that can > overwhelm IPC limits. > > This is only applicable for print preview scenarios that pass Skia > metafile into PDF compositor. Non-modifiable content and basic > printing are unaffected in behavior. > > Bug: 872935 > Change-Id: I1101846a00498956b18c208ada656e65e4f175a3 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1616461 > Commit-Queue: Alan Screen <awscreen@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Lei Zhang <thestig@chromium.org> > Cr-Commit-Position: refs/heads/master@{#715165} TBR=dcheng@chromium.org,thestig@chromium.org,awscreen@chromium.org Change-Id: Ie188aef36f38ea092557a46251507960d33a4ef1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 872935, 1024748 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916121Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#715214}
-
shrekshao authored
Currently chromium doesn't expose these two extensions on platform without base instance native support (android without EXT_base_instance and Mac). Workaround for this is already implemented in Angle though. We will advertise them when either we implement the workaround in validating command decoder as well, or we start using angle for all platforms. This patch also removes remaining webgl_multi_draw_instanced feature flag in feature_info.h Bug: 890539 Change-Id: I9122f8d323f42e30d21fc081db2150890c953e74 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888771 Commit-Queue: Shrek Shao <shrekshao@google.com> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Austin Eng <enga@chromium.org> Cr-Commit-Position: refs/heads/master@{#715213}
-
Mike West authored
This reverts commit 845f18e1. Reason for revert: Both `AccessibilityBridgeTest.CorrectDataSent and AccessibilityBridgeTest.DataSentWithBatching are flakily timing out. See https://ci.chromium.org/p/chromium/builders/ci/Fuchsia%20x64 starting around 44416 (e.g. https://ci.chromium.org/p/chromium/builders/ci/Fuchsia%20x64/44418). Original change's description: > Reland "[Fuchsia] Send Chrome a11y updates to Fuchsia SemanticsManager" > > This is a reland of fc5589ac > > A BUILD.gn dep has been added back, which was causing linker failures. > > Control flow in AccessibilityBridgeBrowsertests has been updated so > tests are no longer failing. > > Original change's description: > > [Fuchsia] Send Chrome a11y updates to Fuchsia SemanticsManager > > > > This change adds functionality to the Accessibility Bridge: > > * Listens for change in AccessibilityMode from Fuchsia > > * Gets the semantic tree from Chrome accessibility and sends it to the > > SemanticsManager > > * Sends live updates when nodes are added, updated or removed > > > > To send updates, this change includes the following: > > * Creates a ViewRefPair and registers it with the SemanticsManager > > * Initialize an AccessibilityBridge > > * Listen for changes in the Chrome Accessibility tree, serializes them > > to Fuchsia data structures and sends that data to the Fuchsia > > SemanticsManager > > > > Testing: Added AccessibilityBridge browsertests. > > > > Bug: 973095 > > Change-Id: I89658288b846b6b9ce84a04e49434ec266b84dfc > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1787171 > > Commit-Queue: Sharon Yang <yangsharon@chromium.org> > > Reviewed-by: Kevin Marshall <kmarshall@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#714588} > > Bug: 973095 > Change-Id: Ie6eeb74146682838efaefb0e1051d3e9442f0d86 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1913569 > Reviewed-by: Kevin Marshall <kmarshall@chromium.org> > Commit-Queue: Kevin Marshall <kmarshall@chromium.org> > Cr-Commit-Position: refs/heads/master@{#715126} TBR=dtseng@chromium.org,kmarshall@chromium.org,yangsharon@chromium.org Change-Id: Ib78d563cbbf468aaf03f1e6f00f616dc03b758a7 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 973095, 1024745 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1916120Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#715212}
-
Kuo Jen Wei authored
Bug: b/117810572 Test: Pass presubmit check. Pass tast run <DUT> 'camera.CCAUI*' Change-Id: I9d71ab1c68cb435759cb0416667aa3af78ec2121 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915645 Auto-Submit: Kuo Jen Wei <inker@chromium.org> Reviewed-by:
Shik Chen <shik@chromium.org> Commit-Queue: Kuo Jen Wei <inker@chromium.org> Cr-Commit-Position: refs/heads/master@{#715211}
-
Roman Sorokin authored
Bug: 1022328 Change-Id: I967618ab4e2b45b3b1b7a5056600c34ead38efcd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1912716Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#715210}
-
David Van Cleve authored
net/dns: Histogram ESNI DNS transaction success This patch adds a histogram that records successes and timeouts for ESNI transactions. (Instead of recording the ESNI timeout parameters directly, their effects can be observed by bucketing the timeout ratio by study group and associating different values of the parameters with different groups.) Test: Adds unit tests Bug: 1003494 R=ericorth,csharrison Change-Id: I435f5a2f048df1eec46650aabc763e0c4a43a339 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900353 Commit-Queue: David Van Cleve <davidvc@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Eric Orth <ericorth@chromium.org> Cr-Commit-Position: refs/heads/master@{#715209}
-
Noel Gordon authored
- rename the input files to match the names of the existing ui PNG icon files used in Files app - there are svg files in the ui directory so use a *_ng.svg suffix for the files-ng svg version Bug: 992819 Change-Id: I6d71d884b9f26569e4522debf4fa8bb157ed280a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915646 Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#715208}
-
Alexey Baskakov authored
This is a reland of https://chromium-review.googlesource.com/c/chromium/src/+/1914005 CL. The problem was that two IsTrackingMetadata() calls were under DCHECK macro. Those calls didn't happen in release builds with `dcheck_always_on=false` GN arg. The fix: promote DCHECK to CHECK to make it debug/release/dcheck_always agnostic. Original description: Improve WebAppSyncBridge::UpdateSync to use both current_state (before commit) and new_state (to be commit). In the future, it will help to skip local updates which don't touch sync data. Add WebAppSyncBridge::CommitUpdate unit tests (a.k.a. Local Changes). These tests is a promised follow up for this CL: https://chromium-review.googlesource.com/c/chromium/src/+/1830494 This code is disabled by default behind kDesktopPWAsWithoutExtensions and kDesktopPWAsUSS base features. TBR=treib@chromium.org Bug: 860583 Change-Id: I80a657c0d595bc03bebb8e241b1866daf5aa077e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1915647Reviewed-by:
Alexey Baskakov <loyso@chromium.org> Commit-Queue: Alexey Baskakov <loyso@chromium.org> Cr-Commit-Position: refs/heads/master@{#715207}
-
Reilly Grant authored
BluetoothAdapter is a reference counted object and so BluetoothDeviceChooserController should own it using a scoped_refptr. Fixing this requires also fixing BluetoothAdapterFactoryWrapper's AcquireAdapterCallback to take a scoped_refptr rather than a raw pointer. A test for proper ownership has been added. Bug: 1024121 Change-Id: I6342322e059f9cbff2a0d5f073f6bccfb0ca7c36 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1914536Reviewed-by:
Matt Reynolds <mattreynolds@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#715206}
-
Yuzu Saijo authored
This CL adds is_in_back_forward_cache_ bit in ServiceWorkerProviderHost instead of each time referring to RenderFrameHost's is_in_back_forward_cache(). Currently, SWPH and RFH have different lifetime and that is why this bit should be kept in SWPH. Once SWPH and RFH have the same lifetime, this bit will not be needed any more. Change-Id: Ie6dfb3608577f663cb37770e4b77166c8fca7e58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1909059Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: Yuzu Saijo <yuzus@chromium.org> Cr-Commit-Position: refs/heads/master@{#715205}
-
Christopher Cameron authored
When launching a PWA, it should start where the user left off (that is, with the profiles open that the user had open when they quit). In order to do this, we need to save in local storage the list of profiles for which each app was last active. While we are creating local storage for PWAs, we often need to know for what profiles an app is installed. This is currently implemented by poking through the user-data-dir in the filesystem. Putting this in local storage is much simpler. To solve both of these problems, add the AppShimRegistry which maintains the following structure in local storage. "app_shims":{ "<app_id>":{ "installed_profiles":[ "<relative_profile_path_1>", "<relative_profile_path_2>", ], "last_active_profiles":[ "<relative_profile_path_1>", ], } } Add hooks to maintain these |installed_profiles| to AppShortcutManager, because all shim creation and deletion goes through that structure. We will at some point want to move this to a PWA-specific component. Do not yet add hooks to maintain |last_active_profiles| because this patch is getting too big. Do add the code indicating where it will be used in ExtensionAppShimHandler, and update its tests to not crash when using AppShimRegistry. Add lots o tests. Bug: 1001213 Change-Id: I51d931567d9c3a4e1db143aa2b6164dcaa1a0f3e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1910719Reviewed-by:
Dominick Ng <dominickn@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#715204}
-