- 22 Aug, 2020 26 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/f6b801a1d036..88624993b0ee 2020-08-22 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: I592abafb175aeea0187bff9b45842dacd4ecb6ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369245Reviewed-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@{#800816}
-
Julie Jeongeun Kim authored
This CL moves PrintHostMsg_ScriptedPrint_Params to print.mojom to pass it to Mojo interfaces. Bug: 1008939 Change-Id: Ib8f84db7fbb2881f1a0cc2c2156a45f9ee88b946 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2345784Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Julie Kim <jkim@igalia.com> Cr-Commit-Position: refs/heads/master@{#800815}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/6821f818e6f5..46b51582a581 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 xhwang@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:1114451,chromium:1117077 Tbr: xhwang@google.com Change-Id: I70edcbbb24843680e68e3d3c89136c6d66e3b867 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369241Reviewed-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@{#800814}
-
Dylan Cutler authored
Before this change it was ukm_parameters(), which is against Blink naming convention. Also have the functions return a const ref. Bug: None Change-Id: Ie527e69712be37ac5b362d462acf9b2e376e7a29 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366115 Commit-Queue: Dylan Cutler <dylancutler@google.com> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#800813}
-
Gang Wu authored
Bug: 1119955 Change-Id: I2b5b3e6f14e96f24d5f21e08cddbae97ebf72c59 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369502Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Gang Wu <gangwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#800812}
-
Yan, Shaobo authored
Current fast path for webgl uploading video frame is to render YUV planars to a RGBA8 intermediate gl texture and than do gpu to gpu copy to update the dst webgl texture. It is possible to further optimize this progress by drawing the content from YUV planars to dst webgl texture directly. This will save the extra gpu-gpu texture copy which may hurt performance when uploading large resolution video (e.g. 4k/8k video). BUG=1108154 Change-Id: I0abb269944f9eac9fb77e13b01860152a918ad95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2319543Reviewed-by:
Brian Salomon <bsalomon@google.com> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Shaobo Yan <shaobo.yan@intel.com> Cr-Commit-Position: refs/heads/master@{#800811}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=mmoss@chromium.org Change-Id: If97cca0ca10a4f3293e05a3ce48068faa477e3c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369421Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#800810}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/bb85de36d277..fbf13333b426 2020-08-22 tsepez@chromium.org Remove core/fxge/fx_ge_fontmap.cpp 2020-08-22 thestig@chromium.org Switch from check_targets to no_check_targets. 2020-08-22 thestig@chromium.org Sync gtest/gmock with Chromium's copy. 2020-08-21 tsepez@chromium.org Remove virtual method SystemFontInfoIface::GetFaceIndex(). 2020-08-21 tsepez@chromium.org Nest CFWL_EventTarget inside CFWL_NoteDriver. 2020-08-21 tsepez@chromium.org Make CXFA_FFApp::m_pFontMgr be a pointer 2020-08-21 tsepez@chromium.org Mark CFWL_Event and its subclasses stack-only types. 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/+doc/master/autoroll/README.md Bug: None Tbr: pdfium-deps-rolls@chromium.org Change-Id: I0aa1c7e9e79e1aaf1dc317fca1d0c75354aa18bb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369716Reviewed-by:
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1598032222-bdae3e34bfdfdc02a739bd58b074dbcf46af85d4.profdata to chrome-mac-master-1598054321-87d797542a8415d31d407f7b135c9596dfac4588.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-mac-chromium Please CC jeffyoon@google.com,liaoyuke@google.com,sebmarchand+pgo_roller@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: chrome/try:mac-chrome Tbr: jeffyoon@google.com,liaoyuke@google.com,sebmarchand+pgo_roller@google.com Change-Id: Icf770efc46f19343a08e1bd1cafb1b9572888b3d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2370088Reviewed-by:
-
Daniele Castagna authored
When the root contents color usage changes between frames the renderer might reshape the output surface, increasing the damage to the whole frame. If SurfaceAggregator ignores quads outside the damage we might end up with a renderer that can't draw the full frame after reshaping the surface. This CL increases the damage in SurfaceAggragtor to the full frame when the color usage changes. Bug: b/158777601 Test: Youtube HDR video, no artifacts when it starts playing. Change-Id: Ibce88f6f4375567c1bbab1761ab7aea53f84c226 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369441 Commit-Queue: Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#800807}
-
Nick Burris authored
When the merchant requests a shipping address but doesn't specify any shipping options, all shipping addresses are invalid. On desktop, an error is shown when any address is selected, but on Android there is no error so the invalid state isn't obvious to developers. The discrepancy is that on desktop, RecomputeSpecForDetails is called even if the payment details are not updated by the merchant, which results in the error being set in UpdateSelectedShippingOption. This patch makes the Android implementation also recompute spec and display the error if necessary. Bug: 1082630 Change-Id: Idff32f2c2b51a617609381fee49a4ac5bd78206b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2280990 Commit-Queue: Nick Burris <nburris@chromium.org> Reviewed-by:
Liquan (Max) Gu <maxlg@chromium.org> Cr-Commit-Position: refs/heads/master@{#800806}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/ea8d6a46aad5..e6244ccef917 2020-08-21 rmhasan@google.com typ: Include tags in Expectation instance returned when querying for 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 zhanliang@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: chromium:1120189 Tbr: zhanliang@google.com Change-Id: Ic8ccfdd481eae070e84dd697f34e8b98cd4cdcca Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369350Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium 1d0b25f6. With Chromium commits locally applied on WPT: 3b705a74 "WPT: Service Worker: Add unexpected state check to wait_for_state()" 9f52c7e4 "COOP: add reporting to redirects" f3b221b0 "[COOP] access reporting: Extend report.body.type values." 5da0ed1e "Allow range requests to pass through a service worker" 3514ff91 "Audio|VideoDecoderBroker: more weakptr, keep callbacks on main thread." Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: foolip@chromium.org, lpz@chromium.org, robertma@chromium.org: external/wpt/tools NOAUTOREVERT=true TBR=foolip@google.com No-Export: true Change-Id: Ia036e83e6dad3829dc901552613206c4d3f22e7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368775Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#800804}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/0a98cc54ea0d..f6b801a1d036 2020-08-21 kahinds@microsoft.com Fix primary button color contrast on hover 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: chromium:1072064 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I3325a37648fcb7eef37f45d813fc46fd658a9b56 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2370003Reviewed-by:
-
Collin Baker authored
Bug: 1106523 Change-Id: I1876b84f2d24a4a664a9aaf5f19eb126d3543e3b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2360503 Commit-Queue: Collin Baker <collinbaker@chromium.org> Reviewed-by:
Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#800802}
-
David Grogan authored
When an aspect-ratio item has a definite cross size, we are supposed to compute the flex base size by multiplying those two. Blink did this when the item had a computed definite cross size, but not when the item had a definite cross size due to stretching. Firefox has the same bug. The modified canvas test relied on the buggy behavior. The tests we now fail also assert the buggy behavior, but are only one-way synced from mozilla's repository, so we can't change them. This fix is behind a flag. Bug: 704294 Change-Id: Ie17c451c7ec81ccb36fb96ff1543885bd7352bcc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368321Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#800801}
-
Chris Cunningham authored
Bug: 1120745 Change-Id: Idd2721941fee38a3ad4c6beb30ed1b39ff7759c9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368284Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Thomas Guilbert <tguilbert@chromium.org> Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Auto-Submit: Chrome Cunningham <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#800800}
-
Arthur Wang authored
Bug: 1060399 Change-Id: Ice41c7ef4154a07c6078a66bda05283161e1c13b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2316624 Commit-Queue: Arthur Wang <wuwang@chromium.org> Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Justin DeWitt <dewittj@chromium.org> Reviewed-by:
Haiyang Pan <hypan@google.com> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#800799}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/13c4dbe2ef24..bf7acb2707b8 2020-08-21 egdaniel@google.com Set input usage flag on all internally created vk RTs. 2020-08-21 reed@google.com remove unused enums for picture-recorder 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 stani@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-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 Cq-Do-Not-Cancel-Tryjobs: true Bug: None Tbr: stani@google.com Change-Id: Idc88f20545988d8f0ebe9316ecc62b5d65b700eb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369713Reviewed-by:
-
Collin Baker authored
This reverts commit 81bbe1e6. Reason for revert: compile error https://ci.chromium.org/p/chromium/builders/try/linux-chromeos-rel/654323? seems to conflict with this recent change: https://chromium-review.googlesource.com/c/chromium/src/+/2355015 Original change's description: > Browser Clipboard: Implements read and write SVG on the browser side > > Implements the SVG format for the clipboard on all blink platforms. The > implementation is very similar to HTML and plain text. Initially we will > put this feature behind a flag before it is exposed to the Web Platform. > > Design Doc: https://docs.google.com/document/d/1jq8QSCQRdNy99rnPusmW8is62c22PVuq-Sk-tMT2tRk/edit# > > Bug: 1110511 > Change-Id: I6d405a9704c431a31995bb50a17dfc309b49b74b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2313961 > Reviewed-by: Jarryd Goodman <jarrydg@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Reviewed-by: Darwin Huang <huangdarwin@chromium.org> > Reviewed-by: Scott Violet <sky@chromium.org> > Commit-Queue: Darwin Huang <huangdarwin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#800774} TBR=sky@chromium.org,dcheng@chromium.org,jarrydg@chromium.org,huangdarwin@chromium.org,dsleeps@google.com Change-Id: I0e550072698323d9a7e0754548821e2c765e3162 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1110511 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369419Reviewed-by:
Collin Baker <collinbaker@chromium.org> Commit-Queue: Collin Baker <collinbaker@chromium.org> Cr-Commit-Position: refs/heads/master@{#800797}
-
Chris Harrelson authored
This reverts commit 60b63538. Reason for revert: failing test was marked as failing in issue 1120106 for a different root cause. Original change's description: > Revert "Don't trigger main-thread overlay scrollbar animations if composited" > > This reverts commit f5cdac12. > > Reason for revert: This might be the cause of test failure virtual/threaded/external/wpt/feature-policy/experimental-features/vertical-scroll-disabled-scrollbar-tentative.html for https://ci.chromium.org/p/chromium/builders/ci/Win10%20Tests%20x64/48699 (and some builds after this). Revert to see if this fixed the test. > > Original change's description: > > Don't trigger main-thread overlay scrollbar animations if composited > > > > There are already hit tests and animations entirely in the impl thread > > for such scrollbars. > > > > Bug: 1116800 > > > > Change-Id: I2714258717c89e0ea41232eb86abe12570526d28 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364972 > > Commit-Queue: Chris Harrelson <chrishtr@chromium.org> > > Reviewed-by: David Bokan <bokan@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#800302} > > TBR=bokan@chromium.org,chrishtr@chromium.org > > Change-Id: Ided42e8aa3339e93f85d9f73f3737197864b493d > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1116800 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369076 > Reviewed-by: Maggie Cai <mxcai@chromium.org> > Commit-Queue: Maggie Cai <mxcai@chromium.org> > Cr-Commit-Position: refs/heads/master@{#800529} TBR=bokan@chromium.org,chrishtr@chromium.org,mxcai@chromium.org # Not skipping CQ checks because this is a reland. Bug: 1116800,1120106 Change-Id: I1f88a6c521618dc1d0b4d7e0d97c44acfc224214 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368490 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#800796}
-
Collin Baker authored
A captured tab going into fullscreen is treated differently: instead of taking up the entire screen, it only fills the browser's contents container. This is to allow tabs to enter fullscreen while being streamed without disrupting the user. For example, fullscreening a video during capture will cause the video to fill the tab contents rather than the whole screen. Thumbnail capture uses this same infrastructure but shouldn't affect any user-visible behavior. This CL prevents thumbnails from being captured on visible tabs. Ideally we'd be able to put a capture hold on a WebContents without affecting fullscreen behavior, but implementing that is a larger investment. Fixing this bug is a high priority. Bug: 1112607 Change-Id: I58cc53b2d881bf65cf2a9cb70d1a5d1592d02ad2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2354728 Commit-Queue: Collin Baker <collinbaker@chromium.org> Reviewed-by:
-
William Lin authored
Unbundled consent will allow GetAuthToken to make API calls to the server to automatically select an account to request permissions from. This account can be one specified by the extension or one that previously granted permissions to the extension. This can be useful for users so they don't need to choose or remember the "correct" account in the account chooser. This CL adds this feature of skipping the account chooser and puts it behind a flag to enable it. Bug: 1100535 Change-Id: I859784b5d3a533144c48802845b3317ce0e01935 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2363879 Commit-Queue: William Lin <williamlin@google.com> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Alex Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#800794}
-
Xiaohan Wang authored
This reverts commit 297abb5a. Reason for revert: Potential cause of crbug.com/1120848 Bug: 1120848 Original change's description: > [ios] noncq to run tests on 14.0 simulators > > ios-simulator-noncq to compile with xc12b5 as well. > Running iOS 14.0 simulators for our test suites on xc12b5. > > Change-Id: I7ca5f763174c296278401ae3563894a4f5210379 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367792 > Commit-Queue: Jeff Yoon <jeffyoon@chromium.org> > Reviewed-by: Garrett Beaty <gbeaty@chromium.org> > Cr-Commit-Position: refs/heads/master@{#800739} TBR=justincohen@chromium.org,lindsayw@chromium.org,gbeaty@chromium.org,jeffyoon@chromium.org Change-Id: I3ad22482336124cfff4edc4f521833cf54507dcb No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368324Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#800793}
-
Brian Malcolm authored
This is intended to replace the PrintingAPIExtensionsWhitelist policy. The old policy will still be used in the absence of the new one for at least a few releases of Chrome. BUG=chromium:1119444 Change-Id: Ia4b6d84cd8e716d3a2fb676df288dace50a3f4e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2365757 Commit-Queue: Brian Malcolm <bmalcolm@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Owen Min <zmin@chromium.org> Reviewed-by:
Nikita Podguzov <nikitapodguzov@chromium.org> Cr-Commit-Position: refs/heads/master@{#800792}
-
Ankit Kumar 🌪️ authored
Update PDFiumPage to use gfx::Rect instead of pp::Rect. Update callers to expect gfx::Rect as return value instead of pp::Rect. Bug: 1101101 Change-Id: Ibf7b57a007545d2f70ca1a4c72bb428d4b30a139 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367133 Commit-Queue: Ankit Kumar
🌪 ️ <ankk@microsoft.com> Reviewed-by:K. Moon <kmoon@chromium.org> Cr-Commit-Position: refs/heads/master@{#800791}
-
- 21 Aug, 2020 14 commits
-
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/9869f187b5ea..c86d8fb76ac3 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-mac-chromium 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 Tbr: perfetto-bugs@google.com Change-Id: I83fd0e4436d11a51c67a22fc0dca66ac0806b43d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369267Reviewed-by:
-
Evan Stade authored
Design doc: go/castinwl Bug: 1057100 Change-Id: Iabb0cdf3e949737e1cbc5b99f3f8ebeb6297a820 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2367696Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Takumi Fujimoto <takumif@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#800789}
-
Lukasz Anforowicz authored
After this CL, the blocklist emitted by the rewriter will include fields that may cause presence of global destructors (if CheckedPtr has a non-trivial destructor, such as required by BackupRefPtr implementation). Bug: 1069567 Change-Id: I8e547c813198220549875a64baa70af53a96f1fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368934 Auto-Submit: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
-
Juliet Levesque authored
Add logging tab and support to send logs created with PA_LOG macro to chrome://multidevice-internals page for display. Logs color coded by severity and displayed with timestamp and log file. Screenshot of display here: https://screenshot.googleplex.com/bc0d994b-4f58-4793-84f8-144a150f1496 Change-Id: Ic68aad70c400c5bdc55f7fa37fb20c36c4669233 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2366654 Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Josh Nohle <nohle@chromium.org> Cr-Commit-Position: refs/heads/master@{#800787}
-
Lukasz Anforowicz authored
This CL verifies that the browser process will reject attempts that a comprimised foo.com renderer can make in an attempt to show a bar.com URL in the Omnibox. Before this CL, there was already a SecurityExploitBrowserTest.DidCommitInvalidURL test: committing A) bar.com URL with B) origin set to foo.com, C) in a process locked to foo.com. This CL adds DidCommitInvalidURLWithOpaqueOrigin test which tests committing A) bar.com URL, B) with opaque origin derived from foo.com, C) in a process locked to foo.com. Fixed: 918565 Change-Id: I332349e889b83d1cd298050adbf9dca2f55125ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368288 Auto-Submit: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#800786}
-
Tommy Martino authored
The text on the Tab Switcher is currently misaligned in landscape mode on notched phones. This affects both the illustrated-empty-states on and off cases. Before and After Screenshots (googlers only, sorry): https://docs.google.com/presentation/d/1cZ151DN3Pywk2Ry-NSrMcM-Vf9v2Qbx35GBZFslSoTk/edit#slide=id.g91d5f68a07_0_6 Bug: 1116296 Change-Id: Icfa15e02abfba5c07b792adb168820755a596206 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369260Reviewed-by:
edchin <edchin@chromium.org> Commit-Queue: Tommy Martino <tmartino@chromium.org> Cr-Commit-Position: refs/heads/master@{#800785}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 17122f17. With Chromium commits locally applied on WPT: 3b705a74 "WPT: Service Worker: Add unexpected state check to wait_for_state()" 9f52c7e4 "COOP: add reporting to redirects" f3b221b0 "[COOP] access reporting: Extend report.body.type values." 5da0ed1e "Allow range requests to pass through a service worker" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=foolip@google.com No-Export: true Change-Id: I643c2298711d319cb9f6f3c7159269e602fb2eb1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368774Reviewed-by:
-
Anna Malova authored
Bug: 1120709 Change-Id: I12841b7bfd5d9a987ec6c9e13016e8109aef447f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368876 Commit-Queue: Anna Malova <amalova@chromium.org> Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#800783}
-
Sundoo Kim authored
This patch fixes typo to replace "liasons" with "liaisons" in the comment. Change-Id: I94edf4ccee8eb75287c1b92f91c4502117ee00b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2362054Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#800782}
-
Noah Rose Ledesma authored
This is a reland of 7d2d8cc1 Original change's description: > GMC: Redesign audio picker UI > > Implement the new UI for the audio device picker in global media > controls. As part of this change, the MediaNotificationBackground will > draw a gradient between the device picker and notification artwork if > the picker is availabile. > > Screenshots of this updated UI will be posted on the crbug. > > Change-Id: Ic7186c770102663b08e85b56eeddfbfa13833c3c > Bug: 1117160 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2358512 > Commit-Queue: Noah Rose Ledesma <noahrose@google.com> > Reviewed-by: Tommy Steimel <steimel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#800243} Bug: 1117160 Change-Id: I3adf5e58dcb8cd77758abd75aa5d497ab0f50c2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368504Reviewed-by:
Tommy Steimel <steimel@chromium.org> Commit-Queue: Noah Rose Ledesma <noahrose@google.com> Cr-Commit-Position: refs/heads/master@{#800781}
-
Yicheng Li authored
When the key type is set to KEY_TYPE_FINGERPRINT, CheckKey will wait for a fingerprint scan result from biometrics daemon and report the result. (See http://crrev/c/1941087 for more details.) This change enables ExtendedAuthenticator to request fingerprint scan result by calling CheckKey(KEY_TYPE_FINGERPRINT). With this change, clients of ExtendedAuthenticator can perform fingerprint auth without having to talk to biometrics daemon directly. Bug: b:156258540, b:144861739 Change-Id: I923a53afa283dd4794ab8af239da894800a7a3f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2364797 Commit-Queue: Yicheng Li <yichengli@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#800780}
-
Kyle Horimoto authored
This CL removes PhoneHubManager::Get(), and instead provides PhoneHubManager to //ash via new function SystemTray::SetPhoneHubManager(). This function is left as a TODO for now; a follow-up CL will use the provided PhoneHubManager pointer to render the UI in //ash. Bug: 1106937 Change-Id: I4a69d700d4e086d3e59ceaaf5ae1a364824a3633 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368485Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#800779}
-
Xiaohan Wang authored
This reverts commit 40efa001. Reason for revert: crbug.com/1120811 Original change's description: > Inhibit hang watching during views::DesktopDragDropClientWin::StartDragAndDrop > > The nested loop process the native events but not the tasks which looks > like a hang if not inhibited. > > Bug: 806174, 1120381 > Change-Id: Ie298b16a4e0f8cdd6c4ba398d71a7fee02b9d46d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2369253 > Reviewed-by: Robert Liao <robliao@chromium.org> > Commit-Queue: Robert Liao <robliao@chromium.org> > Commit-Queue: Oliver Li <olivierli@chromium.org> > Cr-Commit-Position: refs/heads/master@{#800626} TBR=robliao@chromium.org,olivierli@chromium.org Change-Id: I74f4dcda0ef7dcdd47bcd6e8426c4776575c04d1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 806174 Bug: 1120381 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2368336Reviewed-by:
-
Harry Cutts authored
Uses the new gesture type added in https://crrev.com/c/2301084. See (Google-only) go/cros-wheel-tick-counts for the design doc. Bug: 1077644 Test: run unit tests Change-Id: I287740706c77114ac08b84376186096b81138e7e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2337272 Commit-Queue: Harry Cutts <hcutts@chromium.org> Reviewed-by:
Michael Spang <spang@chromium.org> Reviewed-by:
Sean O'Brien <seobrien@chromium.org> Cr-Commit-Position: refs/heads/master@{#800777}
-
