- 16 Dec, 2020 40 commits
-
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4fa506cee933..b528247a5a9e 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 aboxhall@google.com,btolsch@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:1150600 Tbr: aboxhall@google.com,btolsch@google.com Change-Id: Ie96ce47e9ae6ff53a1d46ecbb05d797be05daabe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594125Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#837453}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/b35ae002390e..fea4c14580b1 2020-12-15 rharrison@chromium.org Fix missing include that causes Skia builds to fail 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 rharrison@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: rharrison@google.com Change-Id: I046f7a1bbd48e0c05e9506affcd9f49d1d5718cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593926Reviewed-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@{#837452}
-
Nicholas Hollingum authored
The InstanceRegistry is the new/preferred way for application providers like borealis to observe their apps' windows. When using the instanceregistry, we also avoid the (potential) pitfall due to us performing all of our window registration in a method called "GetShelfAppId". Bug: b/175152663 Change-Id: I1f7893428d76edc507e6841ebd94ffbb68024867 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586445 Commit-Queue: Nic Hollingum <hollingum@google.com> Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#837451}
-
David authored
This lets the Chromium PDF viewer get it's ink resources from chormeos/componeonts/media_app_ui/resources/prod/lib/ink which is built internally and shipped into CrOS as part of the media app. The resources themselves are bundled via grit in chromeos_media_app_bundle_resources.h. In doing so we add the BUILD flag enable_use_media_app_ink: * if true, don't build third_party/ink and thus don't create gen/third_party/ink/ink_resources.pak, instead get resources from gen/chromeos/chromeos_media_app_bundle_resources.pak * if false, do what TOT currently does and build third_party/ink This allows us to only ever build 1 set of ink resources into the Chrome OS binary and avoid bloating the OS by ~ +2.4MB. See size increase in http://b/170161460#comment18 for more context. This change also updates the externs of the ink library which works with new ink resources and also the older ink resources (third_party/ink/build) checked into TOT. Regression tested with various builds of TOT, new ink resources & TOT and new ink resources & this cl for the resource plumbing (grd, flags etc) see regression testing in http://b/170161460#comment18 for more context. Note: * currently enable_use_media_app_ink is just = enable_cros_media_app but that might change. * linux-chromeos-chrome trybot will fail until http://cl/347557653 lands and rolls into chromium. There is some time between http://cl/347557653 rolling in and this cl landing which may mean the OS is bloated with 2 copies of ink resources but this should be ok as long as this is done before branch and in reality they will land in O(day) time. Bug: b/170161460, 1150244 Change-Id: I8e01ce9863dd00bcbab7b322ad3cf33c1424ec6f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2542926Reviewed-by:
David Lei <dlei@google.com> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Bugs Nash <bugsnash@chromium.org> Commit-Queue: David Lei <dlei@google.com> Cr-Commit-Position: refs/heads/master@{#837450}
-
Findit authored
This reverts commit aa51435c. Reason for revert: Findit (https://goo.gl/kROfz5) identified CL at revision 837244 as the culprit for failures in the build cycles as shown on: https://analysis.chromium.org/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2FhNTE0MzVjM2I0MWJkYjk3ZjY5ZmFhYzNmNjE0NDc4MGYyN2U2MWIM Sample Failed Build: https://ci.chromium.org/b/8860789670201602304 Sample Failed Step: unit_tests Original change's description: > Fix the AppWindowLauncherItemController::GetAppWindow crash issue. > > This CL is the 3rd option to resolve the crash of below call stack: > AppWindowLauncherItemController::GetAppWindow(aura::Window*, bool) > AppWindowLauncherItemController::SetActiveWindow(aura::Window*) > AppWindowLauncherController::OnWindowActivated() > > One possible guessing is the AppWindowLauncherItemController object has > been deleted, but still used by AppServiceAppWindowLauncherController. > So modify the destruct function, and set the controller as null in > AppWindowBase. > > BUG=1128491 > > Change-Id: Ied59f857653aa0a136cbe9f903ff79ce5856e34c > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586647 > Commit-Queue: Nancy Wang <nancylingwang@chromium.org> > Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> > Cr-Commit-Position: refs/heads/master@{#837244} No-Presubmit: true No-Tree-Checks: true No-Try: true BUG=1128491 Change-Id: Ia518fe2f01bbb128758982250405ffd470008c0e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594556 Cr-Commit-Position: refs/heads/master@{#837449}
-
Michael Kolomeytsev authored
The test checks window activity which in turn depends on a focus given by OS windowing system. Being run in parallel with other tests it looses the focus very quickly. So the test should be moved to interactive_ui_tests. Bug: 1080602 Change-Id: I39cd0afb41f2841fda467ee1ae89e3ff0dd083bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593370 Commit-Queue: Alexander Dunaev <adunaev@igalia.com> Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
-
Nidhi Jaju authored
This CL changes the status of the ReadableByteStream runtime enabled feature from "experimental" to "stable", so that it is turned on by default. See the Intent To Ship thread at: https://groups.google.com/a/chromium.org/g/blink-dev/c/AQip4hjqt5U/m/OwPgs9NmAgAJ Bug: 614302 Change-Id: Ie224d280b471e3bc8925d188d8cdbc44a8d1240a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586290Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Nidhi Jaju <nidhijaju@google.com> Cr-Commit-Position: refs/heads/master@{#837447}
-
Eugene But authored
Bug: 688063 Change-Id: I427598e4cddad41f3f3628db9507166a30615e84 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2591819 Commit-Queue: Eugene But <eugenebut@chromium.org> Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Cr-Commit-Position: refs/heads/master@{#837446}
-
Kelvin Jiang authored
DONE - remove feature flag DONE - update tests: some rule limit overrides/rule count constants in tests were changed, and tests for the old per-extension limit were deleted. DONE - enable documentation and enable on stable This CL also promotes the getAvailableStaticRuleCount API function to stable. Bug: 983299 No-Presubmit: True Change-Id: I742e66f2c633b4900ff92ed03bd4f234e95e0169 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2563238Reviewed-by:Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org> Cr-Commit-Position: refs/heads/master@{#837445}
-
Francois Beaufort authored
This CL makes sure NFC operations are suspended if NFC host is created when web_contents is not visible. It also cancels a pending push operation when NFC operations are suspended as described in spec. https://w3c.github.io/web-nfc/#handling-visibility-change Bug: 1156805 Change-Id: I143302d292d904b6a25853cc1ecd4e52898efc13 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2579791Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#837444}
-
Gordon Seto authored
Show high visibility dialog when the high visibility button is toggled from the Nearby share subpage. This keeps the behavior consistent with when high visibility is toggled from the tray. Bug: 1154830 Change-Id: I17f6015706bca6f6b7ae85904c768e91923211e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2590652 Commit-Queue: Gordon Seto <gordonseto@google.com> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#837443}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/f32fbb51e8a1..98b56e604256 2020-12-16 m.maiya@samsung.com Vulkan: Accumulate internal cache stats in renderer 2020-12-15 timvp@google.com Vulkan: Ignore glFlush to reduce vkQueueSubmits in Asphalt 9 2020-12-15 paulthomson@google.com Tests: update WhatsApp trace 2020-12-15 cclao@google.com Vulkan: Reduce driver uniform data to minimum if specConst is used 2020-12-15 m.maiya@samsung.com Add EXT_clip_cull_distance extension autogen code 2020-12-15 m.maiya@samsung.com Vulkan: Add support for internal cache hit and miss counts 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 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: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: None Tbr: cwallez@google.com Test: Test: Manual verification with angle_end2end_testsTest: Test: angle_perftests --gtest_filter="*whatsapp*" Change-Id: Id6d8b649d72bc44003bb7918217c65b4cd17b55c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593366Reviewed-by:
-
Hitoshi Yoshida authored
This CL does followings to fix compile errors with new IDL dictionaries - add or replace include files - update GN - fix the way to create a dictionary - fix to check if a member is non-null Bug: 839389 Change-Id: Ic68f2751c06c99204a80f0fa23235cee5d172653 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593849 Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Kentaro Hara <haraken@chromium.org> Auto-Submit: Hitoshi Yoshida <peria@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#837441}
-
Nicholas Hollingum authored
This request came up in crrev.com/c/2586445. We commit the test now, in this cl, to show that the behaviour doesn't change across the refactoring. Bug: b/175152663 Change-Id: I42a20c358e324655fb66b4fb1ec61131b30eab7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2584560Reviewed-by:
-
Ahmed Fakhry authored
See explanation on the bug. BUG=1159264 TEST=Tested with the new test case added in https://crrev.com/c/2553873 And observed no crashes. Change-Id: I3b2abe866dbab3ae9e9cb2547c1e48655ccac0d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594322Reviewed-by:
Sammie Quon <sammiequon@chromium.org> Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Cr-Commit-Position: refs/heads/master@{#837439}
-
Eugene But authored
Bug: 688063 Change-Id: I13f96d3308319fb59038b755273ed4757cce81a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2591918 Commit-Queue: Eugene But <eugenebut@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#837438}
-
Alan Cutter authored
Change-Id: I62ddf7f72a80fe99f89d7d71df16c4d497cfb0cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592290 Commit-Queue: Alan Cutter <alancutter@chromium.org> Auto-Submit: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Glen Robertson <glenrob@chromium.org> Cr-Commit-Position: refs/heads/master@{#837437}
-
Abhijeet Kandalkar authored
Commit(1e7c2506) has added a presubmit warning to prevent further uses of old downcast helpers. All the occurrences of old downcast helpers are removed from the codebase and these changes are not needed now. Bug: 891908 Change-Id: I0090438d1f8c835b153ae2d6d177389891948991 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592539Reviewed-by:
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=bindusuvarna@chromium.org Change-Id: I97d10c2b0bd9a0e8ebf6493cc4e26eea9cf47afb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594569Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#837435}
-
Ryan Keane authored
This CL updates the CastAudioChannelBroker interface to allow for Audio-Audio sync changes to come later without breaking downstream repos that have implementations of this interface. Bug: b/171495618 Change-Id: I5099ee8bf272192c385763148da32d0636b48050 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594168 Commit-Queue: Ryan Keane <rwkeane@google.com> Reviewed-by:
Yuchen Liu <yucliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#837434}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/664301800ba2..d6661897486a 2020-12-16 fancl@chromium.org auto-triage: Rebalance and rerun Estimator in anomaly range 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/+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: skyostil@google.com Change-Id: I327a0953062d524d76feecfe14b5a5e1145ed8d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594000Reviewed-by:
-
adamta authored
Uses new feed initializer with custom scroll delegate. Reorders component creation in NewTabPageCoordinator so that delegates can be passed appropriately. Renamed DiscoverFeedViewController to DiscoverFeedWrapperViewController for clarity on what it does. Bug: 1114792 Change-Id: Iccdc7bf0efeda2252e0937d8665f1c33b09d9b87 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593471 Commit-Queue: Adam Trudeau-Arcaro <adamta@google.com> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#837432}
-
alanlxl authored
Keep them around until Jan 2021. Bug: 1067048, 937063 Change-Id: Iead89a8a5e21c216dce6b9f37df2b0d98fa487d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592071 Commit-Queue: Xinglong Luan <alanlxl@chromium.org> Reviewed-by:
Andrew Moylan <amoylan@chromium.org> Cr-Commit-Position: refs/heads/master@{#837431}
-
Darren Shen authored
[1] broke virtual keyboard PK simulation in some cases, because the VK was sending VKEY_UNKNOWN as the key_code. Previously, when key_code is VKEY_UNKNOWN, the key_code was inferred from the `code`. However, after [1], we only inferred if the key_code was left out. [1] https://chromium-review.googlesource.com/c/chromium/src/+/2566868 Bug: 1159250 Change-Id: Ied31078e213fe9bc2f5129057d1e1d34e41e2779 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592074Reviewed-by:
Keith Lee <keithlee@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#837430}
-
CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-betty-pi-arc-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-eve-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-kevin-chrome TBR=chrome-os-gardeners@google.com Change-Id: I06764cde4f39d22929099b86f6bab6e6adb390e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592141Reviewed-by:
ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Commit-Queue: ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#837429}
-
Azeem Arshad authored
This CL fixes two styling issues with the receive confirmation dialog. 1) spacing between progress icon and dashed line 2) Fix outside margin for confirmation page contents. Before Screenshot: https://screenshot.googleplex.com/4vDbAT4uWSdZsZJ.png After Screenshot: https://screenshot.googleplex.com/rjHhgLqBRozaYKa.png Fixed: 1156037 Change-Id: I034605dca9da7da016ad8da091e9c3c1ece16dfc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593426Reviewed-by:
-
Gary Tong authored
Bumping up to 25% Capacity looks good so far: http://shortn/_0ICs5GbtYR Bug: 1042757 Change-Id: I9ce89c7f8f1c0034660ab64dd58299ffe0d3e80a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593596 Auto-Submit: Gary Tong <gatong@chromium.org> Commit-Queue: Gary Tong <gatong@chromium.org> Commit-Queue: Erik Staab <estaab@chromium.org> Reviewed-by:
Erik Staab <estaab@chromium.org> Cr-Commit-Position: refs/heads/master@{#837427}
-
chromium-internal-autoroll authored
Release_Notes: http://go/media_app-x20/relnotes/Main/media_app_20201215_RC00_ink_swa.html https://chrome-infra-packages.appspot.com/p/chromeos_internal/apps/media_app/app/+/dvLFoYsVHl4TxGr16CGZ27chK2r007z5LZvhTmtAuIsC If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/media-app-chromium-autoroll Please CC media-app@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 Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: None Tbr: media-app@grotations.appspotmail.com Change-Id: Id404bc61300b594e50d0080a077a4819314f4d92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594548Reviewed-by:
-
Heng-Ruey Hsu authored
Creates a virtual device when CCA opens a camera. Bug: b:151047537 Test: Manually test Change-Id: I3a2196e32e18ecfb127553f47178f8ebe43656c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2351933Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Wei Lee <wtlee@chromium.org> Commit-Queue: Heng-ruey Hsu <henryhsu@chromium.org> Auto-Submit: Heng-ruey Hsu <henryhsu@chromium.org> Cr-Commit-Position: refs/heads/master@{#837425}
-
Oriol Brufau authored
It has been enabled by default since M86. Bug: 457718 Change-Id: I89f115d310d5e0a3f83ad7cdcbabe4f52d9c2d72 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593377 Commit-Queue: Oriol Brufau <obrufau@igalia.com> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#837424}
-
Eugene But authored
Bug: 688063 Change-Id: I23fedc798c5f63f4d5b9199079c0840fc898a2c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2591721 Commit-Queue: Eugene But <eugenebut@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#837423}
-
Christopher Cameron authored
CanvasColorParams should have the user-specified canvas parameters. The user specifies color space, alpha, and format. The format can only be things like "unorm8" or "float16". The format doesn't specify things like BGRA vs RGBA vs RGBX. Canvas resource providers, meanwhile, need to be able to specify much finer detail -- the SkColorSpace, SkAlphaType, SkColorType. They also should include a SkFilterQuality (which is passed next to the CanvasColorParams right now). Copy-paste CanvasColorParams into CanvasResourceParams and use CanvasResourceParams instead. The next patches in this sequence will re-specialize CanvasColorParams from what it has drifted to, and make CanvasResourceParams more general. Bug: 1157747 Change-Id: Ia9776d2dea64148fa2e3b2d520428ca9f969cf4e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586835Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: ccameron <ccameron@chromium.org> Cr-Commit-Position: refs/heads/master@{#837422}
-
Brandon Wylie authored
Also drops the requirement that every TabObserver care about activity detachment events by default. Interested clients can still override this method and get custom behavior. Bug: 1157728, 1146320 Change-Id: Icb4ca185fb46075b141aa75d8c0ff57a187f5143 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2585996Reviewed-by:
Mei Liang <meiliang@chromium.org> Reviewed-by:
Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Commit-Queue: Brandon Wylie <wylieb@chromium.org> Cr-Commit-Position: refs/heads/master@{#837421}
-
Brandon Wylie authored
Bug: 1056147 Change-Id: Ib13cb84cc38791afa12b92a3ea4ce12a33a1c434 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2590653 Commit-Queue: Brandon Wylie <wylieb@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#837420}
-
Anne Lim authored
I forgot to set expiry_time_micros_, but it passed the unittest because I also forgot to call RemoveExpiredStrikes() in the unittest. Bug: 1061639 Change-Id: I80570e03202c22cca069cb2084351849a4979484 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594313Reviewed-by:
Jared Saul <jsaul@google.com> Commit-Queue: Anne Lim <annelim@google.com> Cr-Commit-Position: refs/heads/master@{#837419}
-
Yaron Friedman authored
Internal repo migrated to new function. BUG=783819 Change-Id: I00a18ede62983068e2e55698015f156a0c3de216 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593871Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Commit-Queue: Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#837418}
-
Andre Vincent authored
Middle slot promo: screenshot/4ZY3YFud3UN6Apn Modules: screenshot/mwLN3C9mnHP54rb Fixed: 1077442 Change-Id: I2201fcd0cfba081d0cdddd6d28639d10756e0728 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587617 Commit-Queue: Andre Vincent <andrevincent@google.com> Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Cr-Commit-Position: refs/heads/master@{#837417}
-
Azeem Arshad authored
This CL adds a progress icon and dashed line showing the share target details and connection token. File preview icon on the right side is TODO. This CL also updates confirmation token in send confirmation page to match the specs. Screenshot: https://screenshot.googleplex.com/5Vmf5iXCNsGDrdR.png Fixed: 1157297 Change-Id: Ic4378966ea2fe972fcb6d66ff1c2fc9371c41123 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2590591 Commit-Queue: Azeem Arshad <azeemarshad@chromium.org> Reviewed-by:
-
Jinsuk Kim authored
Post-launch clean-up of feature flag/constants used for experiments. Bug: 1006111 Change-Id: I11135d67aec98f9c34d6161122f1529a641c20ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578299Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Reviewed-by:
Mohsen Izadi <mohsen@chromium.org> Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Cr-Commit-Position: refs/heads/master@{#837415}
-
Hidehiko Abe authored
This CL enables Lacros IME. The condition is slightly complicated, because there was a bug in ash-chrome, and lacros-chrome IME requires the fix to be enabled. With this CL, ash-chrome starts to tell lacros-chrome about its internal IME support status. lacros-chrome respects the info to decide whether or not to enable IME on lacros-chrome. If both lacros-chrome and ash-chrome are newer than this CL, IME on lacros-chrome is automatically enabled. If ash-chrome is older, because the support status is not passed to lacros-chrome, it won't enable IME. If lacros-chrome is older, there's no handling code, so IME won't be enabled, either. Because "enable" by default status is introduce, another command line flag to forcibly disable wayland IME is introduced. BUG=1133732 TEST=Made sure IME on lacros-chrome is enabled with new ash-chrome. Made sure IME is not enabled with old ash-chrome. Change-Id: I9bca95ee300962c6af6add8c1f847cc905154230 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2589714 Commit-Queue: Hidehiko Abe <hidehiko@chromium.org> Reviewed-by:Robert Kroeger <rjkroege@chromium.org> Reviewed-by:
Michael Spang <spang@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#837414}
-
