- 09 Sep, 2020 40 commits
-
-
Maksim Ivanov authored
Drop std::move() calls that have no effect and only trigger the "bugprone-use-after-move" clang-tidy warning. Bug: 1122844 Change-Id: Ibdc6b74cd97ef71288e897317bbe81f8f5c740fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382758 Commit-Queue: Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#805504}
-
David Van Cleve authored
This reverts commit fb37fef9. Reason for revert: seems like this breaks the tree, https://ci.chromium.org/p/chromium/builders/ci/mac-arm64-rel/1171? Original change's description: > Roll src/third_party/libjpeg_turbo/ 9d4f8005b..f38898135 (7 commits) > > https://chromium.googlesource.com/chromium/deps/libjpeg_turbo.git/+log/9d4f8005bc6c..f38898135a3b > > $ git log 9d4f8005b..f38898135 --date=short --no-merges --format='%ad %ae %s' > 2020-07-01 jonathan.wright Implement quantization using Arm NEON intrinsics > 2020-07-01 jonathan.wright Implement sample conversion using Arm NEON intrinsics > 2020-08-25 jonathan.wright Add compiler-independent alignment macro > 2020-07-01 jonathan.wright Add Arm NEON implementation of RGB->Grayscale > 2020-07-01 jonathan.wright Implement RGB->YCbCr using Arm NEON intrinsics > 2020-06-30 jonathan.wright Add Arm NEON implementation of h2v2_downsample > 2020-06-30 jonathan.wright Add Arm NEON implementation of h2v1_downsample > > Created with: > roll-dep src/third_party/libjpeg_turbo > > Bug: 922430 > Change-Id: I4edeccd8e58a529a5a7198448877756e7f7b7e22 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401461 > Reviewed-by: Leon Scroggins <scroggo@google.com> > Commit-Queue: Jonathan Wright <jonathan.wright@arm.com> > Cr-Commit-Position: refs/heads/master@{#805494} TBR=scroggo@google.com,cblume@chromium.org,jonathan.wright@arm.com Change-Id: I3386062045c36b44d7c9ef247dd009b978ccd3f0 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 922430 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401585Reviewed-by:
David Van Cleve <davidvc@chromium.org> Commit-Queue: David Van Cleve <davidvc@chromium.org> Cr-Commit-Position: refs/heads/master@{#805503}
-
Dan Sanders authored
Bug: 1125946 Change-Id: I6fd6072073704a6d2b9baa411acc414ebda5e043 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2399083 Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Reviewed-by:
Chrome Cunningham <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#805502}
-
Marijn Kruisselbrink authored
The save file dialog needs to truncate any files that are opened. Before this was using base::File directly. This CL routes truncating through the file system backends to facilitate supporting non-local files. Bug: 1093653 Change-Id: I51c2cf8a776de70375e57be7a8a5d02641226bfa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2392414 Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#805501}
-
Darwin Huang authored
This was likely missed earlier, but was noticed during https://crrev.com/c/2401500. Bug: 832129 Change-Id: Id808812aefc73cf3218db3ee915197b4da7937c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2402101 Commit-Queue: Darwin Huang <huangdarwin@chromium.org> Auto-Submit: Darwin Huang <huangdarwin@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Cr-Commit-Position: refs/heads/master@{#805500}
-
Kevin Ellis authored
When removing a replaced animation, it is important to consider all timelines. Previously, we made the decision on a per timeline basis, resulting in failing to flag some animations as replaceable. Bug: 10293484 Change-Id: I8256f2eb69f710ba1ec04b095446bd30c1b64a97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401505Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Kevin Ellis <kevers@chromium.org> Cr-Commit-Position: refs/heads/master@{#805499}
-
Miguel Casas authored
This CL extends Media.VaapiWrapper.VADisplayStateInitializeSuccess and takes the chance to clean up the owner/cc list. Bug: 1125687 Change-Id: I43b0bc32ad915d7e5680c64fafc73a2719e98146 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401998 Commit-Queue: Miguel Casas <mcasas@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Auto-Submit: Miguel Casas <mcasas@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#805498}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/48df087f8a86..cee574832664 2020-09-09 jdapena@igalia.com GCC: init kImageMemoryBarrierData with initializer constructor 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/+doc/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;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: chromium:819294 Tbr: ianelliott@google.com Change-Id: I4c6a9f9257140433134e766123350219a28d2452 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401869Reviewed-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@{#805497}
-
Sven Zheng authored
This reverts commit ee48012a. Reason for revert: Breakage. Original change's description: > [lacros] LUCI config for chromeos-amd64-generic-lacros-dbg > > Adds CI bot for chromeos-amd64-generic-lacros-dbg. > Also changes chromeos_amd64-generic_lacros, > chromeos_amd64-generic_lacros_rel to use other pre-defined configs > and it should have no differences. > > Bug: 1121667 > Change-Id: I6405da5c9ec0464fed59effeadcb3758f18ce927 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2398976 > Reviewed-by: Yuke Liao <liaoyuke@chromium.org> > Commit-Queue: Sven Zheng <svenzheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#805467} TBR=erikchen@chromium.org,liaoyuke@chromium.org,svenzheng@chromium.org Change-Id: Ibd04e9914cae8fcd6ad3d35ea6ad6a40970af9f1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1121667 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2402091Reviewed-by:
Sven Zheng <svenzheng@chromium.org> Commit-Queue: Sven Zheng <svenzheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#805496}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/6acdaf066c12..b0da2f1d4f9e 2020-09-09 heiserya@google.com Update WebPageReplayGo dependency documentation 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 rmhasan@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/+doc/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: None Tbr: rmhasan@google.com Change-Id: I955905340920a1544868bdd751d89d4ca97e2e8b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2402040Reviewed-by:
-
Jonathan Wright authored
https://chromium.googlesource.com/chromium/deps/libjpeg_turbo.git/+log/9d4f8005bc6c..f38898135a3b $ git log 9d4f8005b..f38898135 --date=short --no-merges --format='%ad %ae %s' 2020-07-01 jonathan.wright Implement quantization using Arm NEON intrinsics 2020-07-01 jonathan.wright Implement sample conversion using Arm NEON intrinsics 2020-08-25 jonathan.wright Add compiler-independent alignment macro 2020-07-01 jonathan.wright Add Arm NEON implementation of RGB->Grayscale 2020-07-01 jonathan.wright Implement RGB->YCbCr using Arm NEON intrinsics 2020-06-30 jonathan.wright Add Arm NEON implementation of h2v2_downsample 2020-06-30 jonathan.wright Add Arm NEON implementation of h2v1_downsample Created with: roll-dep src/third_party/libjpeg_turbo Bug: 922430 Change-Id: I4edeccd8e58a529a5a7198448877756e7f7b7e22 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401461Reviewed-by:
Leon Scroggins <scroggo@google.com> Commit-Queue: Jonathan Wright <jonathan.wright@arm.com> Cr-Commit-Position: refs/heads/master@{#805494}
-
Peter Boström authored
This makes views::Combobox use a base::RepeatingCallback internally. Pre-existing uses of ComboboxListener are wrapped in a callback in set_listener(). Cleanups removing ComboboxListeners will follow. Bug: 772945 Change-Id: I2c1b4a7637ba5125fd7774e1f18f4751ced9959c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401951 Commit-Queue: Peter Boström <pbos@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Auto-Submit: Peter Boström <pbos@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#805493}
-
Prabir Pradhan authored
The geometry sent to the client is in screen coordinates in DP, as opposed to the display's coordinates. Scaling this value does not make sense, as different displays could have different scales. Ideally, we would send a display identfier and the geometry in display coordinates. However, since we don't have a mapping of display_id to wl_output yet in zcr_remote_shell, to avoid adding another soon-to-be-deprecated event to the protocol, we just remove the scaling. BUG=b:167267365 TEST=Manual with client-side change: verify input works on multiple displays with different scale factors. Change-Id: I250bb3f0c631d8783edae39a71b4fb84532d1729 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2388720 Commit-Queue: Prabir Pradhan <prabirmsp@chromium.org> Reviewed-by:
Mitsuru Oshima (Slow: gardener) <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#805492}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/ce08265ba5b7..b24900a1aa39 2020-09-09 sugoi@google.com Update PowerVR Samples to ToT 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/+doc/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;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: swiftshader-team+autoroll@google.com Change-Id: I4d8259c816e9ce5f69fc22ccc5765010c06e818e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401877Reviewed-by:
-
Lukasz Anforowicz authored
CorbAllowlistAlsoAppliesToOorCors (aka CORS-for-content-scripts) has been shipping to the stable channel with Chrome85 since Aug 25th (~2 weeks ago). There have been just a handful of allowlisting requests so far - therefore, it seems okay to declare that this change has successfully shipped to the stable channel. This CL transitions enabling CorbAllowlistAlsoAppliesToOorCors away from 1) fieldtrial_testing_config.json and 2) field trial config (this includes preserving/copying the allowlisted extensions from site_isolation/data/CorsAndCorbAllowlist.gcl to //extensions/browser/url_loader_factory_manager.cc). After this CL we still retain an ability to allowlist extensions: 1. Via field trial config (affecting Chrome versions prior to this CL - Chrome 85, Chrome 86 and a little bit of Chrome 87) 2. Via the hardcoded allowlist in //extensions/browser/url_loader_factory_manager.cc (although this ability will go away soon - after https://crrev.com/c/2388709). This CL unblocks the following follow-up CLs: 1. Adding |max_version| to the field trial config (google3 - cl/330527651) 2. Starting deprecation of the allowlist (chromium3 - https://crrev.com/c/2388709) Bug: 920638 Change-Id: I635eb0777dc83a7ad448f57b9de388ae071e0c9b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2359393 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Cr-Commit-Position: refs/heads/master@{#805490}
-
Marc Treib authored
This is essentially a revert of crrev.com/c/1082267 (undoing the actual 1-line change from that CL, but keeping the comment improvements). This is a temporary measure to investigate crbug.com/1120899. Bug: 1120899, 1082267 Change-Id: I7761418c038824218a7fc36b325f1b364f7bbbe5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401460 Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#805489}
-
Siyu An authored
1. Implemented CreateMetadataChangeList function. 2. Load metadata and notify processor ready to sync in the bridge initializer. Bug: 1112095 Change-Id: Id510d65145085d55676a4fc492e560fd194ac292 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2399818 Commit-Queue: Siyu An <siyua@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#805488}
-
Reilly Grant authored
Now that we are starting to use the Core Location API for geolocation the Chrome app should probably have the location entitlement. Bug: 1051591 Change-Id: I66ff154a77298179737f12d4c00d43153422e325 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401269Reviewed-by:
Leonard Grey <lgrey@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#805487}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/21205c372020..eedd5bcc37d4 2020-09-09 hjd@google.com Merge "Replace size() == 0 with empty()" 2020-09-09 hjd@google.com Merge "trace_processor: DecompressTrace should fail on non-protos" 2020-09-09 fmayer@google.com Merge "Update docs to reflect heap_profile works on Windows." 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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: I3468d611801729a1d3bf91a947c1610cc358d055 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401864Reviewed-by:
-
David Van Cleve authored
This change expands the Trust Tokens browser tests to cover additional redemption-related cases: redemption to issuer with no tokens -> expect error redemption after successful issuance, to other issuer -> expect error redemption after successful issuance, against same issuer -> should handle server error, failing the operation successful issuance, redemption, another redemption -> should hit SRR cache successful issuance, redemption, issuer-prompted refresh -> should execute redemption despite SRR cache successful issuance, redemption, refresh prompted from non-issuer context -> should error issuance from origins A+B, cors mode redemption to origin A, redirect to origin B -> redemption succeeds, using origin B as issuer issuance from origin A yielding exactly 1 token, cors mode redemption to origin A, redirect to origin A -> redemption succeeds (same token is sent) issuance from origin A, no-cors mode redemption to origin A, redirect to origin B -> redemption succeeds, obtain origin A SRR A prior change (crrev.com/c/2308860) expanded coverage by adding more issuance-related tests; a subsequent change will add more signing tests. Bug: 1071293 Change-Id: Iaa654c99162dbaa735b3b9fe2e3aad359e9c27ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2309007 Commit-Queue: David Van Cleve <davidvc@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#805485}
-
Josip Sokcevic authored
code.google.com redirects no longer work. This CL replaces links in /testing/libfuzzer to source.chromium.org equivalent. This CL was uploaded by git cl split. R=kcc@chromium.org Change-Id: I232ff49bcc35a9f8df60868b6f8bb6b7544b8eef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2392750 Auto-Submit: Josip Sokcevic <sokcevic@google.com> Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Dirk Pranke <dpranke@google.com> Cr-Commit-Position: refs/heads/master@{#805484}
-
Josip Sokcevic authored
code.google.com redirects no longer work. This CL replaces links in /docs to source.chromium.org equivalent. This CL was uploaded by git cl split. Change-Id: I8014645395846f3e57117b7faaae9dd8c0ef6e3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2392761Reviewed-by:
-
Miguel Casas authored
This CL extends the histogram Media.VAVDA.VaapiWrapperCreationSuccess to "2021-08-05", just like its sibling Media.VaapiVideoDecoder.VaapiWrapperCreationSuccess. I took the chance to clean up the owners/cc list. Bug: 1125689 Change-Id: I9a459ac4dcf5480aa60bc08cf0c25ce2b968a087 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401511 Commit-Queue: Miguel Casas <mcasas@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Auto-Submit: Miguel Casas <mcasas@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#805482}
-
Becca Hughes authored
Moves the promise into the private part of the module this is so we don't show the module until the data has been loaded from the server. Change-Id: Ia770729be44ce0f802d462162132ad09c16e6169 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367876Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#805481}
-
Josip Sokcevic authored
code.google.com redirects no longer work. This CL replaces links in /native_client_sdk to source.chromium.org equivalent. This CL was uploaded by git cl split. R=bradnelson@chromium.org Change-Id: I82869389eff1a7fb5d2e04c9843f4a5f4dbc6d7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2392708 Auto-Submit: Josip Sokcevic <sokcevic@google.com> Reviewed-by:
Brad Nelson <bradnelson@chromium.org> Commit-Queue: Brad Nelson <bradnelson@chromium.org> Cr-Commit-Position: refs/heads/master@{#805480}
-
Sylvain Defresne authored
Generated by //tools/gn/roll_gn.py https://gn.googlesource.com/gn/+log/6f13aaac..e002e68a e002e68a Add support for symbolset to bundle_data targets 32c8eaf9 Add netbsd as a platform c0669368 Allow ".c++" as a C++ source file extension 58d9b89d [format] Sort visibility like deps 484245b9 clang: Fixes to module support b722d8ba Identify current toolchain on error Bug: none Change-Id: I588cb58da4ef228abf1f3e7c75cf557d6ba020d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401160 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Dirk Pranke <dpranke@google.com> Reviewed-by:
-
Stephanie Kim authored
Bug: 1066875 Change-Id: I888a0271e4c6cf8e1c0fdfd6a77460e504319c9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2399559Reviewed-by:
Erik Staab <estaab@chromium.org> Commit-Queue: Stephanie Kim <kimstephanie@google.com> Cr-Commit-Position: refs/heads/master@{#805478}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/0a1061da0c7f..97b880e6ff82 2020-09-09 aleksi@sapon.ca Add missing CMake spirv-cross dependency 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/+doc/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 Bug: None Tbr: cwallez@google.com Change-Id: I0d31065838b15da83f79e1354463c9d598bf1c8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401614Reviewed-by:
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: Ifb4f0aa2f2619a8fed01d5b5667b592e61c7afc6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401844Reviewed-by:
Ben Mason <benmason@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Commit-Queue: Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#805476}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/9b74c18d1728..4d42f9890bdc 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 davidvc@google.com,adithyas@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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:1109276 Tbr: davidvc@google.com,adithyas@google.com Change-Id: If9aeed590e88801bb54ebdb19f0c5240f22d11a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401874Reviewed-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@{#805475}
-
Kyle Milka authored
When content is shared in the Sharing Hub v1 include a preview showing the page favicon, title, and url. http://screen/704072c1-8ebf-4764-8660-0aa67c95dfa4.png Bug: 1120093 Change-Id: Iea5b67f063fcf663917badd84bdeacb5942959dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367442 Commit-Queue: Kyle Milka <kmilka@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Reviewed-by:
Tanya Gupta <tgupta@chromium.org> Cr-Commit-Position: refs/heads/master@{#805474}
-
David Van Cleve authored
This reverts commit 4f07b55d. Reason for revert: * DevTools browser tests failing on Win7 Tests (dbg)(1) * Culprit range https://chromium.googlesource.com/chromium/src/+log/21a14c8bdf3e885c3131f7f8b94c16bed42df603%5E..3f69f1b139fce1d827865806b5c1432a477e34cd * This is the only DevTools-related change in that range Original change's description: > Roll DevTools Frontend from 88a5d76f5f62 to 214971471446 (2 revisions) > > https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/88a5d76f5f62..214971471446 > > 2020-09-09 jacktfranklin@chromium.org Expose the live directive from LitHtml > 2020-09-09 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools DEPS. > > 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/+doc/master/autoroll/README.md > > Bug: None > Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com > Change-Id: Ie7af2d4c541d054337a72daa4e11c9530b120927 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401299 > Reviewed-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@{#805365} TBR=chromium-autoroll@skia-public.iam.gserviceaccount.com,devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Idd4098ed2e359c4513a2479e3fdd02e1c9249499 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: None Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401908Reviewed-by:
-
Noah Rose Ledesma authored
Add an off-by-default flag to enable the redesigned Global Media Controls UI. Bug: 1124401 Change-Id: I499eba511d873d1152bfd58b68948e311a50fc2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2398889 Commit-Queue: Noah Rose Ledesma <noahrose@google.com> Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#805472}
-
Anne Lim authored
Please see bug for design doc. Bug: 1121806 Change-Id: I2b847d3720f4b4c5fc2c9d7db4195c1df5367c72 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2390970Reviewed-by:
Jared Saul <jsaul@google.com> Reviewed-by:
Tommy Martino <tmartino@chromium.org> Reviewed-by:
Cait Phillips <caitkp@chromium.org> Commit-Queue: Anne Lim <annelim@google.com> Cr-Commit-Position: refs/heads/master@{#805471}
-
Vincent Boisselle authored
Bug: 1126510 Change-Id: I7ca19081e54aaee7d6c4116a0773fda2da14efe8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401701 Commit-Queue: Vincent Boisselle <vincb@google.com> Reviewed-by:
Dan H <harringtond@chromium.org> Cr-Commit-Position: refs/heads/master@{#805470}
-
Alison Maher authored
As resolved by the CSSWG [1], we should respect system color rules set by the author/user in Forced Colors Mode. This CL adds the logic to accomplish this. [1] https://github.com/w3c/csswg-drafts/issues/4178 Bug: 970285 Change-Id: I9eb4a33f8767f48d0c0a6fd65faec15f11edc140 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2390865 Commit-Queue: Alison Maher <almaher@microsoft.com> Reviewed-by:
Anders Hartvoll Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#805469}
-
Daniel Classon authored
The initializeKeyboardWatcher WebUI event was missing from the Device Page browser proxy. Add it in, and replace chrome.send in the Manage A11y subpage. Bug: None Change-Id: I5cc568e52ab6c602b1af49794e60368aa9d9f170 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2399602 Commit-Queue: Daniel Classon <dclasson@google.com> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#805468}
-
Sven Zheng authored
Adds CI bot for chromeos-amd64-generic-lacros-dbg. Also changes chromeos_amd64-generic_lacros, chromeos_amd64-generic_lacros_rel to use other pre-defined configs and it should have no differences. Bug: 1121667 Change-Id: I6405da5c9ec0464fed59effeadcb3758f18ce927 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2398976Reviewed-by:
Yuke Liao <liaoyuke@chromium.org> Commit-Queue: Sven Zheng <svenzheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#805467}
-
Aaron Colwell authored
This change converts the site_url param to a SiteInfo in preparation for making a similar change to DoesSiteURLRequireDedicatedProcess(). This is part of an ongoing process to replace site URL usage in method signatures with SiteInfo. This change also includes a few minor cleanups related to default SiteInstances. This change does not introduce any user visible behavior changes. - Changed CanBePlacedInDefaultSiteInstance() to take a SiteInfo instead of site URL. - Removed allow_default_site_url from GetSiteForURLInternal() and updated callers to handle default SiteInstance cases instead. - Added SiteInfo::CreateForDefaultSiteInstance() and updated code to use that instead of doing a complicated path through ComputeSiteInfo(). - Added SiteInstanceImpl::SetSiteInfoToDefault() to make the conversion to a default SiteInstance a little more explicit. Bug: 1085275 Change-Id: Ib65a41201902c48b13b82172304c8afe15b997fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2399016 Commit-Queue: Aaron Colwell <acolwell@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#805466}
-
Kurt Catti-Schmidt (SCHMIDT) authored
This change fixes a particular issue where the Javascript "onfocus()" event doesn't fire when accessibility API's call focus() on a specific node when the frame containing that node doesn't currently have focus. Blink's implementation of "onfocus()" in Document::SetFocusedElement has a condition under GetPage()->GetFocusController().IsFocused() to prevent focus from being fired if the page doesn't have focus. This is correct behavior from Javascript's perspective. From the accessibility side, behavior was already added to focus on the document before focusing on a specific node in https://chromium-review.googlesource.com/c/chromium/src/+/1809943 However, when that change calls into RenderFrameHostImpl::AccessibilityViewSetFocus, if the view is a RenderWidgetHostViewChildFrame, nothing will happen, as that method is empty. So this change completes that method with a simple focus implementation. A browser test was added to confirm that the Javascript onfocus event is properly fired. Bug: 1124067 Change-Id: I5a77fcdfba96da45fe548e93c2da8c9a69181066 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2389005 Commit-Queue: Kurt Catti-Schmidt <kschmi@microsoft.com> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#805465}
-
