- 12 Oct, 2020 40 commits
-
-
Victor Hugo Vianna Silva authored
(Firstly, this CL is only big because it it removes 3 similar methods at each time - Status/Commits/Updates counters - and the existing plumbing goes through many layers. All the changes are trivial code removal and no behavior is changed.) The last real implementation of TypeDebugInfoObserver was removed in crrev.com/c/2465743. The remaining implementation in SyncEngineBackend actually just propagates the notifications up to ProfileSyncService::type_debug_info_observers_, which is now empty. So this CL removes the interface and all associated code: - Implementations of On[Commit/Update/Status]CountersUpdated and their associated plumbing methods. - Methods to add/remove an observer, and observer lists. - Methods to enable the notifications (e.g EnableDirectoryTypeDebugInfoForwarding). A few points are left for future CLs: - Properly repurposing DataTypeDebugInfoEmitter. This class now only records UMA for the status/commits/updates counters. It should be renamed or at least have its documentation updated. It can also be merged with its only implementation, NonBlockingTypeDebugInfoEmitter. - Removing DataTypeController::GetStatusCounters(), which is now unused. Bug: 1102849 Change-Id: Ic993f60995f2c48ad7d284ebe218136d8ebe564d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465744 Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#816152}
-
Luciano Pacheco authored
These files are dependency used by Files app which is process to generate JS modules. Bug: 1133198 Change-Id: Ib749ad8cf9c96be96f6c0372d3c80a2f74a91afb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2440376 Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#816151}
-
Tanmoy Mollik authored
If the user has only signed in without giving sync consent then the sign out button in account management page should not show the sign out dialog. Instead it will sign the user out directly without deleting data saved on device. Bug: 1136075 Change-Id: Ie2485e2896518113d62cec4a1c9fd57bfbaef2b1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2460817 Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#816150}
-
Koji Ishii authored
This patch uses the return value of |insert| to check if the key is already in the map or not. This patch has no behavior changes. Bug: 652964 Change-Id: I6d53b923a7dbb8b39fdb03ec1a833ff8e88b5c9f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462856 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#816149}
-
Marc Treib authored
Contrary to the code comment, it was only used for the PROXY_TABS data type. This is now handled explicitly instead. Bug: 647505 Change-Id: I5fd3fdb4d93230acac32966c1ee54334983dc6b8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462093 Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
-
Tanmoy Mollik authored
Change ManageSyncSettings title to "Sync". Screenshot: https://crbug.com/1135923#c6 Bug: 1135923 Change-Id: I4a4c732b5149ae78476186c07893a1b61b0ad7b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461341Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> Cr-Commit-Position: refs/heads/master@{#816147}
-
Aaron Tagliaboschi authored
This reverts commit 0e9a9078. Reason for revert: The surfaces need be measured against the BluetoothRemoteGATTCharacteristic that a BluetoothCharacteristicProperties describes Original change's description: > Instrument BluetoothCharacteristicProperties for identifiability study > > Bug: 973801 > Change-Id: I0c1616e85080c1048f6777caaa7769ce8398e507 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2354270 > Commit-Queue: Aaron Tagliaboschi <aarontag@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Cr-Commit-Position: refs/heads/master@{#799143} TBR=dcheng@chromium.org,asanka@chromium.org,aarontag@chromium.org Bug: 973801 Change-Id: Ia92e1bff1ef21b1167786ceb3fca1ffe257e2852 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463595Reviewed-by:
Aaron Tagliaboschi <aarontag@chromium.org> Reviewed-by:
Asanka Herath <asanka@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Aaron Tagliaboschi <aarontag@chromium.org> Cr-Commit-Position: refs/heads/master@{#816146}
-
Internal Frameworks Autoroller authored
TBR=bling-team@google.com Change-Id: I2ef9b64449927ea2e77e1c90edba51dc31833781 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464092Reviewed-by:
Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: Internal Frameworks Autoroller <bling-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#816145}
-
Gyuyoung Kim authored
This CL replaces all uses of web_pref::ImageAnimationPolicy with mojom::ImageAnimationPolicy to reduce unnecessary type conversion. Bug: 1097943 Change-Id: I608bd5919c7ae5d20ed8eba02c6bb9a1bcba9348 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465802Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Gyuyoung Kim <gyuyoung@igalia.com> Cr-Commit-Position: refs/heads/master@{#816144}
-
Kuo-Hsin Yang authored
Before this patch, GetSystemMemoryInfo() reads both /proc/meminfo and /run/debugfs_gpu/i915_gem_objects (for x86 CPU family). Reading /run/debugfs_gpu/i915_gem_objects could take more than 500 ms under memory pressure on Intel platform with kernel 5.4, while reading /proc/meminfo only takes about 1 ms. Add GetGraphicsMemoryInfo() to read the gpu memory info, GetSystemMemoryInfo() reads /proc/meminfo only. GetGraphicsMemoryInfo() would return false on Intel platform with kernel 5.4 to avoid the slow i915_gem_objects access. Bug: b/170150180, 1134131 Change-Id: I0a0cfb868826b59f508a478d3db69ec089f5d4d5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2449519 Commit-Queue: Kuo-Hsin Yang <vovoy@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Cheng-Yu Lee <cylee@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Brian Geffon <bgeffon@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#816143}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/ec23cfada4cd..30127e006a5b 2020-10-12 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: Ica475455f5f046fb09f2e6aa98d90f71b8d0b0c7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465563Reviewed-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@{#816142}
-
Julie Jeongeun Kim authored
This CL introduces AwPrintManager::UpdateParam() which updates |settings_|, |fd_|, and |pdf_writing_done_callback_| in order to reuse AwPrintManager avoiding creating it every time. It fixes a regression issue by crrev.com/c/2326857. The problem is that WebContentsFrameReceiverSet is created always along with AwPrintManager but WebContentsImpl::AddReceiverSet() expects that it's called once for an each interface. So, this CL doesn't create AwPrintManager always like other print managers such as PrintViewManager or HeadlessPrintManager. Bug: 1137016 Change-Id: I11c176009ccdad643ebf495cc4b983209ba8b1c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463270Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Julie Kim <jkim@igalia.com> Cr-Commit-Position: refs/heads/master@{#816141}
-
Rune Lillesveen authored
Bug: 1068610, 1136862 Change-Id: Iecebc6f71099695a60ee65eb4899c1bd2437704c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463920Reviewed-by:
Anders Hartvoll Ruud <andruud@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#816140}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/26d93a4d..66b5d4eb Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I811f8d7526b825a24db210c963797ae911a10f71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465983Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#816139}
-
Reda Tawfik authored
Bug: 1104132 Change-Id: I9a1d60eaa56b985e0a4f605f2b5f607ed935acc2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463578 Commit-Queue: Reda Tawfik <redatawfik@google.com> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#816138}
-
Koji Ishii authored
This patch implements |HyphenationComponentInstallerPolicy|, a component updater for hyphenation dictionaries. The code is tested locally, but this patch: * Does not pass `install_dir` to the hyphenation engine yet. * `RegisterHyphenationComponent` is not enabled on any platforms yet, as there are no platform that use the `#if` combination yet. Bug: 652964 Change-Id: I9afc09dfff166d5d903372bd5af2471635d90f7e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2437314Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#816137}
-
Denis Kuznetsov authored
Bug: 1134114 Change-Id: I6f85ad53b5be40dd53485efde930362fc73188e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2452473 Auto-Submit: Denis Kuznetsov [CET] <antrim@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Denis Kuznetsov [CET] <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#816136}
-
Maksim Moskvitin authored
top_level_permanent_item_ids_ wasn't persisted and could potentially introduce problems with PRE_ tests. It was used only for Nigori and it's safe to replace it with LoopbackServerEntity::GetTopLevelId() (because permanent items are permanent). Bug: 1104225 Change-Id: Ie92a54b0cfc80d671be96922c6dc265ed62594f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465846 Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com> Commit-Queue: Marc Treib <treib@chromium.org> Auto-Submit: Maksim Moskvitin <mmoskvitin@google.com> Reviewed-by:
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/6c3da3dc5b87..936ef0a5f21e 2020-10-12 hao.x.li@intel.com Query API: Timestamp Query on Metal If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC kainino@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel Bug: None Tbr: kainino@google.com Change-Id: I32cddc5010f679dcedddb9260821bc41ba9cf8cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465923Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium 445fb131. With Chromium commits locally applied on WPT: a46c52a5 "Clean ImageCapture web platform tests" 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=smcgruer@google.com No-Export: true Cq-Include-Trybots: luci.chromium.try:linux-wpt-identity-fyi-rel,linux-wpt-payments-fyi-rel Change-Id: I359795825eec6e2b6df780e3b26e350cce6381f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465679Reviewed-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@{#816133}
-
Marc Treib authored
It was always empty in the USS world (the concept of data type association doesn't really exist anymore). Bug: 1102837 Change-Id: Ib8e9fbc13b94bafd8fddde253fde514d1c65f6e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463917 Commit-Queue: Rushan Suleymanov <rushans@google.com> Auto-Submit: Marc Treib <treib@chromium.org> Reviewed-by:
Rushan Suleymanov <rushans@google.com> Cr-Commit-Position: refs/heads/master@{#816132}
-
chrome://sync-internalsVictor Hugo Vianna Silva authored
This debugging page hasn't been used by the Sync team in a while, so this CL removes it. Consequently, SyncInternalsMessageHandler no longer needs to implement TypeDebugInfoObserver. This unblocks the removal of such observer interface in future CLs (chrome://sync-internals was its last "real" implementation). Bug: 1102849 Change-Id: Idb2ed8af56c9133a884b79ccfa131dce630fa1bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465743Reviewed-by:
-
lonyele authored
When selection is happening, apply text-decoration-color from selection pseudo style to the selected element Bug: 1110393 Change-Id: I3829c7071b85458c43975d622f0913b8f8c64fd7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2383212Reviewed-by:
Stephen Chenney <schenney@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#816130}
-
Nico Weber authored
Normally, we allow running several links in parallel if there's enough memory, since links tend to be memory- and io-bound, not compute bound. However, in LTO builds the linker does codegen for every linked translation unit, which needs a lot of CPU. Previously, we used to hardcode 8 threads per link for thinlto builds, which makes builds needlessly slow on machines with many cores. Instead, automatically use all cores for a thinlto link, but only run one link in parallel at once. chromeos-chrome-9999.ebuild in the Chrome OS build and link_jobs_32 in mb_config.pyl currently manually set the number of threads per lto link to the number of cores. Once this change here is in, we should stop explicitly setting the arg there and then we can turn this from a gn arg to a regular variable that's not settable in args.gn Bug: 1132930 Change-Id: If2fda3e38b2b1827beb2da844f024f52211210db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464782Reviewed-by:
Hans Wennborg <hans@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Auto-Submit: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#816129}
-
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-sdk-chromium-autoroll Please CC cr-fuchsia+bot@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 Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast Tbr: cr-fuchsia+bot@chromium.org Change-Id: I6892611036fe227ce0b4437dd8206494367e1571 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465686Reviewed-by:
-
Igor authored
This will disallow the usage of Lacros browser by default to managed users. For unmanaged users Lacros will be available. Also it will be available for users where the admin set the policy LacrosAllowed to true. BUG=chromium:1136199 TEST=Manual test on device Change-Id: I60554ab7ed7813207aa602eb17801a4e82be47d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2456808 Commit-Queue: Igor <igorcov@chromium.org> Reviewed-by:
Bartosz Fabianowski <bartfab@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Igor <igorcov@chromium.org> Cr-Commit-Position: refs/heads/master@{#816127}
-
Stephen Chenney authored
For tiled image painting the image was not correctly oriented. Fix it by pre-orienting the image before creating the tiling shader. The test still fails due to a lack of fuzzy matching, but the result is now correct. See external/wpt/css/css-images/image-orientation/image-orientation-background-properties.html Bug: 1066634 Change-Id: Ie0a2f534bf760174d337fdaa6838e82b33280d46 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464902Reviewed-by:
Fredrik Söderquist <fs@opera.com> Commit-Queue: Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#816126}
-
Yuki Shiino authored
This is a follow-up of https://crrev.com/c/2418513. We need to intercept getcwd and realpath, which allocate memory internally. The reason is the same as 2418513. Bug: 1111332 Change-Id: I50c43056609b69020dc84bf181ec6247067cb982 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2438093 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Will Harris <wfh@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Reviewed-by:
-
Anders Hartvoll Ruud authored
This is only used by test code, and it's standing in the way of implementing tree-scoped references, so this CL removes this capability. Most of the tests could be fixed with just a simple Reset (for example: BasicCycle). This means that the cascade is empty after each Reset, so those tests are no longer implicitly testing that declarations appearing later overwrite those that appear earlier. But that's OK, there are still loads of tests which cover this explicitly. RevertInKeyframeResponsive is supposed to test what happens when --x changes during an animation that depends on --x _via_ revert. So here we can't just do an plain Reset, we need to re-add all the other declarations as well, to simulate a situation where only --x changed. Then there were a few tests which for some reason calculate the animation update and _then_ Add stuff to the MatchResult (for example: AnimatingVarReferences). This does not happen in production. Moved the Add calls to before the first Apply call. Change-Id: I00c3b26cbd7b9581e1bd61b5cd27163ebe651b92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464935Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Anders Hartvoll Ruud <andruud@chromium.org> Cr-Commit-Position: refs/heads/master@{#816124}
-
Ehimare Okoyomon authored
Screenshot: https://crbug.com/1136533#c1 Bug: 1136533 Change-Id: I6140f9e49a723afd0dd4ae22b059784b2cf73d2f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461308 Commit-Queue: Ehimare Okoyomon <eokoyomon@chromium.org> Reviewed-by:
Clark DuVall <cduvall@chromium.org> Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Cr-Commit-Position: refs/heads/master@{#816123}
-
Eric Seckler authored
Currently, base::CPU::GuessCoreTypes() is called in multiple places and executes the same code multiple times, even though the result is constant. There's already some static caching in different places. This patch moves the static caching into base::CPU::GuessCoreTypes() and renames it to base::CPU::GetGuessedCoreTypes(). Bug: 1081760 Change-Id: Ifaff01cc19abc48a36d5fe19bc4cb9a9cfa4fd24 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463309 Commit-Queue: Eric Seckler <eseckler@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/9024c3764962..ac0366e76f36 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 amraboelkher@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: None Tbr: amraboelkher@google.com Change-Id: If146743d69e69ea72e8f52539482c7cfbbe0e4e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465922Reviewed-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@{#816121}
-
Marc Treib authored
It was unused. Bug: 647505 Change-Id: I19b16e8a5c99d0c490e8e0df05ec29985d98b7cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2463918Reviewed-by:
Maksim Moskvitin <mmoskvitin@google.com> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#816120}
-
Finnur Thorarinsson authored
This change of function contract is the by-product of a discussion on fixing issue 1130909, where we decided to do a bit of additional cleanup as well. Bug: 1130909 Change-Id: I28d0e244ead44b8f6525bf1e0d86633256d06281 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2456786Reviewed-by:
Peter Conn <peconn@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Gang Wu <gangwu@chromium.org> Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Cr-Commit-Position: refs/heads/master@{#816119}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/68bd685a7643..dec35c569444 2020-10-12 angle-autoroll@skia-public.iam.gserviceaccount.com Roll Chromium from 77bfcdcd to efbcef25 (569 revisions) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC ynovikov@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 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: ynovikov@chromium.org Change-Id: Ib16c6154813710d0a5b60de4b176b7f1dbdc0cb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2462740Reviewed-by:
-
Victor Hugo Vianna Silva authored
The code to migrate sync.local_device_guids to the new sync.local_device_guids_with_timestamp has been active for 1 year (crrev.com/1827435), so it can be removed now. TBR=treib@chromium.org TBR=droger@chromium.org Fixed: 994339 Change-Id: If6fed35ada178808a4171d790f51451f5a8192df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2464934 Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by:
-
Fredrik Söderqvist authored
This allows "content negotiation" to detect support for SVG favicons. Bug: 1026564 Change-Id: I612a08872f16bdb0dc71b9779090b5759806f815 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1926312 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
David Benjamin <davidben@chromium.org> Reviewed-by:
Mathias Bynens <mathias@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#816116}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/07b0b1dc..26d93a4d Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ib963131764aef6cec54151d7f9ea26f6bd8e1859 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465670Reviewed-by:
-
Victor Hugo Vianna Silva authored
Before this CL, screenreaders would read the buttons in the sync consent dialog, but wouldn't read the text content unless the user focused it with the mouse. With this CL, we ensure that at least the dialog title ("Turn on sync?") is read once before the buttons. Screencast: crbug.com/1130433#c3 Fixed: 1130433 Change-Id: I2dbdae2489be7672bd871a63384db2885d1bfcc1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2465842Reviewed-by: -
Paula Vidas authored
This is necessary for two client tests that use the new invalidations system. Different clients need different tokens because the (fake) server will use the tokens to determine invalidations recipients. Bug: 1135167 Change-Id: I0f60de8d27ad6b6a338818cde530df2f860c056f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2461012Reviewed-by:
-
