- 07 Nov, 2019 19 commits
-
-
Alexey Baskakov authored
Add follow up tests for this CL: https://chromium-review.googlesource.com/c/chromium/src/+/1797824 (Init, GetData, Identities). In next CL: Add more WebAppSyncBridge unit tests. Bug: 860583 Change-Id: Ifb7f0a66b78867ba7618a911de2847c4cc75752a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900523 Commit-Queue: Alexey Baskakov <loyso@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#713252}
-
Lei Zhang authored
PrinterCapabilitiesTest currently exercises GetSettingsOnBlockingTaskRunner() on the main thread, because blocking is allowed on the main thread in unit tests by default. Improve upon this by disallowing blocking on the main thread. Then change PrinterCapabilitiesTest test cases to call GetSettingsOnBlockingTaskRunner() on the appropriate task runner. Change-Id: Ic97f03b38775338134f7d80de6b6aa52fc21e21b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900772 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#713251}
-
Juanmi Huertas authored
After https://chromium-review.googlesource.com/c/chromium/src/+/1702861 there was missing the force_rgba boolean to be inside the serializer. There was also no a test written to validate RGB order in offscreencanvas vs onscreen canvas. Bug: 1007197 Change-Id: Icc51bd6f6c34dec8e7b70f12759d9b2ae7ef48b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1894037 Commit-Queue: Juanmi Huertas <juanmihd@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Reviewed-by:
Yi Xu <yiyix@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Cr-Commit-Position: refs/heads/master@{#713250}
-
John Rummell authored
Now the EME supports AV1 video, this is often bundled with OPUS audio streams in a MP4 container. So updating the list of audio formats supported in audio/mp4 streams to include OPUS. Also adds several new tests: 1) tests that verify unencrypted AV1 video + OPUS audio in a MP4 container can be handled. 2) a test to verify that encrypted OPUS in a MP4 container can be handled. Bug: 1009132 Test: new tests pass Change-Id: I0cf8d1d781073630e767326bfbcba06f5f29324a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1830402 Commit-Queue: John Rummell <jrummell@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#713249}
-
behdad authored
The story is not working as expected TBR=crouleau@chromium.org Bug: chromium:1021682 Change-Id: Id7d9f9ed59e5f51297cc04ef6deee21881b001c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901357Reviewed-by:
Behdad Bakhshinategh <behdadb@chromium.org> Commit-Queue: Behdad Bakhshinategh <behdadb@chromium.org> Cr-Commit-Position: refs/heads/master@{#713248}
-
Michael Spang authored
These functions do the same thing. The code in Display::GetRotationTransform computes a matrix that will be exact. This should remove the need for RoundNearZero(), but this is catiously left as a followup. Bug: 1019015 Change-Id: I64e30913094fd79212a9ded533ccb1bb39b7cbe3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1885634 Commit-Queue: Michael Spang <spang@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#713247}
-
David Staessens authored
This CL makes the video_decode_accelerator_(perf_)tests print an error message when waiting for a specific event times out. Currently a generic error will be printed when a timeout occurs: video_decode_accelerator_tests.cc:179: Failure Value of: tvp->WaitForFlushDone() Actual: false Expected: true This CL makes it easier to see that a timeout happened: Timeout while waiting for 'FlushDone' event TEST=./video_decode_accelerator_tests on nocturne BUG=1020668 Change-Id: I58d609056ece2685c882a96691061a164dda2996 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898895Reviewed-by:
Chih-Yu Huang <akahuang@chromium.org> Reviewed-by:
Andres Calderon Jaramillo <andrescj@chromium.org> Commit-Queue: David Staessens <dstaessens@chromium.org> Cr-Commit-Position: refs/heads/master@{#713246}
-
Lei Zhang authored
Change it to GetSettingsOnBlockingTaskRunner(), to better reflect the fact that the blocking pool no longer exists. Also rename GetPrinterCapabilitiesOnBlockingPoolThread() for the same reason. Change-Id: I4014eecc30611e94f31429477de56c54f90b3377 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900769Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#713245}
-
Etienne Bergeron authored
This CL introduce utility function to replace codepoints. * GetCodepointAtIndex(...) * ReplaceCodepointAtIndex(...). This is a step forward to fix crbug/1021720. The CL is lifting the logic to create a obscured text and refactor the code to use the introduced functions. * CreateObscuredText(...) It is fixing the rewrite of obscured characters when text is obscured. Both feature are probably not used together but the fuzzer is detecting mismatch of amount of lines when rewriting the text. The CL is also fixing an incorrect character replacement in ReplaceControlCharactersWithSymbols. The replacement was replacing a codepoint by two codepoints, which is incorrect based on assumption (crbug/1021720). Bug: 1021720 Change-Id: I6fed96819d4a07358170c696fb6a3429a9a2c0c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901428Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Robert Liao <robliao@chromium.org> Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Cr-Commit-Position: refs/heads/master@{#713244}
-
Ben Kelly authored
This CL removes various cross-thread affordances necessary to run cache_storage on a sequence off the IO thread. Specifically: 1. We bind the BlobDataItemReader implementation directly on the cache_storage sequence. 2. We remove cross-thread hopping that were previously required when blob Read/ReadSideData were called on the IO thread. These implementations are now generally inlined. 3. We remove class members used to cache values for GetSize(). These can now be computed on demand. 4. Many thread assertions were converted to sequence checks. We still need one cross-thread hop to register the blob at creation time since the BlobStorageContextImpl is still bound to the IO thread. Bug: 1021210 Change-Id: I6232d27959bcd1b00e2fb117ce443c53460ecff6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898173 Commit-Queue: Ben Kelly <wanderview@chromium.org> Reviewed-by:
enne <enne@chromium.org> Cr-Commit-Position: refs/heads/master@{#713243}
-
Peter Kotwicz authored
This CL moves TabObserverRegistrar to org.chromium.chrome.browser.customtabs.content in preparation for adding CustomTabActivity-specific logic BUG=1020911 R=peconn TBR=yfriedman (for moving TabObserverRegistrar to customtabs/content) Change-Id: I41b3a525590939104f5557a390ee5c862ac7bc03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901833 Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Cr-Commit-Position: refs/heads/master@{#713242}
-
Xianzhu Wang authored
The tests now test the common path of CAP and pre-CAP after crrev.com/c/1895081. Change-Id: Id4d567c23ccafcb318e687fe20833803bdce19fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901906Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#713241}
-
Wez authored
- Remove the unused WebEngineScreen stub. - Remove an errant BASE_EXPORT macro on cr_fuchsia::AgentManager. Change-Id: I5546c78b9428b56a707fa1b059fb60e9875cd337 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1896393 Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Auto-Submit: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#713240}
-
Eugene But authored
Bug: 1022152 TBR: gambard@chromium.org Change-Id: I51871d233e7ae1bce32d63525b551ac1a0d4faeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902519Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#713239}
-
Wez authored
Some allocators (e.g. the Blink heap) call DecommitSystemPages() to return pages in their free memory pools to the system, or DiscardSystemPages() to make the pages available to the kernel without necessarily de-committing them, rather than actually freeing them. At present Fuchsia does not implement page swapping, nor does it provide a "mark discardable" API, so we implement both APIs using vmar_op_range(DECOMMIT) for now. Bug: 927411, 1022062 Change-Id: Icc61e7b46ebdc4d3fa5308e1b318a7685a6ceb19 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892295Reviewed-by:
Albert J. Wong <ajwong@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Wez <wez@chromium.org> Auto-Submit: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#713238}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/61ec88a68d6d..c1501a6caf64 git log 61ec88a68d6d..c1501a6caf64 --date=short --no-merges --format='%ad %ae %s' 2019-11-06 vapier@chromium.org licensing: block new uses of LICENSE=Google-TOS 2019-11-06 chrome-bot@chromium.org Update config settings by config-updater. 2019-11-06 cjmcdonald@chromium.org DependencyService: Provide exact eclass dependency paths Created with: gclient setdep -r src/third_party/chromite@c1501a6caf64 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: I05ce0f96a5b1bc0182cf67a9d69360401139daeb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902366Reviewed-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@{#713237}
-
Sophie Chang authored
This also leaves a ShouldTargetNavigationAndCanApplyOptimization that replaced the existing CanApplyOptimization that did both. This is required for supporting the dev incentives project since they do not have an optimization target. Bug: 969558 Change-Id: If30266c282519697c27bdb35e93d91ad370c70c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899087Reviewed-by:
Michael Crouse <mcrouse@chromium.org> Reviewed-by:
Doug Arnett <dougarnett@chromium.org> Commit-Queue: Sophie Chang <sophiechang@chromium.org> Cr-Commit-Position: refs/heads/master@{#713236}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/2d87f9edb488..b4b80864f88f Created with: gclient setdep -r src-internal@b4b80864f88f If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None,chromium:None,chromium:None,chromium:922813,chromium:None Change-Id: I30b0ce12b7effb23cced55e19b00af036045b7fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902129Reviewed-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@{#713235}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/1171cdd4f791..ed392790fed1 git log 1171cdd4f791..ed392790fed1 --date=short --no-merges --format='%ad %ae %s' 2019-11-06 treehugger-gerrit@google.com Merge "processor: Some more fixes for chromium json export" Created with: gclient setdep -r src/third_party/perfetto@ed392790fed1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=perfetto-bugs@google.com Bug: None Change-Id: If2ac60b6aa82a8c529b68c6b68c7a8f47472a741 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902128Reviewed-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@{#713234}
-
- 06 Nov, 2019 21 commits
-
-
tby authored
There is a bug in the provider for zero state Drive file results that means it returns results even when a search query is issued, messing up the results for a search. This is caused by the result cache being cleared at the wrong time. Bug: 1022150 Change-Id: Ic1382e37a7991284c492963c57681d63f2f02a35 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902167Reviewed-by:
Thanh Nguyen <thanhdng@chromium.org> Reviewed-by:
Jia Meng <jiameng@chromium.org> Commit-Queue: Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#713233}
-
Tommy Steimel authored
This CL adds media session metadata (title, artist, and album) to the Now Playing Info Center when available. Bug: 936513 Change-Id: Id95728fd4ad1a06b242f6596475315fa65ec913c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902509Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#713232}
-
Nasko Oskov authored
WebUI has moved to be a per-frame object, but the SetWebUIProperty method is still on the RenderViewHost object. This CL moves it to be per-frame, therefore on the RenderFrameHost object, but does not make it available to subframes. The goal is to be a purely code move CL. Bug: 763548, 1002276, 1019739 Change-Id: Ie857a253f750316fe090ac2daa6686376cdb1833 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901587Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Commit-Queue: Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#713231}
-
Sarthak Shah authored
Currently, this feature flag exists in content. In master CL https://chromium-review.googlesource.com/c/chromium/src/+/1773872, this feature flag is utilized in CreateFromDefaultPlatformCurve to decide whether to use experimental fling curve or existing fling curve, which lives in ui. This CL moves the experimental fling animation flag from content to ui. Bug: 647137 Change-Id: Ia8051ec250ed6c8c8170d41eb3e52b2e32e9ffab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1884515Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Daniel Libby <dlibby@microsoft.com> Commit-Queue: Sarthak Shah <sarsha@microsoft.com> Cr-Commit-Position: refs/heads/master@{#713230}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/3dce06be0e24..714f8cc3ff4b git log 3dce06be0e24..714f8cc3ff4b --date=short --no-merges --format='%ad %ae %s' 2019-11-06 mtklein@google.com add vcmpps 2019-11-06 herb@google.com Improve the dynamic range of SkPackedGlyphID ctor 2019-11-06 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 2019-11-06 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 2019-11-06 mtklein@google.com unnest matrix multiply Created with: gclient setdep -r src/third_party/skia@714f8cc3ff4b 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 ethannicholas@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=ethannicholas@google.com Bug: None Change-Id: I64caa48b6a8c90eddfd0221ef372fcce8a0c96f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902127Reviewed-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@{#713229}
-
Mason Freed authored
This CL removes (except for the enum) the kEventPath RAPPOR metric, which was not being used. Bug: 1016906 Change-Id: Iaf10b47805a3054f39568d82a8720f52722d752f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1896055Reviewed-by:
Philip Rogers <pdr@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Commit-Queue: Mason Freed <masonfreed@chromium.org> Auto-Submit: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#713228}
-
Robert Ma authored
TBR=smcgruer Bug: 1022141 Change-Id: I01cd99945a58e8f8cba5ae2bbbcf977ece7f4c0d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902510Reviewed-by:
Robert Ma <robertma@chromium.org> Commit-Queue: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#713227}
-
Varun Khaneja authored
R=holte Bug: 963165 Change-Id: Iad7981b06d2541623bf58d0db98c314ed9f4ec30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898749 Commit-Queue: Varun Khaneja <vakh@chromium.org> Auto-Submit: Varun Khaneja <vakh@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Cr-Commit-Position: refs/heads/master@{#713226}
-
John Lee authored
Bug: 992972 Change-Id: Ic5490f7444f7783c7874502ea0ce75cdb6ba7576 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902237Reviewed-by:
Peter Boström <pbos@chromium.org> Commit-Queue: John Lee <johntlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#713225}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/33cff37c6035..4dffa04d4f00 git log 33cff37c6035..4dffa04d4f00 --date=short --no-merges --format='%ad %ae %s' 2019-11-06 mbonadei@webrtc.org Revert "Disable goma on GCC bots." 2019-11-06 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 3cb32697..33f41173 (712927:713078) 2019-11-06 danilchap@webrtc.org Stop using and delete DEPRECATED_SingleThreadedTaskQueueForTesting 2019-11-06 alessiob@webrtc.org RNN VAD: FC layer with SSE2 impl Created with: gclient setdep -r src/third_party/webrtc@4dffa04d4f00 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=webrtc-chromium-sheriffs-robots@google.com Bug: chromium:1021931,chromium:None Change-Id: I9b6579ba98806688b0ad7e73ab9c268c22a36743 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902126Reviewed-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@{#713224}
-
Michael Spang authored
Implementations of GetTransformedRootWindowBoundsInPixels() are currently applying the inverse of the desired transformation: the root window transform converts root window pixel coordinates to host pixel coordinates. This function needs to do the reverse: it takes host pixel coordinates, and returns root window pixel coordinates. The reason this was not causing bugs is that in production Chrome OS & Cast both override this function to throw out the origin of the returned rect. Setting the origin to zero after transforming corrects for applying the wrong transformation: the rest of the transformation is a 0, 90, 180, or 270 degree rotation. Since 0 and 180 degree rotations are self-inverses, and a 90 degree and a 270 degree rotation have identical effect on sizes, keeping only the size ensured correct results. Clean up this confusion by inverting the transformation applied by GetTransformedRootWindowBoundsInPixels(). This means that the origin of the transformed rect will automatically be correct (i.e., it will be zero), except during configuration (because bounds & transform aren't updated atomically). We still force the origin to zero to avoid any behavioral change from roundoff. There are some minor discrepancies related to that: - Chrome OS puts the window at origin, and rounds the size down - Cast puts the window at the origin, and rounds the size up - The default implementation allows arbitrary positioning, and takes an enclosing rect These discrepancies are unchanged in this patch. Bug: 1019015 Change-Id: I9bb40872be2fc76f04e63350a67965d44b71cc6e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1885633Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#713223}
-
Mason Freed authored
The prior test checked for deprecation messages, which is quite roundabout. This just feature-detects all three features. Change-Id: I7e034061b0c041b6341ec9eae9a5e3ab3b643c78 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900917 Auto-Submit: Mason Freed <masonfreed@chromium.org> Commit-Queue: Avi Drissman <avi@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#713222}
-
Sam Goto authored
Design-Doc: https://docs.google.com/document/d/1da8CjO71DlFbBzDcSosFRXTvDsKII_XMCohpZLQ_QbM/edit# Bug: 1015645 Change-Id: I81d0d8971b313bed6829614de431b2943173a0e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895319 Commit-Queue: Sam Goto <goto@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Alex Chau <alexchau@chromium.org> Cr-Commit-Position: refs/heads/master@{#713221}
-
Mustaq Ahmed authored
Bug: 959850 Change-Id: I15d8d20a9fbb9e5670c49d7fb51b36f7d7d6cb12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1899198Reviewed-by:
Alexander Cooper <alcooper@chromium.org> Reviewed-by:
Brandon Jones <bajones@chromium.org> Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Cr-Commit-Position: refs/heads/master@{#713220}
-
Charlene Yan authored
editor bubble. This is a temporary entry point for feedback while dog-fooding the feature. Bug: 1019903, 1015634 Change-Id: Ic51f4d5c58d313a7c2fe0ab7b10687fcd54aaea4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890993 Commit-Queue: Charlene Yan <cyan@chromium.org> Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#713219}
-
Steven Bennetts authored
This reverts commit 17342442. Reason for revert: This will cause problems on a device as locales/*.pak.gz will not get copied correctly. We need to update chromite first. Original change's description: > chrome_repack_locales: compress .pak files on Chrome OS > > For trivial change to resource_bundle.cc: > TBR=sky@chromium.org > > Bug: 1017864 > Change-Id: I2afbfa8e02f952d03b36112e0093af92753876ab > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1883292 > Commit-Queue: Steven Bennetts <stevenjb@chromium.org> > Reviewed-by: Andrew Grieve <agrieve@chromium.org> > Cr-Commit-Position: refs/heads/master@{#713205} TBR=stevenjb@chromium.org,agrieve@chromium.org Change-Id: Ia72498979433c12facbb4a797582bd48d18ce83e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1017864 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1902168Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#713218}
-
Luke Zielinski authored
Change-Id: I368ce92d78a771b1107e94963e7ed588fcb67ff1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1900561 Commit-Queue: Luke Z <lpz@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> Auto-Submit: Luke Z <lpz@chromium.org> Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Cr-Commit-Position: refs/heads/master@{#713217}
-
Mohammad Refaat authored
sameSitePolicy property was added to NSHTTPCookie with iOS 13, consider it when switching between canonical cookies and system cookies. Initially use CookieOption with access semantic Strict, but eventually download_session_cookie should update the semantic based on the origin of the request. This CL accommodates the changes that happened recently on the canonical cookie . So in iOS 12 we don't miss cross site non-secure cookies. and on the same time starting from iOS 13 check on same-site policy value and follow chromium rules in sending cookies across requests. Bug: 1018272, 1006785 Change-Id: I77213f528a9f533998e783db897a80a990bea87d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890507Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Maksim Orlovich <morlovich@chromium.org> Reviewed-by:
Lily Chen <chlily@chromium.org> Commit-Queue: Mohammad Refaat <mrefaat@chromium.org> Cr-Commit-Position: refs/heads/master@{#713216}
-
Frédéric Wang authored
This prepares implementation of the MathML Core specification: https://www.chromestatus.com/feature/5240822173794304 Minimal changes are applied to the build system in order to generate element classes for MathML elements, similarly to how it is currently done for HTML and SVG elements. A folder third_party/blink/renderer/core/mathml is created with the json files describing MathML attributes/elements as well as a simple implementation of a new MathMLElement class. For now, no new MathML attributes/elements are added and MathMLElement derives from Element without performing any implementation change. MathMLElement also contains standard helper functions in its header file. A new experimental feature MathMLCore is also introduced. When it is enabled, the document class will create MathMLElement instances for all MathML elements. When it is disabled, it will continue to just create Element instances. No new tests are added since there is no visible behavior change, independently of whether MathMLCore is enabled or not. Bug: 6606 Change-Id: I59c2ad8f70383716c5514f2ab3c818bdc11cb8e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1895661Reviewed-by:
Mason Freed <masonfreed@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Frédéric Wang <fwang@igalia.com> Cr-Commit-Position: refs/heads/master@{#713215}
-
Reilly Grant authored
The DeviceSensors.DeviceMotion and DeviceSensors.DeviceOrientation RAPPOR metrics can be replaced by the existing UKM-enabled Blink use counters for this feature. Bug: 1016906 Change-Id: I50cf7ac9ed26ac9b57f47cb8f74b74bb5f133612 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1898297 Auto-Submit: Reilly Grant <reillyg@chromium.org> Commit-Queue: Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Cr-Commit-Position: refs/heads/master@{#713214}
-
Eric Seckler authored
To enable us to judge the impact of crbug.com/1019181. Bug: 1019181 Change-Id: Iff85c850d61f10f4b5dfbfb6d996a050e99fd806 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1901469 Auto-Submit: Eric Seckler <eseckler@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#713213}
-