- 02 Dec, 2019 40 commits
-
-
Daniel Murphy authored
See go/chrome-storage-dashboard R=mpearson@chromium.org Bug: 1021738 Change-Id: Id255b4124a426ac67a99e710bbb951d3e82dda68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1927001Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Daniel Murphy <dmurph@chromium.org> Auto-Submit: Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#720565}
-
Daniel Murphy authored
R=rbpotter@chromium.org Bug: 1008122 Change-Id: I32be6eb85be6ae221f477cedf3a83dd9a4562d68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1945922Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#720564}
-
Istiaque Ahmed authored
We switched to using OnMessage::kEventName at some point but forgot to remove the unused definition. Bug: None Change-Id: I6c03968487bcf615564d9796736f5ce81f03ba9f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1930686 Auto-Submit: Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
David Bertoni <dbertoni@chromium.org> Commit-Queue: Istiaque Ahmed <lazyboy@chromium.org> Cr-Commit-Position: refs/heads/master@{#720563}
-
Elly Fong-Jones authored
This test is already disabled as flaky on Windows & Linux; it appears to flakily time out on Mac (quite often actually) as well. Bug: 1014229, 1028239 Change-Id: I278543f670e5723ecb0428d09863367bbf8c772e Tbr: thestig@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946824Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#720562}
-
Abhijeet Kandalkar authored
This CL has two goals, 1. Use To<SVGImageElement> and DynamicTo<SVGImageElement> as new downcast helper 2. Use IsA<SVGImageElement>(element) in place of IsSVGImageElement(element) Bug: 891908 Change-Id: I81b15e047f79e18df4bc2816caecaa92990f3b48 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1936607 Commit-Queue: Abhijeet | Igalia <abhijeet@igalia.com> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#720561}
-
Nicolás Peña Moreno authored
The ClusterFuzz testcase implies that the DCHECK is incorrect, so replace with an actual boundary check. Bug: 1027948 Change-Id: I4f9a26fbe37a76f1786554dfa942285723e6e3fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1945360 Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Reviewed-by:
Steve Kobes <skobes@chromium.org> Cr-Commit-Position: refs/heads/master@{#720560}
-
Yun Liu authored
Change-Id: I136d3d955059e9bd602bbedac7e5c6cd8b598424 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1918719Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Yun Liu <yliuyliu@google.com> Cr-Commit-Position: refs/heads/master@{#720559}
-
Alison Maher authored
Set the NewSystemColors flag to stable and update test expectations. Spec: https://drafts.csswg.org/css-color-4/#css-system-colors Intent-to-ship: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/zv6k2X7Wr1M Bug: 1015968 Change-Id: I754cf7daefe4a61cc76c4b074aa0641c56c92904 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1931268Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Alison Maher <almaher@microsoft.com> Cr-Commit-Position: refs/heads/master@{#720558}
-
Peter Kasting authored
This should be non-null because this function is only called during painting, when there should always be a widget. The behavioral effect is that the location bar border color should better reflect the theme toolbar color for popup windows. I believe when this code was originally written, popup windows were not themed, so this wasn't necessary. Bug: none Change-Id: Ie4b33531cae28ce9a72c7571ebb579345fef83d0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1941189 Auto-Submit: Peter Kasting <pkasting@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#720557}
-
David Benjamin authored
This dates to http://codereview.chromium.org/2870062 as a workaround for std::map::operator[] creating the key if missing. Use the helper function. Bug: none Change-Id: I92b21fd0f7834f8aee5560d080971fa576cc9d88 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1941007Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Reviewed-by:
John Chen <johnchen@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Commit-Queue: David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#720556}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/d523ce04e70a..b97a24a009a1 git log d523ce04e70a..b97a24a009a1 --date=short --first-parent --format='%ad %ae %s' 2019-12-02 reed@google.com Revert "fix sense of ifdef for deprecated method" 2019-12-02 reed@google.com fix sense of ifdef for deprecated method Created with: gclient setdep -r src/third_party/skia@b97a24a009a1 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 bsalomon@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: bsalomon@google.com Change-Id: Idb6c806872695535ed31e697afe9dee6fa938a36 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946434Reviewed-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@{#720555}
-
Ioana Pandele authored
The bold spans are no longer used for any of the dialogs. Bug: 1029828 Change-Id: I4dbf79c2900a629e1e3424151e4e2a0bd5d7a583 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1882317Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Commit-Queue: Ioana Pandele <ioanap@chromium.org> Cr-Commit-Position: refs/heads/master@{#720554}
-
Varun Khaneja authored
Change-Id: I5901b42fce94a3ec71f67ed101dcc5053cdbc98f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944946Reviewed-by:
Varun Khaneja <vakh@chromium.org> Reviewed-by:
Bettina Dea <bdea@chromium.org> Auto-Submit: Varun Khaneja <vakh@chromium.org> Commit-Queue: Bettina Dea <bdea@chromium.org> Cr-Commit-Position: refs/heads/master@{#720553}
-
Nikita Podguzov authored
* Change structure of the policy information propagated there. This allows to add further printing policies easier. * Fix printing.print_header_footer pref description. * Restructure Print Preview policy tests. Bug: 994786 Change-Id: Ie70238205980bbf1eb81ae66c0be78812f798b42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1928787 Commit-Queue: Nikita Podguzov <nikitapodguzov@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#720552}
-
James Cook authored
As part of the SplitSettingsSync project we want to use the browser's first-run sync opt-in flow on Chrome OS. We're trying to align browser sync with Windows/Mac/Linux. PRD in the bug. This CL adds SyncConfirmationUI, SyncConfirmationHandler, an existing unit test and associated dependencies to the build. It also fixes some minor style guide issues in the code. Chrome OS does not use the DICE identity model (it has its own account manager and behaves more like Android/iOS). Some of the dependencies therefore need BUILDFLAG(ENABLE_DICE_SUPPORT) checks. There are no user-visible changes yet. Bug: 1013466 Test: unit_tests Change-Id: I3b1a5c968279ffe48a32e9f6765bc6f6fe975e2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917742Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#720551}
-
Carlos Caballero authored
MessageLoop is going away soon. This CL was uploaded by git cl split. R=dewittj@chromium.org Bug: 891670 Change-Id: I23c7baa0f9429ba704a6ebfb356542eeb0cc8ce7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1943185 Auto-Submit: Carlos Caballero <carlscab@google.com> Reviewed-by:
Justin DeWitt <dewittj@chromium.org> Commit-Queue: Justin DeWitt <dewittj@chromium.org> Cr-Commit-Position: refs/heads/master@{#720550}
-
Elly Fong-Jones authored
The specific notion of overscroll this test is checking for is only implemented on Aura platforms. Bug: 1028239 Change-Id: Ic40d5d504e463517cd28fc28b3a6d36a7e5fb8a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946683Reviewed-by:
Leonard Grey <lgrey@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#720549}
-
Ben Kelly authored
Intent-to-experiment thread with LGTM: https://groups.google.com/a/chromium.org/d/msg/blink-dev/exoGLTG8m-c/5EUdD5zYBQAJ Bug: 1007272 Change-Id: If1ced44cff78c8d957fe5792e47fbe38264b5048 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1940137Reviewed-by:
Yoav Weiss <yoavweiss@chromium.org> Commit-Queue: Ben Kelly <wanderview@chromium.org> Cr-Commit-Position: refs/heads/master@{#720548}
-
behdad authored
. README added for representative perf tests. . Update pixel-wrangling instructions to include looking at repr_perf_test results. . Linked the pixel-wrangling doc to the representative perf test's for more detail. Bug: chromium:1000837 Change-Id: Iefe03f7c722ccc9c8d72cf3e5ada7816e240d1cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1937848Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Behdad Bakhshinategh <behdadb@chromium.org> Cr-Commit-Position: refs/heads/master@{#720547}
-
Ilia Samsonov authored
googletest will no longer support empty prefix for parameterized INSTANTIATE_TEST_SUITE_P. Adding "All" prefix to existing empty prefix test suites. This CL was uploaded by git cl split. R=mpearson@chromium.org Bug: 1023677 Change-Id: I20d5e41c5adc458b0528d668157687d5bf9e758a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1925744 Auto-Submit: Ilia Samsonov <isamsonov@google.com> Reviewed-by:
-
Alex Deymo authored
These metrics were already deprecated in M79. Bug: 1024570 Test: blink_platform_unittests still pass Change-Id: Ibf7dbfe93616ccb0ba151841b524a0256ec343bb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1917521Reviewed-by:
Stephen Chenney <schenney@chromium.org> Commit-Queue: Alex Deymo <deymo@google.com> Cr-Commit-Position: refs/heads/master@{#720545}
-
Mike Reed authored
Change-Id: Ia2e059473343a5672df601ff3c56b1fb3e6ebf17 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946684 Commit-Queue: Mike Reed <reed@google.com> Commit-Queue: Florin Malita <fmalita@chromium.org> Reviewed-by:
Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#720544}
-
Friedrich Horschig authored
This assert prevents the usage of VectorDrawables in TintedDrawable. Before this CL, TintedDrawable would fail silently and return an empty image (which leads to regression like the linked bug). Bug: 1028925 Change-Id: I65ff97599c8dba5b90a07087335a63a39903c2d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946332 Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#720543}
-
Alexander Timin authored
Address post-commit comments. TBR=haraken@chromium.org CC=hajimehoshi@chromium.org,kouhei@chromium.org BUG=1004676 Change-Id: Ib017771ca40c2c5a4fabe1057bcae81e88487059 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946192Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#720542}
-
Marc Treib authored
This reverts commit bd94f94a. Reason for reland: Fixed the crash (by handling null SyncService). SyncService is null in incognito, so we need to handle that case. The reland also adds feature checks (which should have been there the first time around), and while we're here, makes the handling of empty account IDs more explicit. Original change's description: > Revert "Collect user opt-in before enabling account-storage for passwords" > > This reverts commit c3c37466. > > Reason for revert: canary top crasher; see https://crbug.com/1029631. > > Original change's description: > > Collect user opt-in before enabling account-storage for passwords > > > > This CL adds a pref that stores which accounts have opted in to use the > > Google account storage for passwords. While this pref is false, > > passwords can not be saved to the account, and passwords in the account > > are not downloaded. > > It also adds an entry to the password autofill dropdown that lets the > > user set the opt-in. > > > > Bug: 1024332 > > Change-Id: Iac552c310d0e9f19ee2bb1893e217e60b1114d31 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645294 > > Commit-Queue: Marc Treib <treib@chromium.org> > > Reviewed-by: Vadym Doroshenko <dvadym@chromium.org> > > Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> > > Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#720206} > > TBR=sdefresne@chromium.org,treib@chromium.org,dvadym@chromium.org,jzw@chromium.org,mamir@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 1024332,1029631 > Change-Id: Ib859957d27a898652e001515652cde25a30ed1b3 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944497 > Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> > Reviewed-by: Greg Thompson <grt@chromium.org> > Commit-Queue: Marc Treib <treib@chromium.org> > Cr-Commit-Position: refs/heads/master@{#720424} Bug: 1024332 Change-Id: I57e9ae4c636b9ad6e7d6da83c5eeaf0c7283d970 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946370 Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Cr-Commit-Position: refs/heads/master@{#720541}
-
Joe Downing authored
Since I am removing the NaCl and WebApp references in the remoting directory, I figured that we may as well remove this too. Change-Id: Id35f11b8a1ddcaf8d063ba61379b507da818ab58 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1934647 Commit-Queue: Joe Downing <joedow@chromium.org> Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Cr-Commit-Position: refs/heads/master@{#720540}
-
Fredrik Söderquist authored
When an arc segment was decomposed into more than one cubic bezier segment, some "residue" (the length of the first part of the cubic that was "flat enough") would accumulate from any segments that followed the one that contained the point. This would make the computed |offset| incorrect. Switch to checking, for each bezier segment, if the we've passed the point (length) that we're querying, and skip adding anything for that bezier segment. Bug: 719516 Change-Id: I0c54e1a07be5fb5edddaded542df0ffd5b0a087c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1939979 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
-
Abhijeet Kandalkar authored
This CL has goal to use IsA<SVGStopElement>(element) in place of IsSVGStopElement(element) Bug: 891908 Change-Id: Iba08d8b45ee7440957528beccccbea6ad3d4378f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1936621Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Abhijeet | Igalia <abhijeet@igalia.com> Cr-Commit-Position: refs/heads/master@{#720538}
-
Sahel Sharify authored
On android web-based payment apps with missing name or icon are not pre-selectable. This cl is changing the desktop implementation to match Androud. Bug: 1029373 Change-Id: Ic496ba7d9e9884615e58c4c8fc00882be51417a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1943267 Commit-Queue: Sahel Sharify <sahel@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#720537}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/4b5276f1ad99..b2efa52611c0 git log 4b5276f1ad99..b2efa52611c0 --date=short --first-parent --format='%ad %ae %s' 2019-12-02 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update Devtools DEPS. 2019-12-02 tvanderlippe@chromium.org Reland "Remove "Remote Devices" panel" 2019-12-02 aerotwist@chromium.org Fixes string interpolation in exclusive changes dir 2019-12-02 aerotwist@chromium.org Updates node_modules Created with: gclient setdep -r src/third_party/devtools-frontend/src@b2efa52611c0 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I39eda57accfdfc059da8545a196f634afc178bcc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946428Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/ded19c4711b5..37c0baa6b7b4 Created with: gclient setdep -r src-internal@37c0baa6b7b4 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:None Tbr: jbudorick@google.com Change-Id: I39483948cb45461b94a842aa145fda633a2f84cf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1945358Reviewed-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@{#720535}
-
Fredrik Söderquist authored
This is more of a post-processing effect, and since we already handle 'auto' vs. <angle> in LayoutSVGResourceMarker, we might as well move the handling of the other 'auto' value there. Bug: 674797, 633012 Change-Id: I8b06800201b923167cfac5a03f8cd78335b02f6a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942334Reviewed-by:
-
Min Qin authored
This reverts commit bac094f7. The CHECK captures the crash, no longer needed Original change's description: > Convert some DCHECK to CHECK to find cases that download file is not > properly deleted > > > BUG=1028666 > > Change-Id: I59b60a897a5f2529d4e61d8632aebe421dbc71b3 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1933738 > Reviewed-by: Xing Liu <xingliu@chromium.org> > Commit-Queue: Min Qin <qinmin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#719294} TBR=qinmin@chromium.org,xingliu@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1028666 Change-Id: Ia0cdac36ec67ad41a669b55bf32343a1dd18afba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946430Reviewed-by:
Min Qin <qinmin@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#720533}
-
Joshua Pawlicki authored
Bug: 1029787 Change-Id: I1f30a649fcaa7deeb62a1f0b168c89e7555dad5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1946146 Auto-Submit: Joshua Pawlicki <waffles@chromium.org> Commit-Queue: Sorin Jianu <sorin@chromium.org> Reviewed-by:
Sorin Jianu <sorin@chromium.org> Cr-Commit-Position: refs/heads/master@{#720532}
-
Yoav Weiss authored
CSS images should take Content-DPR into account when calculating their intrinsic dimensions. They currently don't. This CL fixes that. Bug: 1029313 Change-Id: Iafb2e5d88c1a0bbf0a13b00ddb46ad643f38c8d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944496 Commit-Queue: Yoav Weiss <yoavweiss@chromium.org> Reviewed-by:
-
Sylvain Defresne authored
The font is no longer used by Chrome on iOS nor any of its dependency from the internal repository so stop initializing the font loader and remove the font data from the application binary. This will reduce the app bundle size by ~ 1.1 Mo (size of the unpacked fonts data) without any impact on the application since UI lead has confirmed that the font is not used on iOS. Bug: 1017165 Change-Id: Ia07c168be58ef30f75bba0f902c0d0e0b8631a63 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944477 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#720530}
-
Nicolás Peña Moreno authored
Bug: 1016815 Change-Id: I6c3ab2e1551de2bea9eaacd211c976bf64da66bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1942968Reviewed-by:
-
Julie Jeongeun Kim authored
This CL cleans up |bindings_| from FakePrivilegedHostDeviceSetter since it's not used. Bug: 955171 Change-Id: I35ab1100e8732aba2fe904ebb4ac7dcc40c0a0de Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1939130 Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Cr-Commit-Position: refs/heads/master@{#720528}
-
Matt Menke authored
We're working on splitting up network resources by NIK to prevent tracking users across browsing contexts for different first parties. Classes that perform network activity on behalf of a RenderFrame need that frame's NetworkIsolationKey. Also update consumers to use the method. Bug: 997049 Change-Id: I685335d1ec84f858276f89ab6ebb1b114c9bcc4f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1938029Reviewed-by:
Shivani Sharma <shivanisha@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Reviewed-by:
Alex Ilin <alexilin@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#720527}
-
Vadym Doroshenko authored
Bug: 959776 Change-Id: I58863e2b625bd55242974283646bfff44bb04fc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1944865 Commit-Queue: Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#720526}
-
