- 17 Jul, 2020 40 commits
-
-
Evan Stade authored
It doesn't appear to have an effect outside of instrumented tests. Bug: none Change-Id: If90c21806a5aff8b7380d06cfccfe0d8ce70a555 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302731Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#789576}
-
Wei-Yin Chen (陳威尹) authored
Before this CL, this script builds DOM distiller, pushes the compiled artifacts to the dom-distiller/dist repo, and generates a local commit for rolling. The local commit needs to be uploaded as a CL. In order to avoid pushing without reviewing, we want to generate a CL in the dom-distiller/dist repo instead of directly pushing to it. When reviewing the CL in the dom-distiller/dist repo, the reviewer needs to run update_domdistiller_js.sh in validation mode to make sure the artifacts in the CL are identical to local output. Bug: None Change-Id: Ia29c1686106c3516f341e4d826aca69842ac5668 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304338Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#789575}
-
Kevin McNee authored
We already reject activation for error pages, so we now treat a renderer crash the same. This also resolves a browser crash when attempting to activate in this state. Bug: 1106490 Change-Id: I569a24e27ce0128e6204c539a8e7c41041b3ae81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303562 Commit-Queue: Kevin McNee <mcnee@chromium.org> Reviewed-by:
Lucas Gadani <lfg@chromium.org> Cr-Commit-Position: refs/heads/master@{#789574}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/89b37f84e65c..2bcf0b808628 2020-07-17 changhaohan@chromium.org Add e2e test for computed tab's "Show all" functionality 2020-07-17 sigurds@chromium.org Link to related panels from the frame details view 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:1093247,chromium:1106251 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I72e7aea3fdeb6be253cf53bde8b1960e2919a231 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304795Reviewed-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@{#789573}
-
David Maunder authored
Bug: 1101760 Change-Id: I0c7c11662aac96d7e0b1ccda09e98d73472a852f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2299853Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Commit-Queue: David Maunder <davidjm@chromium.org> Cr-Commit-Position: refs/heads/master@{#789572}
-
W. James MacLean authored
Try to reduce usage of GURL::is_valid for ProcessLock. Bug: 1067389 Change-Id: I7a88814bb95bf8aa7e28524c40499234b6c95ef4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2300769 Commit-Queue: James MacLean <wjmaclean@chromium.org> Reviewed-by:
Aaron Colwell <acolwell@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#789571}
-
Vasilii Sukhanov authored
The metrics was off by 1 in the dashboard. Bug: 1049200 Change-Id: Iec92bb88e3a745e9a0d68269042e55c3cabf2ff0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302996 Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#789570}
-
Wei-Yin Chen (陳威尹) authored
"None" would be ignored, and add space after ",". Bug: None Change-Id: I85167f2e9ba0cea7affd754825fd34e817568bd5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304337Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#789569}
-
Tao Bai authored
The event sequence was changed by the previous patch https://chromium-review.googlesource.com/c/chromium/src/+/2296487 The AUTOFILL_VIEW_EXITED is only expected on Android o platform, and will be removed for others. This issue wasn't uncoverred because we don't have O trybot by default. Bug: 1106457 Change-Id: Id838f7085ae49fd8d968929a7a08d7918c818770 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302743 Commit-Queue: Dominic Battré <battre@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#789568}
-
Victor Hugo Vianna Silva authored
Implementation: http://go/chrome-prsc/44462b2636bd411cd843d38282745059c3ca56ca Mock: https://screenshot.googleplex.com/pGV0iMTQBxw Bug: 1102294 Change-Id: I94a34dd5576bbeb1afd467f3160292e33cd4b7d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2290734 Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#789567}
-
Matt Reynolds authored
media::VideoFrameCompositor uses two base::RetainingOneShotTimers to handle Start/StopForceBeginFrames and background rendering. base::RepeatingCallback is correct because the timer is meant to retain and reuse the callback after a reset. BUG=1102651 Change-Id: I7c028cc08ba0d07c1f7e85518f9346a279313469 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2284148 Auto-Submit: Matt Reynolds <mattreynolds@chromium.org> Reviewed-by:
Chrome Cunningham <chcunningham@chromium.org> Commit-Queue: Chrome Cunningham <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#789566}
-
K. Moon authored
Instead of directly creating and binding pp::Graphics2D in PaintManager, delegate these operations to PaintManager::Client. This provides several benefits: 1. Allows the PaintManager::Client to return a different pp::Graphics2D implementation in the future. This will be useful especially once we replace pp::Graphics2D with a Pepper-free API. 2. Eliminates the need for PaintManager to interact with pp::Instance. Bug: 1099020 Change-Id: I46ce18cfb3ced088afdf6581fc335aa56b8be195 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303989 Commit-Queue: K. Moon <kmoon@chromium.org> Reviewed-by:
Daniel Hosseinian <dhoss@chromium.org> Reviewed-by:
Henrique Nakashima <hnakashima@chromium.org> Cr-Commit-Position: refs/heads/master@{#789565}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-aemu-chromium-autoroll Please CC chonggu@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: chonggu@google.com Change-Id: Ib9c0b7ec8b7639a006ac2c6a80f93817fd53dc22 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304799Reviewed-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@{#789564}
-
Andrew Grieve authored
Bug: 1029357 Change-Id: I3989927be656c47f70740317d9e7b81e86c83bf9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304156 Auto-Submit: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Paul Jensen <pauljensen@chromium.org> Commit-Queue: Paul Jensen <pauljensen@chromium.org> Cr-Commit-Position: refs/heads/master@{#789563}
-
Robert Ogden authored
This has the potential to leak user information during an isolated prerender and isn't otherwise beneficial in the context of a prerender. Bug: 1103392 Change-Id: Ic1806bcdf3476bc2c864f91028125d380b0b1be1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2300805Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#789562}
-
Mirko Bonadei authored
Change Log: https://chromium.googlesource.com/external/github.com/abseil/abseil-cpp/+log/23f1f9cf6d..f624790b7f Full diff: https://chromium.googlesource.com/external/github.com/abseil/abseil-cpp/+/23f1f9cf6d..f624790b7f Bug: None Change-Id: I877dfe42561531e71ae3bf7a69cd5566bfaad300 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303397Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Mirko Bonadei <mbonadei@chromium.org> Cr-Commit-Position: refs/heads/master@{#789561}
-
Yuke Liao authored
Remove unused ash-chrome versions to avoid consuming too much disk space. Bug: 1104318 Change-Id: Ie5b74a271389dcb3b34fa1872f5d5b31b1d1af8d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303975 Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Reviewed-by:
Sven Zheng <svenzheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#789560}
-
Xianzhu Wang authored
This is a part of the work of removing DisplayItemClient::VisualRect() which is an obstacle of integrating FragmentData with LayoutNGBlockFragments. Bug: 1104064 Change-Id: I47ef92d3a30fa178006e3c696b110de75cca7543 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304658Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#789559}
-
Anders Hartvoll Ruud authored
The CSSInterpolationEnvironment constructor that accepts a CSSVariableResolver should only be used when the CSSCascade flag is disabled. Change-Id: I7a7b984c16dcc684e5d76d412a0e1565d7efb1c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302113 Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org> Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Robert Flack <flackr@chromium.org> Cr-Commit-Position: refs/heads/master@{#789558}
-
Charlie Hu authored
Bug: 1106206 Change-Id: Id49eace861305bb57819a34824c06a7583f16518 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304769Reviewed-by:
Ian Clelland <iclelland@chromium.org> Commit-Queue: Charlie Hu <chenleihu@google.com> Cr-Commit-Position: refs/heads/master@{#789557}
-
Will Harris authored
The issue previously manifested with the MSVC linker, which is no longer in use. BUG=812421,584575 Change-Id: I92f5703408c64bf5fb2962507b26eab7f1ace013 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304137 Commit-Queue: Will Harris <wfh@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Bruce Dawson <brucedawson@chromium.org> Cr-Commit-Position: refs/heads/master@{#789556}
-
Mei Liang authored
This CL adds user actions and histograms for Grouping tabs suggestion. In addition, it modifies the metric names for Closing tabs suggestion to yield cleaner code. The new user actions and histograms are: * TabsSuggestions.Considered.Accepted.Grouping * TabsSuggestions.Considered.Dismissed.Grouping * TabsSuggestions.NotConsidered.Grouping * Tabs.Suggestions.NumSuggestionsChanged.Grouping The modified user actions and histograms are: * TabsSuggestions.Close.Accepted renames to TabsSuggestions.Considered.Accepted.Closing * TabsSuggestions.Close.Dismissed renames to TabsSuggestions.Considered.Dismissed.Closing * TabsSuggestions.Close.SuggestionsReview.Accepted is removed. * TabsSuggestions.Close.SuggestionsReview.Dismissed renames to TabsSuggestions.NotConsidered.Closing * Tabs.Suggestions.Close.NumSuggestionsChanged renames to Tabs.Suggestions.NumSuggestionsChanged.Closing Bug: 1076538 Change-Id: I9429c0d2923b52f2c708872d8f5fcf8cb41b1ca5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2295800 Commit-Queue: Mei Liang <meiliang@chromium.org> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Cr-Commit-Position: refs/heads/master@{#789555}
-
oysteine authored
This reverts commit ec645d52. Reason for revert: crbug.com/1106867 Original change's description: > [Clank SSM]: Enable stack sampling in android browsertests. > > android_browsertests must depend on libunwindstack directly instead of > through DFM, since loading of a partitioned library is not supported in > an APK. > (internal discussion) > https://groups.google.com/a/google.com/g/clank-components/c/ktt285Gtax4/m/yRy8qm8LAQAJ > > This CL also re-enables tracing use of StackSampingProfiler, which > was accidentally disabled in https://crrev.com/784147? > > Bug: 1004855 > Change-Id: I5967bf3815008843ba8a77e926977e39a4ccfbd8 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2293071 > Reviewed-by: Wez <wez@chromium.org> > Reviewed-by: Mike Wittman <wittman@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@google.com> > Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> > Cr-Commit-Position: refs/heads/master@{#789530} TBR=wez@chromium.org,wittman@chromium.org,dpranke@google.com,etiennep@chromium.org Change-Id: I61309dcf5ceb5591dbf9f40e506ee25ee2ac475c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1004855 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304932Reviewed-by:
oysteine <oysteine@chromium.org> Commit-Queue: oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#789554}
-
Dana Fried authored
See attached bug for specific cases addressed. Needs testing on MacOS. Bug: 1106506 Change-Id: I65cf6236d618b1cf8ada7b8b4ef07dbd7ed74e12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303775Reviewed-by:
Caroline Rising <corising@chromium.org> Commit-Queue: Dana Fried <dfried@chromium.org> Cr-Commit-Position: refs/heads/master@{#789553}
-
Sangwoo Ko authored
In the previous CL, a code calling ExtensionToolbarButton::UpdateIcon() from ExtensionToolbarContainer was removed. (https://chromium-review.googlesource.com/c/chromium/src/+/2253600/29/chrome/browser/ui/views/extensions/extensions_toolbar_container.cc) However, ExtensionToolbarButton should update icon color when WebAppFrameToolbar overrides icon color for ExtensionToolbarButton. (https://chromium.googlesource.com/chromium/src/+/master/chrome/browser/ui/views/web_apps/web_app_frame_toolbar_view.cc#471) Otherwise, ExtensionToolbarButton ignores the overridden color. When WebAppFrameToolbar overrides icon color, ExtensionToolbarContainer::UpdateAllIcon() we be called. Thus, we should call ExtensionToolbarButton::UpdateIcon() in it. Bug: 1106698, 1048901 Change-Id: I2082a16d18b312cd5a578f2c84b709e60cee6c5b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303418Reviewed-by:
Peter Kasting <pkasting@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#789552}
-
Wen-Chien Wang authored
Fixed the bug that when user changes the position of an app in folder, the change won't update the syncable service so that the user will lose their changes if they logout. R=tbarzic@chromium.org Bug: 1041633 Change-Id: I14be6e26f3fc7b4532aad2f4b17199d7bbaabce1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2293470 Commit-Queue: Wen-Chien Wang <wcwang@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Jenny Zhang <jennyz@chromium.org> Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#789551}
-
yjliu authored
Ideally, frame submission is expected to be at full rate of 60fps. However, in many scenarios, this is not necessary. For example, when a window is (partially) occluded by another window, or when a window is behind a backdrop filter that blurs its content, or when a window is going through an animation (e.g. going in or out of overview mode). In such cases, we'd like to submit frames to a lower fps to improve performance - this is where the frame throttling comes in. This changelist only deals with browser windows, and when UI is going into and out of overview mode. We'd like to throttle the BeginFrames (ShouldSendBeginFrames), so that all the browser windows in overview mode will update at a reasonable lower fps. Here is the design doc if you'd like to inspect more. https://docs.google.com/document/d/1kvCIuA_xsh1VAsFiC5i1paAtjkGm2z-hFhFXWYfL7Sg/edit?usp=sharing Bug: 1095670 Change-Id: I8c9697fa5c0886032525395355d3c8ce76e439cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2246716Reviewed-by:
kylechar <kylechar@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Commit-Queue: Jun Liu <yjliu@chromium.org> Auto-Submit: Jun Liu <yjliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#789550}
-
Xing Liu authored
If the user scheduled a download that starts right away, or the network condition met right away. Download schedule struct should be cleared, or the download will be shown in the "scheduled for later" section, but it's actually completed. Bug: 1106208,1078454 Change-Id: I70526221839d16c96a31b1409c28e5227cc9daa2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2301379Reviewed-by:
Min Qin <qinmin@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#789549}
-
Mohamed Amir Yosef authored
Screenshots (in Arabic UI) Before: https://screenshot.googleplex.com/upvG78K5yqY.png After: https://screenshot.googleplex.com/TDMh0T0XzbR.png Bug: 1044038 Change-Id: Ibe5f960629cd506762ac14af44a7818f238215f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303715 Commit-Queue: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#789548}
-
Justin DeWitt authored
Bug: 1106425, b/161142289 Change-Id: If3c89644289b6eb1d5b61f6680f03bf95e3b6888 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303172Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Carlos Knippschild <carlosk@chromium.org> Commit-Queue: Justin DeWitt <dewittj@chromium.org> Cr-Commit-Position: refs/heads/master@{#789547}
-
Finnur Thorarinsson authored
These continue to be important, especially with video support being added to the PhotoPicker. Bug: 1105074 Change-Id: If3bb8d759f7a18cf7fc2fba838927517b15f094b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2299628 Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Auto-Submit: Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#789546}
-
Rainhard Findling authored
Bug: 1087263 Change-Id: I4f94508c5940858a63321812569d75c4064c29bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302191Reviewed-by:
Martin Šrámek <msramek@chromium.org> Commit-Queue: Rainhard Findling <rainhard@chromium.org> Cr-Commit-Position: refs/heads/master@{#789545}
-
Torne (Richard Coles) authored
Include build-tools 30.0.1 which is also needed by the R SDK. The Android R SDK sources aren't available yet as R has not been released to AOSP. Disable updating the sources package for now. Bug: 1106471 Change-Id: I2ec942069f3ca95c694be0cbf3510700afcd1b19 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304533Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Garrett Beaty <gbeaty@chromium.org> Commit-Queue: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#789544}
-
Jose Dapena Paz authored
Bug: 819294 Change-Id: I79cb83515d197106c70345526a2a17177e8ca9e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303437Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: José Dapena Paz <jdapena@igalia.com> Cr-Commit-Position: refs/heads/master@{#789543}
-
chromium-autoroll authored
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-win-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: Ia35e6f02796ec299fde4e4ae6384cb1bcd7e2090 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2304153Reviewed-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@{#789542}
-
Xing Liu authored
The download later dialog needs to be scrollable. In phone landscape mode, not all the view is shown. Bug: 1106363,1078454 Change-Id: Iae8d649b7259fee0a8800aa71e7a14cd759d49e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2301039Reviewed-by:
Min Qin <qinmin@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Commit-Queue: Xing Liu <xingliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#789541}
-
Clark DuVall authored
Used sed to replace 's/base::Bind(/base::BindOnce(/g' then fixed compile errors that needed BindRepeating. Bug: 1007660 Change-Id: Ib7acf35a67fdfea520f9ebbe1f18543801a30da7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2290077Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#789540}
-
Wan-Teh Chang authored
Since third_party/libavif is only used by third_party/blink/renderer/platform/image-decoders, it makes more sense for third_party/libavif to be in the same component as third_party/blink/renderer/platform/image-decoders. Change-Id: I92c464f662065267311c412e18279e76086ac364 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302941Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Wan-Teh Chang <wtc@google.com> Cr-Commit-Position: refs/heads/master@{#789539}
-
Torne (Richard Coles) authored
Just use the general pool at best-effort priority for logging command line and field trials during startup, since it doesn't need to be on the UI thread. Change-Id: I73dfb14ae67bcc953fa7c90b5840354d5b4a84b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2302946Reviewed-by:
Nate Fischer <ntfschr@chromium.org> Commit-Queue: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#789538}
-
Sophey Dong authored
Bug: 1105139 Change-Id: Idf56cc5a8fdc30490c750f22e3e6341100fa0370 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2303462Reviewed-by:
Travis Skare <skare@chromium.org> Commit-Queue: Sophey Dong <sophey@chromium.org> Cr-Commit-Position: refs/heads/master@{#789537}
-