- 16 Mar, 2020 40 commits
-
-
Dan Harrington authored
Updated protos so we can access the xsurface data. Updated proto converter to grab that data. Bug: 1044139 Change-Id: I25e5f8584a0be4dc27c20ac8000cfa1957fc6fed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105420Reviewed-by:
Ian Wells <iwells@chromium.org> Commit-Queue: Dan H <harringtond@chromium.org> Cr-Commit-Position: refs/heads/master@{#750630}
-
Adam Langley authored
GoogleG2 and G3 are retired. Instead include the new GTS roots. Change-Id: Iea2952ad55c4e4ec26f5d7421a89ead93196f8a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105402 Commit-Queue: Adam Langley <agl@chromium.org> Commit-Queue: Nick Harper <nharper@chromium.org> Auto-Submit: Adam Langley <agl@chromium.org> Reviewed-by:
Nick Harper <nharper@chromium.org> Cr-Commit-Position: refs/heads/master@{#750629}
-
Xiaocheng Hu authored
Font = FontDescription + FontFallbackList In canvas font resolution, the result is just a FontDescription. However, we currently cache the resolved fonts as Font objects with a null FontFallbackList, and pass Font objects to CanvasRenderingContext2DState::SetFont(), which is unnecessary. Hence, this patch changes the code to use FontDescription instead. This patch also makes the code easier to analyze, so that we can eventually eliminate Font::Update(font_selector). Bug: 1049295 Change-Id: I1e3c5a34347355aad2a24f26d35685e9125f7c5b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103136Reviewed-by:
Fernando Serboncini <fserb@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#750628}
-
Xiaocheng Hu authored
The site creates a new ComputedStyle(), and then immediately uses its FontSelector, which is always nullptr at the moment, to call Font::Update(). Hence, this patch simpilies it with nullptr. This is preparation for eliminating Font::Update(). Bug: 1049295 Change-Id: I7279993d62c0a6a25e34fbfce8b5cd3dc6e6a3f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103647Reviewed-by:
Anders Hartvoll Ruud <andruud@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#750627}
-
Marc Treib authored
Test-only cleanup: Before this CL, BubbleObserver::AcceptUpdatePrompt accepted a |form| parameter that it didn't use. This CL removes the param and updates call sites accordingly. Bug: none Change-Id: I4798237434dd02b7ff852a29f70a6cd891234fa0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105300Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#750626}
-
Xiaocheng Hu authored
This function does the following: 1. Create a temporary ComputedStyle |font_style| 2. Make |font_style| adopt its current FontSelector, which is always nullptr at the call site 3. Use StyleResolver::ComputeFont() to resolve font-related properties, which overrides the FontSelector of |font_style| with the underlying document's FontSelector This isn't correct, as it switches the FontSelector of |font_style|, while ComputeFont() can already use the document's FontSelector. The null FontSelector is actually never used. Hence, this patch removes the adoption of null FontSelector. Bug: 1049295 Change-Id: I2d3b2d1affd9317c6535933c1e4cc2421f5be1c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103258Reviewed-by:
-
Charlie Harrison authored
I was staring at some code trying to figure out why ruleset updating worked from the case of an invalid ruleset and wrote out a test with documentation. Bug: None Change-Id: Ibc49461eed421a6a834e966df6a10ea24503429e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1228441Reviewed-by:
John Delaney <johnidel@chromium.org> Commit-Queue: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#750624}
-
Alison Maher authored
Refactor the fieldset Layout() function to increase readability. This CL should have no functional change. Bug: 875235 Change-Id: Ie9081f419fd06f99a50c3def3747b48d08471c9d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2102765 Commit-Queue: Alison Maher <almaher@microsoft.com> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#750623}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/71e412b29c10..e27c5b9cb084 Created with: gclient setdep -r src-internal@e27c5b9cb084 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: None Tbr: jbudorick@google.com Change-Id: I731999e155e41a8ebcd8f038556a347da3dadffd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105583Reviewed-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@{#750622}
-
James Cook authored
I accidentally left in an old version of a comment. TBR=treib@chromium.org Bug: none Change-Id: I6fdb1761e840b58e3fe76b9079343a094b7a2ee2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2104988Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#750621}
-
Sean Gilhuly authored
Fixes for these tests have been landed in Skia. Re-enable them on the SkDawn bot. Bug: 1021566 Change-Id: I83dc03e3c855b23aceb6244d9feff3e19c19c6d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105394Reviewed-by:
Jonathan Ross <jonross@chromium.org> Commit-Queue: Sean Gilhuly <sgilhuly@chromium.org> Cr-Commit-Position: refs/heads/master@{#750620}
-
Kyle Ju authored
Improve logging for unprocessable entity 422 issue because GitHub does not return an informative HTTPError. 422 error is caused by (1) branch already exists but somehow create_pr is reached (2) missing Change_Id in the original CL. Bug: 1056378 Change-Id: I92e0c59f30c5436121ac88af3972a3be0e2a46bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097156 Commit-Queue: Kyle Ju <kyleju@chromium.org> Reviewed-by:
Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#750619}
-
Nicolás Peña Moreno authored
Bug: 1050727 Change-Id: Ie8a33b4b63a3bbec0e85b7d3c9f1dc82a8db59cc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101541 Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Reviewed-by:
Bryan McQuade <bmcquade@chromium.org> Cr-Commit-Position: refs/heads/master@{#750618}
-
Dave Tapuska authored
Implement DragMsg_SourceSystemDragEnded as a FrameWidget message. This change is mostly plumbing giving blink the ability to bind and acquire Widget based mojo channels. BUG=1007957,1039255 Change-Id: I1a619fbb9270a21795411b210dcda70356c45594 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2090297 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#750617}
-
Toni Barzic authored
Adds an animation to shelf drag handle and gesture contextual nudge for login shelf gesture controller. The login shelf gesture handler is active during the last step of OOBE, where the user can swipe up from the shelf to start the session. This CL adds an animation that moves the drag handle and the contextual nudge up and down, to bring more attention to it. Also, updates the drag handle size and color based on the updated specification. BUG=1061627 Change-Id: Iab380a84e8ff769ae16f82f5591fe9c8befaa980 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103655 Commit-Queue: Toni Baržić <tbarzic@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#750616}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/717aa4d9..3c330b35 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: I9fcf9826563de6bd53e76d8c401172bf3deb36b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105733Reviewed-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@{#750615}
-
W. James MacLean authored
This test is also flakey on Linux, and to some extent Win. TBR=mkwst@chromium.org Bug: 1060175 Change-Id: I95c4c0cca2a42ae2883254d89f6c2fcd1f56f0fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105756Reviewed-by:
James MacLean <wjmaclean@chromium.org> Commit-Queue: James MacLean <wjmaclean@chromium.org> Cr-Commit-Position: refs/heads/master@{#750614}
-
Adam Langley authored
With this change, the caBLE authenticator will advertise with it's long-term keys and (always) exchange pairing information when doing QR handshakes. This removes the need to do a QR pairing for every operation. BUG=1002262 Change-Id: I1999e3de347f278a460f126c491d185d1cb18e95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101288 Commit-Queue: Adam Langley <agl@chromium.org> Auto-Submit: Adam Langley <agl@chromium.org> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#750613}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/f94815068d75..6c138716e9ad git log f94815068d75..6c138716e9ad --date=short --first-parent --format='%ad %ae %s' 2020-03-16 hcm@google.com fix up release notes file for milestone 83 2020-03-16 csmartdalton@google.com Remove GrMesh::setVertexData 2020-03-16 csmartdalton@google.com Rearrange Vulkan asserts for samplers 2020-03-16 mtklein@google.com begin refactoring SkTDynamicHash and SkTMultiMap 2020-03-16 mtklein@google.com Shrink SkTHashTables 2020-03-16 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ../src 907b992f..6d7621ab (422 commits) 2020-03-16 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 6963f58c7678..cb9a86a9d867 (13 commits) Created with: gclient setdep -r src/third_party/skia@6c138716e9ad 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 egdaniel@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: egdaniel@google.com Change-Id: If9a160f7f413276aca518068af73eb309b292f83 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105580Reviewed-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@{#750612}
-
David Maunder authored
serialization method to protobufs. Currently in TabState fields are persisted by storing in a file one after another. This approach does not scale well - fields are not well organized and this approach makes it difficult to deprecate fields. PersistedTabData enables persisted fields to be better organized and protobufs offer better support for adding and deprecating fields. PersistedTabData also enables different storage methods - in an upcoming CL a PersistedTabData which uses a Level DB backend will be used for data for building new features but is not pertinent to the running of the app. Bug: 1046367 Change-Id: Id73074303641babb24e2f6109254c53d0d763e2c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2025317 Commit-Queue: David Maunder <davidjm@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Cr-Commit-Position: refs/heads/master@{#750611}
-
Friedrich Horschig authored
Users should not be able to retry the passwords check immediately if they don't have passwords to check. Saving at least one password will put it back into an idle state. Screenshots are in the linked bug. Bug: 1061935 Change-Id: I2eb101a32ca5a6a44e86a7cfbd458af1fd2efec3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105220 Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#750610}
-
Robert Ogden authored
Adds the following checks: * Prefetching cannot start if the tab is not visible. * When a tab becomes visible, prefetching is restarted. Note that a tab becoming hidden does not stop the current prefetch request. Instead, the current prefetch is allowed to finish in the hidden tab but no more will begin. This is partly done to avoid additional code complexity in storing the current prefetch url and partly because a user is likely to switch between a SRP tab and its results, so prefetching those links is beneficial in the end. Bug: 1023486 Change-Id: I03cff0f77721f450ac97b4aec39ad87290cbc886 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105403Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Commit-Queue: Robert Ogden <robertogden@chromium.org> Cr-Commit-Position: refs/heads/master@{#750609}
-
Justin Cohen authored
Changes in autocomplete logic broke some of the assertions in testTapPrerenderSuggestions. Since those assertions are just helping to set up the test, simply remove them. Bug: 1061373 Change-Id: Ie7da0302b3de5afe920aa2bd84bc5342cc80bbe6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103710 Auto-Submit: Justin Cohen <justincohen@chromium.org> Commit-Queue: Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#750608}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/29de50e4d8ee..27657d9727a1 git log 29de50e4d8ee..27657d9727a1 --date=short --first-parent --format='%ad %ae %s' 2020-03-16 ddrone@google.com Add brief instructions on IDE setup Created with: gclient setdep -r src/third_party/perfetto@27657d9727a1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: None Tbr: perfetto-bugs@google.com Change-Id: I541c08a07ca273ba17b0d9683a0efb915197434f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105579Reviewed-by:
-
Clark DuVall authored
It was a bit weird having embedders implement PermissionPrompt::Create() for desktop. This change implements it in //components/permissions by calling through to the embedder through PermissionsClient. Bug: 1025609 Change-Id: Icc1fd1e4a5f6efc2a8141a469cfb322579384e2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101407 Commit-Queue: Clark DuVall <cduvall@chromium.org> Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#750606}
-
Jun Mukai authored
This has been disabled due to flakiness. The flakiness was observed on asan bot and win7 bot. Now this test is only built on non-sanitizer builds for chromeos (see r749747), so it's safe to re-enable it. As I tried running, it still fails sometimes on debug builds on waiting logic, so I modified it slightly. Bug: 1057864, 1057868 Test: interactive_ui_tests --gtest_repeat=20 Change-Id: I9e8112ffbd6f30f042ffd03f6b29bc7e1dbe18b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103521Reviewed-by:
-
kylechar authored
Bug: none Change-Id: I5be6711cbe726abfdc5bf7c21e7ec2e17e35b61d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105401Reviewed-by:
danakj <danakj@chromium.org> Commit-Queue: kylechar <kylechar@chromium.org> Cr-Commit-Position: refs/heads/master@{#750604}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/9b620aa0c12d..4b2483ee88ab git log 9b620aa0c12d..4b2483ee88ab --date=short --first-parent --format='%ad %ae %s' 2020-03-16 mbechard@users.noreply.github.com Fix #2005. Allow multiple compilation units to declare identical push_constant blocks (#2123) Created with: gclient setdep -r src/third_party/glslang/src@4b2483ee88ab If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/glslang-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,jmadill@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 Bug: None Tbr: radial-bots+chrome-roll@google.com,jmadill@google.com Change-Id: Ie9e2fed7147a8549b1cd977f09134c2884dab27e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105577Reviewed-by:
-
Roman Sorokin authored
Sometimes network is not connected yet when online load is happening. So in case of a network error - try again while within timeout. Fixed: 1061202 Change-Id: Ic5a71eab806bfc48615d6f4c684f90abc1aeef3f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2102491Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#750602}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/1a56ce54e070..022952303c1a git log 1a56ce54e070..022952303c1a --date=short --first-parent --format='%ad %ae %s' 2020-03-16 thakis@chromium.org Suppress two -Wunreachable-code warnings. Created with: gclient setdep -r src/third_party/dawn@022952303c1a 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 cwallez@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: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: chromium:346399 Tbr: cwallez@google.com Change-Id: Icdb88268a388dfcb4d1f11d2f7f0b44be3b89b68 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105581Reviewed-by:
-
ckitagawa authored
This CL is intended to integrate the hit tester into the player delegate. Note: it doesn't actually perform navigation or handle overlapping links yet (see crbugs). Bug: 1019883 Change-Id: I490e420ab31b2a828e8aa53779513ea4313c0935 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101489Reviewed-by:
Mehran Mahmoudi <mahmoudi@chromium.org> Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org> Cr-Commit-Position: refs/heads/master@{#750600}
-
James Cook authored
Fix the tests in anticipation of enabling the feature by default. The tricky bit is that UserSelectableType::kApps isn't used on Chrome OS when the feature is enabled, but it is still part of UserSelectableTypeSet::All(). Update the tests to use the registered selectable types, which is closer to what the production code does. Test: components_unittests --enable-features=SplitSettingsSync Bug: 1060289 Change-Id: I16d4f119c1daa1cb51f2b9a35ea247beec75ece9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101722 Commit-Queue: James Cook <jamescook@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Auto-Submit: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#750599}
-
Side Yilmaz authored
ExploreSitesBackgroundTask. Profile#getLastUsedProfile| is deprecated and replaced with |Profile#getLastUsedRegularProfile|. This CL updates ExploreSitesBackgroundTask class. Note: This change is only code clean-up, does not change any behavior. Bug: 1041781 Change-Id: I8e6be991e2358fdb2130508c8baf0a46861a3e52 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095131Reviewed-by:
Justin DeWitt <dewittj@chromium.org> Commit-Queue: Side YILMAZ <sideyilmaz@chromium.org> Cr-Commit-Position: refs/heads/master@{#750598}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/d1d96d73bff7..a8631f682eb5 git log d1d96d73bff7..a8631f682eb5 --date=short --first-parent --format='%ad %ae %s' 2020-03-16 jmadill@chromium.org Fix running trace tests in non-standard directories. Created with: gclient setdep -r src/third_party/angle@a8631f682eb5 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/+/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 Bug: None Tbr: ynovikov@chromium.org Change-Id: Ia08c531ab4b9f62ab8a4ddd07f2d6f82749326df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105573Reviewed-by:
-
Hailey Wang authored
Add a policy in Performance Manager that will be responsible for background tab loading in replacement of TabLoader class. As of now, this policy consist of loading in a FIFO order with no additional restraint. Bug: 1059341 Change-Id: I1fbbfa096f44f75cad018c81b5ce2f921863503c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2098743Reviewed-by:
Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Commit-Queue: Hailey Wang <haileywang@google.com> Cr-Commit-Position: refs/heads/master@{#750596}
-
Eugene But authored
I looked at existing Study and the number of entries is too low for statistically significant experiment. We should get more events by disabling the animations even if battery life is higher than 20%, so we can determine if animations affect PLT. This CL raises threshold to 50%, which is arbitrary chosen number. The experiment is only running on dev and canary, and not intended for beta or stable. Bug: 1006431 Change-Id: I8b6f0cb1ba876c7db11ec23169136e41c21b058e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2096170Reviewed-by:
David Jean <djean@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#750595}
-
Robbie Gibson authored
Now, the tab helper holds whether the ui is active or not. Besides disabling the Text Zoom menu option when the UI is active, this also allows for better state logic to re-present the bar after a temporary hide. For instance, it is hidden when you begin a side swipe-to-navigate. Before, it wouldn't come back if you interrupted the swipe, but now it does. Fixed: 1056814, 1056821 Change-Id: I59d0b885bb4143c63563569837d1558af0b58024 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2080413Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Robbie Gibson <rkgibson@google.com> Cr-Commit-Position: refs/heads/master@{#750594}
-
Joe DeBlasio authored
This CL fixes a mistaken NOTREACHED() that was reached when a user opened PageInfo on a page that triggered the (metrics-only) sensitive-keyword heuristic. It also adds a test for such. Fixed: 1061244 Change-Id: I11e6be8c4f75f308cc8f75f48523c4f64a275917 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103398Reviewed-by:
Christopher Thompson <cthomp@chromium.org> Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Commit-Queue: Joe DeBlasio <jdeblasio@chromium.org> Auto-Submit: Joe DeBlasio <jdeblasio@chromium.org> Cr-Commit-Position: refs/heads/master@{#750593}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/496706c9672e..98b339fb3c01 git log 496706c9672e..98b339fb3c01 --date=short --first-parent --format='%ad %ae %s' 2020-03-16 wolfi@chromium.org New icons for breakpoints Created with: gclient setdep -r src/third_party/devtools-frontend/src@98b339fb3c01 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: I4493d277b78505954032b4c6b8036707790a0957 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105574Reviewed-by:
-
Xiaocheng Hu authored
Both call sites of Font::Update() in FontFaceSetDocument are redundant, so this patch removes them. The first call site is redundant as StyleResolver::ComputeFont() will set the underlying Document's FontSelector to the font. The second call site is redundant as it repeatedly sets the Document's FontSelector to the font. This is also preparation for eliminating Font::Update(). Bug: 1049295 Change-Id: I43196beb8c77e420083a125a6050d73d06e7b9c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103146Reviewed-by:
-
