- 04 Mar, 2020 40 commits
-
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/2fdaf67e5529..f1fe4488c8cf Created with: gclient setdep -r src-internal@f1fe4488c8cf 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 Bug: None Tbr: jbudorick@google.com Change-Id: I8d2a986c634a7e4ad45487d7adb1e2eae88876e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086995Reviewed-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@{#746705}
-
Carlos Caballero authored
A navigation might start but not commit, that is the navigation fails and the page will not change. A good example thereof is a navigation that returns a 204. Still in this case some state will have been cleared in the DidStartNavigation method. Which means the state shows does no longer correspond to the page we are on. There is really no reason to not wait until DidNavigationFinish to reset all this state, so this patches does so. The only exception are cookies, as these might get accessed during the navigation. A follow up patch will take care of these. Bug: 998171 Change-Id: I6146986a8b146128fad388da88f4fd2238571822 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895512Reviewed-by:
Balazs Engedy <engedy@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Commit-Queue: Andy Paicu <andypaicu@chromium.org> Cr-Commit-Position: refs/heads/master@{#746704}
-
Anastasiia Nikolaienko authored
This is a reland of e78513d7 Original change's description: > Migrate authenticator.js and it's dependencies to JS modules > > Bug: 1054394 > Change-Id: I106bd0edc547dc6eb2d680b409d1348d5b964079 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2066548 > Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> > Reviewed-by: Lei Zhang <thestig@chromium.org> > Reviewed-by: dpapad <dpapad@chromium.org> > Cr-Commit-Position: refs/heads/master@{#745762} Bug: 1054394 Change-Id: Ie1a63f522f5e29b2af327010236f962836a36074 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2083317Reviewed-by:
dpapad <dpapad@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> Cr-Commit-Position: refs/heads/master@{#746703}
-
Mario Sanchez Prada authored
Based on crrev.com/c/1847304, by Colin Blundell <blundell@chromium.org> This CL starts converts the message that kills the renderer, on top of the infrastructure already laid out by crrev.com/c/1991446. In addition to that, this CL implements the required changes to move to a model where the mojo pipe for RenderAccessibility is only valid while a kWebContents-compatible accessibility mode is enabled, in order to address some security-related concerns. And it also adds a new unit test to check that this new logic works as expected (i.e. that the mode changes in the renderer when the pipe is closed from the browser side). Bug: 1010831 Change-Id: Ic71c33f654af9fbc1c453d2aa04c858e8285c1a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2015045Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#746702}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/e1688b60caf7..a6d3a2dd41d4 git log e1688b60caf7..a6d3a2dd41d4 --date=short --first-parent --format='%ad %ae %s' 2020-03-04 andreperezmaselco.developer@gmail.com Refactor FuzzerPass::ApplyTransformation code duplication. (#3206) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@a6d3a2dd41d4 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,ianelliott@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 Bug: None Tbr: radial-bots+chrome-roll@google.com,ianelliott@google.com Change-Id: Ie83162057831184bf50b5ba5eba19e4b2c9c090d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087171Reviewed-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@{#746701}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/9e9493f29dbf..028a7a725cc6 git log 9e9493f29dbf..028a7a725cc6 --date=short --first-parent --format='%ad %ae %s' 2020-03-04 ancheng.qiao@arm.com Skip if GL_EXT_texture_compression_dxt1 isn't supported Created with: gclient setdep -r src/third_party/angle@028a7a725cc6 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 ianelliott@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-asan;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: ianelliott@google.com Change-Id: I4b79ec813633c06b5dd08481b4a0f1ebd9a5d355 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086991Reviewed-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@{#746700}
-
Kent Tamura authored
- DidDetachLayoutTree() - DidRecalcStyle() - DidSetSuggestedOption() - PopupDidHide() Move parts of the contents of DetchLayoutTree(), DidRecalcStyle(), SetSuggestedOption(), and PopupDidHide() of HTMLSelectElement to them. We need to call SelectType::DidDetachLayoutTree() in |HTMLSelectElement ::ChangeRendering()| before replacing select_type_. When a SELECT changed its type from MenuList to ListBox, DetachLayoutTree() in this function triggered the popup-disconnecting code after switching to ListBox. After this CL, select_type_->DidDetachLayoutTree() in DetachLayoutTree() doesn't trigger the popup-disconnecting code because select_type_ is a ListBoxSelectType. This CL has no behavior changes. Bug: 1052232 Change-Id: Iaff5691a670bd7570c7c47334f548cc84c4dc0bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2084332Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#746699}
-
Frédéric Wang authored
This CL implements new API in order to determine metrics of stretchy math operators and shape them, following the algorithm of [1]. This will be used to implement stretchy MathML symbols (for radicals and operators). It relies on information from the OpenType MathVariants table [2] and corresponding iterators implemented in [3]. [1] https://mathml-refresh.github.io/mathml-core/#the-glyphassembly-table [2] https://docs.microsoft.com/en-us/typography/opentype/spec/math#mathvariants-table [3] https://chromium-review.googlesource.com/c/chromium/src/+/2074678 Bug: 6606 Change-Id: I362edd91fa0ff7f1d2da4381dac1e731be1e0858 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2051923Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Commit-Queue: Frédéric Wang <fwang@igalia.com> Cr-Commit-Position: refs/heads/master@{#746698}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/8985fc91089f..c40c4bc38a76 git log 8985fc91089f..c40c4bc38a76 --date=short --first-parent --format='%ad %ae %s' 2020-03-04 johnkslang@users.noreply.github.com Add "news" section and component status. Created with: gclient setdep -r src/third_party/glslang/src@c40c4bc38a76 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/glslang-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,ianelliott@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 Bug: None Tbr: radial-bots+chrome-roll@google.com,ianelliott@google.com Change-Id: Ide6e636cc97c1529aee89a1e590c8a1781523b9f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087031Reviewed-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@{#746697}
-
James Cook authored
We're breaking the SplitSettingsSync project into two milestones: SplitSettingsSync is now: 1. The new OS sync settings section 2. The new OS prefs data model type 3. Toggles for Apps, OS prefs, Wallpaper and Wi-Fi in the new section 4. OS data types use the sync transport layer The new "SplitSyncConsent" feature is: A. New out-of-box consent dialog for OS sync B. Browser sync opt-in flow on Chrome OS C. Unconsented primary account on Chrome OS This CL adds the new base::Feature and moves the appropriate parts of the code to the new flag. Bug: 1013466 Test: updated browser_tests and unit_tests Change-Id: I5439b7dbac29a3a242e82ff63e099512e2aac655 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2073161 Auto-Submit: James Cook <jamescook@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#746696}
-
Chih-Hsuan Kuo authored
Bug=1054761 Test=build and pass CQ Merge-With: eureka-internal/367058 Change-Id: I64e33d87398eab09704ea63b6eaf8e3922e3f98d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2066397 Commit-Queue: Chih-Hsuan Kuo <chkuo@google.com> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Yuchen Liu <yucliu@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#746695}
-
Omer Katz authored
All known data races in HashTable based collections have been resolved. Bug: 986235 Change-Id: Ia7a22d707aaaea1ab3ebc70c47da0876fc426aa9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078617Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#746694}
-
Andy Paicu authored
Change-Id: I23074a6dbc88cc10aa5b6067377fd73d67e6b739 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087051 Auto-Submit: Andy Paicu <andypaicu@chromium.org> Reviewed-by:
Kamila Hasanbega <hkamila@chromium.org> Commit-Queue: Kamila Hasanbega <hkamila@chromium.org> Cr-Commit-Position: refs/heads/master@{#746693}
-
Matthew Mourgos authored
This change adds an accessibility page that can be shown by clicking a link on the marketing opt in screen. The link to show the accessibility screen is only shown if the user was shown the gesture navigation screen before getting to the marketing opt in screen. The accessibility page contains a toggle for changing the kAccessibilityTabletModeShelfNavigationButtonsEnabled pref. Bug: 976949 Change-Id: Ibbb1568389b86cd336d8c4b65fe275ca2b256ce7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078740 Commit-Queue: Matthew Mourgos <mmourgos@chromium.org> Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#746692}
-
Alan Cutter authored
This CL updates the web app manifest update checker to check the contents of icons specified in the latest manifest for differences against the locally installed icons. If there are any differences it will trigger the web app to update with the latest manifest data. Screencast: https://bugs.chromium.org/p/chromium/issues/attachment?aid=435726&signed_aid=__Gv5ahpYIVLkTCEKEXscA==&inline=1 Bug: 926083 Change-Id: If5d0229b41931cad305a164e420ba4b917df02d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2072344 Commit-Queue: Alan Cutter <alancutter@chromium.org> Auto-Submit: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Alexey Baskakov <loyso@chromium.org> Cr-Commit-Position: refs/heads/master@{#746691}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/779e58d10e2f..2fdaf67e5529 Created with: gclient setdep -r src-internal@2fdaf67e5529 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 Bug: None Tbr: jbudorick@google.com Change-Id: I3943626768e6b54a6eade1741c074795787c980a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086190Reviewed-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@{#746690}
-
Josh Simmons authored
Multiple places in the code check for Quick View being open after performing a series of actions that should open it. These have been refactored in a single helper function. Bug: 1058289 Change-Id: I8a55d141895395637d6679fe7ad22df3c50baf7e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086469 Commit-Queue: Josh Simmons <simmonsjosh@google.com> Commit-Queue: Noel Gordon <noel@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#746689}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/57b83355c515..a40862f237fc git log 57b83355c515..a40862f237fc --date=short --first-parent --format='%ad %ae %s' 2020-03-03 nigi@chromium.org Remove the prefix in BaseFont when trying to find a substitute font. 2020-02-29 nigi@chromium.org Enable RenderManyRectanglesWithExternalMemory for SkiaPaths/Skia. 2020-02-28 thestig@chromium.org Add dsymutil to DEPS for Mac builds. Created with: gclient setdep -r src/third_party/pdfium@a40862f237fc If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pdfium-autoroll Please CC pdfium-deps-rolls@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 Bug: None Tbr: pdfium-deps-rolls@chromium.org Change-Id: I2cace5a1262f00b82c47cc2429e6170930bb43fa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086953Reviewed-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@{#746688}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/64a3f8fcb749..c6d0fdfb4098 git log 64a3f8fcb749..c6d0fdfb4098 --date=short --first-parent --format='%ad %ae %s' 2020-03-04 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update Go Deps 2020-03-04 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ../src 13184a2d..2f96e324 (443 commits) 2020-03-04 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 b84969ad69c3..9e9493f29dbf (7 commits) 2020-03-04 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader 176f3a3dbbc9..126720bd2e57 (2 commits) Created with: gclient setdep -r src/third_party/skia@c6d0fdfb4098 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 herb@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 Bug: None Tbr: herb@google.com Change-Id: Ie3f871db69e66b34785230212e10480a1afd3c72 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086951Reviewed-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@{#746687}
-
Alan Cutter authored
This CL fixes the theme service to be able to toggle native increased contrast support. We were not considering IncreasedContrastThemeSupplier as a default theme and were not removing it when native increased contrast was disabled. Bug: 1056335 Change-Id: I42b59e58912de8224b3d7d4e84fb355cd7851dc5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078376 Commit-Queue: Alan Cutter <alancutter@chromium.org> Auto-Submit: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#746686}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/f6edc90900fe..bbd4f3e6052c git log f6edc90900fe..bbd4f3e6052c --date=short --first-parent --format='%ad %ae %s' 2020-03-04 dberris@chromium.org Pinpoint: Add logging to results2 generation Created with: gclient setdep -r src/third_party/catapult@bbd4f3e6052c 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 skyostil@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:chromeos-kevin-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 Bug: chromium:1050740 Tbr: skyostil@google.com Change-Id: I29ede5d35c19b58cfff0fe76dfe0c753353590d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086952Reviewed-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@{#746685}
-
Eric Willigers authored
All tests using SystemWebAppIntegrationTest are now parameterized, and run with both bookmark and web app backend implementations. Bug: 1058274 Change-Id: Iffd8f1bf1242bab126de70d36fb4683981915ff9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086461Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#746684}
-
Eric Willigers authored
The test suite SystemWebAppManagerBrowserTest is now parameterized, and runs with both bookmark and web app backend implementations. Not yet updated: SystemWebAppManagerAdditionalSearchTermsTest.AdditionalSearchTerms and SystemWebAppManagerNotShownInLauncherTest.NotShownInLauncher currently fails with web apps. Tracked in crbug.com/1054195 In system_web_app_manager_browsertest.cc, helper methods EvalJs and ExecJs are retired as they have no clients. TBR=pkasting@chromium.org Bug: 1054195 Change-Id: Ib76ed1e2b14f42bf81138c61f5c263652dbe1ad2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086355 Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Auto-Submit: Eric Willigers <ericwilligers@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#746683}
-
Alexey Baskakov authored
The directory structure before: <ProfileDir> WebApps Sync Data LevelDB Temp <temp_name> <app_id> Web Applications _crx_<app_id> After: <ProfileDir> Web Applications _crx_<app_id> Sync Data LevelDB Temp <temp_name> Manifest Resources <app_id> This is breaking change, all local BMO profile icons will be clean. We should land it now to avoid file migrations in the future. Note: "Sync Data/LevelDB" directory is created only if kDesktopPWAsSharedStoreService feature is off. We use shared <ProfileDir>/Sync Data/LevelDB/ otherwise. This code is disabled behind kDesktopPWAsWithoutExtensions feature. It introduces no behavior changes. Bug: 877898 Change-Id: I96af88386f3ce6031881d713b330eff2e32f33e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2084215 Commit-Queue: Alexey Baskakov <loyso@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#746682}
-
Luciano Pacheco authored
Bug: 992822 Change-Id: I0a5cbd7f2233a9079065fe9d85dffe4b39462007 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086352 Commit-Queue: Noel Gordon <noel@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#746681}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/18eac1a4..56ff573a Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Iec678ba39cecd98fee1cf96603d78d1afcdffd31 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086880Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#746680}
-
Tal Pressman authored
Bug: 1049056 Change-Id: I45bfba484c6d8c99190568a5965a33e996c37bbe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078520Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Commit-Queue: Tal Pressman <talp@chromium.org> Cr-Commit-Position: refs/heads/master@{#746679}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/40dd59329abe..f92fd4849f47 git log 40dd59329abe..f92fd4849f47 --date=short --first-parent --format='%ad %ae %s' 2020-03-04 szuend@chromium.org [utilities] Use Map instead of Array in TestFileSystem.Entry Created with: gclient setdep -r src/third_party/devtools-frontend/src@f92fd4849f47 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.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 Bug: chromium:1050549 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I1de31644754ce6983612c47cdc1f736b3f7fe3f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086189Reviewed-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@{#746678}
-
Giovanni Ortuño Urquidi authored
Example failed run: https://ci.chromium.org/p/chromium/builders/ci/Linux%20CFI/16572 TBR=altimin@chromium.org Bug: 1054194 Change-Id: I955aa5674f3b5896b5360d1b9be6d450820f4b9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086383Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#746677}
-
Regan Hsu authored
When the user switches the radio button from "PIN" to "PASSWORD" back to "PIN" quickly, the "PASSWORD" button is re-selected on it's own. In the brief moment that that "PIN" is selected, the button on the RHS, which should be labeled as "Set up Pin" is "Change Pin". In order to prevent the radio button from toggling from "PIN" back to "Password" on its own (without user action), the selectedUnlockType is checked again after active lockscreen preferences are changed asynchronously. Replacing "hidden" with a dom-if template shortens the delay of "Set up PIN" and "Change PIN" in a noticeable way, but it doesn't resolve the fact that "incorrect strings may be shown for a brief period, and updating them causes UI flicker" [1]. This still happens from time to time, and is not consistent. [1] https://tinyurl.com/vdxo262 In order to reduce UI flicker, which is unavoidable in instances of failure (fail to change from PIN to password), the "Set up PIN" and "Change PIN" button is modified to change immediately on user action to the intended value: if the user changes from password to PIN, "Set up PIN" becomes the default immediately (as it should). If the lock type fails to change from PIN to password, then UI flicker occurs in that "Set up PIN" changes back to "Change PIN" (as opposed to seeing the "Change PIN" change to "Set up PIN" with higher frequency). There is not a good way around this, the alternative being that a pinwheel of some sort should be visible as async tasks are being performed, and additional UI to show that CrOS was unable to change from PIN to password and vice versa. Bug: 1054327 Change-Id: Iade95bb3c3a44c90a69b069faa160f17b093107b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2079834 Commit-Queue: Regan Hsu <hsuregan@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#746676}
-
Giovanni Ortuño Urquidi authored
This reverts commit 57466f0e. Reason for revert: Tests are consistently timing out on debug builds: https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg?limit=50 https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/17377 Original change's description: > Restore End-To-End test on ChromeOS > > Bug: 1054935 > Change-Id: I2da8e4c8240647614728c21e351a6b2e940aaae0 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2074882 > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org> > Cr-Commit-Position: refs/heads/master@{#744800} TBR=xiyuan@chromium.org,antrim@chromium.org,rsorokin@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1054935, 1058022 Change-Id: Ie52bbd3036bebee69aaf836556aceb5e3322f970 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086385 Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#746675}
-
Peter Kotwicz authored
This CL fixes a crash when TrustedWebActivityClient#createLaunchIntentForTwa() is given a non-HTTPS URL BUG=1057524 TEST=TrustedWebActivityClientTest#createLaunchIntentForTwaNonHttpScheme() Change-Id: Ib16ebd309c2e450ac40183f1ef4f7c7c6bb4876f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2082485Reviewed-by:
Peter Conn <peconn@chromium.org> Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Cr-Commit-Position: refs/heads/master@{#746674}
-
Zain Afzal authored
This is an upstream of web_app_file_handling.externs.js to cl/294762966. Additional context: chooseFileSystemEntries is still in the draft stage and as such is not known to closure. This cl adds the function to the native file handling externs file and gives it a type definition matching the wicg spec. https://wicg.github.io/native-file-system/#api-choosefilesystementries Bug: 1040328,996088 Change-Id: I6f3d2efacc092d36149866f05fccac9cb6e98ada Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2084039 Commit-Queue: Zain Afzal <zafzal@google.com> Reviewed-by:
Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#746673}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/54bd00057324..9e9493f29dbf git log 54bd00057324..9e9493f29dbf --date=short --first-parent --format='%ad %ae %s' 2020-03-04 ynovikov@chromium.org Skip crashing dEQP-GLES3.functional.samplers.*_tex_3d.* on SwANGLE Created with: gclient setdep -r src/third_party/angle@9e9493f29dbf 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 ianelliott@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-asan;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: ianelliott@google.com Change-Id: Ibeeee8d1d36ff623e176883cca0caeef80f6177a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086185Reviewed-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@{#746672}
-
Giovanni Ortuño Urquidi authored
See bugs for details TBR=rtoy@chromium.org Bug: 941219, 1047163 Change-Id: Ic6e8111c9432a530a3b0947b7581d66c37209db6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086460Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#746671}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/20ed48e8e963..64a3f8fcb749 git log 20ed48e8e963..64a3f8fcb749 --date=short --first-parent --format='%ad %ae %s' 2020-03-04 bsalomon@google.com Revert "Use spin lock in SkIDChangeListener" Created with: gclient setdep -r src/third_party/skia@64a3f8fcb749 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 herb@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 Bug: None Tbr: herb@google.com Change-Id: I14c7d9df1208d4b74bd81abf6a596de03073805b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086187Reviewed-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@{#746670}
-
Alan Cutter authored
Bug: 1007786 Change-Id: I674710a2fd92a669c636ab190337483825bac6c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2076222 Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#746669}
-
Eric Willigers authored
GetAppIdForTab now supports web apps, in addition to bookmark apps and Chrome platform apps. AppServiceAppWindowWebAppBrowserTest is now parameterized by web_app::ProviderType. Bug: 1048055 Change-Id: I2fc9f656f88634d6b0c2f03a2531ea7914452797 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078377Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Commit-Queue: Eric Willigers <ericwilligers@chromium.org> Cr-Commit-Position: refs/heads/master@{#746668}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/eedab7912f7c..779e58d10e2f Created with: gclient setdep -r src-internal@779e58d10e2f 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 Bug: None Tbr: jbudorick@google.com Change-Id: Ie0890ec326eb07a1a407d0dc07fe976fecf6fed8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086186Reviewed-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@{#746667}
-
Giovanni Ortuño Urquidi authored
Flakes started again around: https://ci.chromium.org/p/chromium/builders/ci/Win7%20Tests%20%28dbg%29%281%29/81647 TBR=avi@chromium.org Bug: 985255 Change-Id: I1827cb12df13a490c8de00ae753c223d17c62f89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2086458Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#746666}
-