- 19 Jun, 2020 40 commits
-
-
chromium-autoroll authored
Roll Chrome Win64 PGO profile from chrome-win64-master-1592524547-1d6c7cf94491bfcfe03077087e03362dbc4dc51a.profdata to chrome-win64-master-1592545853-4aad13dd3d5b31e6b22212a49c45cb09e0a49e05.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win64-chromium Please CC jeffyoon@google.com,liaoyuke@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: chrome/try:win64-chrome Tbr: jeffyoon@google.com,liaoyuke@google.com Change-Id: I07f17ab4f967f5471bf1e8ef4b90ce46317d476c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253718Reviewed-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@{#780217}
-
Nohemi Fernandez authored
Part of the cleanup effort to fully support the new sign-in architecture. Bug: 971989 Change-Id: Iddc3e58ecb08cb0bc7bbd4f74f9bc3e5af3f650c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2251922Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Commit-Queue: Nohemi Fernandez <fernandex@chromium.org> Cr-Commit-Position: refs/heads/master@{#780216}
-
Yuki Shiino authored
Bug: 839389 Change-Id: Icc338ca2394f1f32265d318ab7ea530784f4b29b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249403 Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#780215}
-
Rahul Singh (EDGE) authored
This CL adds support for Shortcuts Menu to be updated when an installed PWA's manifest is updated. Code in manifest_update_task.cc detects changes to the shortcuts member of the manifest. It then triggers a reinstallation of the PWA per the new info in the manifest. As part of this flow we Unregister the existing Shortcuts Menu with the OS and then Reregister it as part of the reinstallation flow. To help detect changes in the manifest, this CL adds new methods to AppRegistrar that help read existing ShortcutsMenu related info. These methods have been implemented for both BookmarkApp and WebApp systems. Testing: This CL adds Shortcuts Menu related browser tests in manifest_update_manager_browsertest.cc. It also adds new tests in web_app_install_task_unittest.cc for invoking UpdateWebAppFromInfo with Shortcuts Menu data. Design doc: https://docs.google.com/document/d/1KhHWikx1z-604LIjDKEdnL9MyezJklxM8kEYNnMLIQs/ Bug: 1087212 Change-Id: Ie825494ab137724e07e55bc12098cbfb386fb6ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2244111Reviewed-by:
Alexey Baskakov <loyso@chromium.org> Reviewed-by:
Alan Cutter <alancutter@chromium.org> Commit-Queue: Rahul Singh <rahsin@microsoft.com> Cr-Commit-Position: refs/heads/master@{#780214}
-
Hajime Hoshi authored
We are adding a new metrics related to back-forward cache e.g., first input delay after back-forward cache navigation happens. This CL is a preparation for that change. Bug: 1014174 Change-Id: I38549e4acb0659877bcd3aa976f0697a596e45dc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2246360 Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Reviewed-by:
Nicolás Peña Moreno <npm@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#780213}
-
Side Yilmaz authored
This CL replaces usages of |profile_key_util.cc#GetLastUsedProfileKey| with |profile_key_util.cc#GetLastUsedRegularProfileKey| in notifications classes. Note: This CL is only clean-up and does not change code behaviour. Bug: 1041781 Change-Id: Id4583df9615bded3a779834d9bf7a78e10ad5f2c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2247773Reviewed-by:
Xing Liu <xingliu@chromium.org> Reviewed-by:
Ramin Halavati <rhalavati@chromium.org> Commit-Queue: Side YILMAZ <sideyilmaz@chromium.org> Cr-Commit-Position: refs/heads/master@{#780212}
-
Kenichi Ishibashi authored
This CL adds a storage operation called GetStorageUsageForOrigin() which calculates the total resource size stored in the database for a given origin. ServiceWorkerQuotaClient::GetOriginUsage() uses the new operation instead of GetAllOriginsInfo(). The motivation of this change is to eliminate unnecessary memory allocations and data conversions GetAllOriginsInfo() performs (see [1] for details). When the browser has many service worker registrations for different origins, these allocations/conversions have considerable performance impact. In my local experiment, this CL makes GetOriginUsage() ~10x faster when there are 15k registrations/origins (340-390ms -> 40-45ms). [1] https://docs.google.com/document/d/1iaQGQzovFVL3LxDiyIvVgEUFWMhzrXpdf_6TmuwIICA Bug: 807440 Change-Id: Ice16640811a0f9f92ac0a71134e7b5666b13e29f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2250141 Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#780211}
-
Rachel Wong authored
This CL defines and implements the search service proxy API. - IndexProxy implements an interface that is a wrapper around an Index; - LocalSearchServiceProxy implements an interface that is a wrapper around LocalSearchService and owns various IndexProxies; - LocalSearchServiceProxyFactory is responsible for the creation of LocalSearchServiceProxies. Bug: 1092767 Change-Id: Icb2fa359fe9d05bc4ae5b5bc977550ed659e8262 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249428 Commit-Queue: Rachel Wong <wrong@chromium.org> Reviewed-by:
Sam McNally <sammc@chromium.org> Reviewed-by:
Jia Meng <jiameng@chromium.org> Reviewed-by:
Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#780210}
-
Andy Paicu authored
TBR=battre@chromium.org Bug: 1096966 Change-Id: I01e9d9d0acc27c8651b7a27bd09b132cd183914e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252193Reviewed-by:
Andy Paicu <andypaicu@chromium.org> Commit-Queue: Andy Paicu <andypaicu@chromium.org> Cr-Commit-Position: refs/heads/master@{#780209}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/3d25c77940d2..889563cbeb36 2020-06-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ANGLE from fa3d64bf72d6 to c81f0b7b921f (6 revisions) 2020-06-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll SwiftShader from 7a1d99d0be42 to eb6824408d08 (3 revisions) 2020-06-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll Chromium from 2adbb514 to 4a914f21 (507 revisions) 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 djsollen@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: chromium:1096510 Tbr: djsollen@google.com Test: Test: Test: angle_perftests --gtest_filter="*Trace*" Change-Id: I1a8fe09803d08e91f185ea5495720f1576a37eba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253720Reviewed-by:
-
Alexander Dunaev authored
The drag handler delegate should receive the resulting drag and drop operation via its OnDragFinished, but the Ozone/X11 implementation returned 0 always. This CL fixes that. Bug: 1014860 Change-Id: Ie339b7c7beb374bc58d08a332c2b817b8dc0b04f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253738Reviewed-by:
Maksim Sisov <msisov@igalia.com> Commit-Queue: Alexander Dunaev <adunaev@igalia.com> Cr-Commit-Position: refs/heads/master@{#780207}
-
Side Yilmaz authored
This CL replaces deprecated functions of IdentityServicesProvider with the newer service getters with profile parameter. By this CL, HistoryManager uses IdentityServicesProvider functions with the profile generated regarding to isIncognito parameter from the constructor. Bug: 1041781, 1075562 Change-Id: I04c0a14b5f9e912cda7fb34a581bea4d18023518 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2242078Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
-
Tal Pressman authored
For now, the only active strategy is a "no-op" strategy that has no effect on scheduling behavior. This CL also includes 2 implementations of strategies that (i) disables or (ii) de-prioritzes timer queues for non-main agents, if there are still main frames waiting for FMP. This CL also fixes a UAF bug exposed in frame_scheduler_impl_unittest: frame_scheduler_ was reset after delegate_, but could still access the "old" delegate. Now, the new delegate is created first, passed to the new (reset) frame_scheduler_, and finally delegate_ is reset. Change-Id: Ibd24fc742de5b048495af924122e2a095462f4f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2206301 Commit-Queue: Tal Pressman <talp@chromium.org> Reviewed-by:
Scott Haseley <shaseley@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#780205}
-
Matt Falkenhagen authored
This adds initial support for doing full prefetching of subresources, beyond preconnecting to their hosts. The code is gated behind a feature flag kLoadingPredictorPrefetch. The PrefetchManager logic is strongly based on the PreconnectManager. But splitting them into two separate classes seemed cleaner overall. Bug: 1092329 Change-Id: I26b244105d59e8765fc24c109ec165b7def39f89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2235192 Commit-Queue: Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Sophie Chang <sophiechang@chromium.org> Reviewed-by:
-
Yutaka Hirano authored
Bug: 1011392 Change-Id: Ibd0c765f6a6af28ae91eccdc69997bc53ee4abfa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249400 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Victor Vasiliev <vasilvv@chromium.org> Reviewed-by:
Adam Rice <ricea@chromium.org> Reviewed-by:
-
calamity authored
Bug: 1096670 TBR=ntfschr@chromium.org Change-Id: Id46aea904a83e918c0a819d604f24db4654723f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2251782Reviewed-by:
calamity <calamity@chromium.org> Commit-Queue: calamity <calamity@chromium.org> Cr-Commit-Position: refs/heads/master@{#780202}
-
Christopher Lam authored
This CL refactors the model to more closely match the data. In particular, the model used to be colors[mode][name] and is now colors[name][mode], which matches the data. This is in preparation for implementing opacities, a 'generate-dark-mode-only' mode, and dark-mode-agnostic properties. Other minor changes: - BaseGenerator.colors is now BaseGenerator.model[VariableType.COLOR] - ModeVariables now has methods that let it be used more like a dict() Bug: 1018654 Change-Id: I5422178053434b49ac64a6290983092cbbbf0ab0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2245968 Commit-Queue: calamity <calamity@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#780201}
-
Tal Pressman authored
Change-Id: I9689fd0a2501e810e0ea1ac91d8042e697b5f422 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253599 Commit-Queue: Tal Pressman <talp@chromium.org> Commit-Queue: Kouhei Ueno <kouhei@chromium.org> Auto-Submit: Tal Pressman <talp@chromium.org> Reviewed-by:
-
Alexander Alekseev authored
This adds settings page to Ash HUD and makes HUT semi-transparent. Just the HUD: https://screenshot.googleplex.com/ZBqYnQDomH5.png With the settings button: https://screenshot.googleplex.com/pNCfTAw9J92.png With settings UI displayed: https://screenshot.googleplex.com/SwyNb0xVDbc.png Bug: 1075612 Change-Id: I9f26e8b99f03e0d638f854312820f10ec25ee6a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2223974 Commit-Queue: Alexander Alekseev <alemate@chromium.org> Reviewed-by:
Mitsuru Oshima (slow:gardening) <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#780199}
-
Lingqi Chi authored
This CL replaces "whitelist" with "allowlist" in IgnoreErrorsCertVerifier. Bug: 842296 Change-Id: I501b451fbb76dc48e5977b2a855913ee79028462 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2247947 Commit-Queue: Lingqi Chi <lingqi@chromium.org> Reviewed-by:
Takashi Toyoshima <toyoshim@chromium.org> Cr-Commit-Position: refs/heads/master@{#780198}
-
Louis Collard authored
These still seem pretty relevant given the recent issues DA lockout. Bug: 1089292 Change-Id: If8c21536a30b3d57cc035c59a9669c1b9ffc81a9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2241056 Commit-Queue: Louis Collard <louiscollard@chromium.org> Auto-Submit: Louis Collard <louiscollard@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#780197}
-
Callistus authored
CIPD package: https://chrome-infra-packages.appspot.com/p/chromeos_internal/apps/help_app/app/+/tpCFs6vz-vM58hBheqFjcjl6XHXroSgYifCz3f5OmjsC Release notes: http://go/help_app-x20/relnotes/Nightly/help_app_nightly_202006180800_RC00.html Cq-Include-Trybots: chrome/try:linux-chromeos-chrome Change-Id: I384a64a1ec217ee6d7148e8e2f1b2ae9527345f5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2251522Reviewed-by:
Rachel Carpenter <carpenterr@chromium.org> Commit-Queue: Callistus Tan <callistus@google.com> Cr-Commit-Position: refs/heads/master@{#780196}
-
calamity authored
TBR=wjmaclean@chromium.org Bug: 1092915 Change-Id: Ic2a2b55c695126547e1e11385adf4e235d627c88 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253484 Commit-Queue: calamity <calamity@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/openscreen.git/+log/ec44eb332577..dd945dd1c033 2020-06-19 rwkeane@google.com Discovery: Add support for non-cyclic SRV pointers 2020-06-19 rwkeane@google.com DNS-SD: Update DnsDataGraph 2020-06-19 rwkeane@google.com Discovery: Add support for subtypes to the public headers If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/openscreen-chromium Please CC jophba@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: jophba@chromium.org Change-Id: I017657733b50b9d8e899d62bd8754edb8b8aa5cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253233Reviewed-by:
-
Louis Collard authored
There is no data and no mention of it in the codebase. Bug: 1089292 Change-Id: I11ac83d0243248887d19524026ed05dbe90ab595 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2241055 Commit-Queue: Louis Collard <louiscollard@chromium.org> Auto-Submit: Louis Collard <louiscollard@chromium.org> Reviewed-by:
-
Devlin Cronin authored
Instead of "Ignore", use "Keep it". Bug: 1079364 Tbr: treib@chromium.org (per offline discussion) Change-Id: I8905d3576b129a064d124007fa1ed9317c280ed8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253367Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#780192}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/d610c751569a..3d25c77940d2 2020-06-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll dawn from 3f4f35661151 to 581c407cd20b (3 revisions) 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 djsollen@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: djsollen@google.com Change-Id: Id64378920af5d7df7add8db5d33af06f3c622f9a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253010Reviewed-by:
-
David Bertoni authored
This CL moves the test to the interactive_ui_tests test suite. This fixes the flakiness with the test since the main window needs focus for the test to work properly. Bug: 477691 Change-Id: I0d8ab99a80a47c11ed9cc676a5fde23fa21b4c26 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252600Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: David Bertoni <dbertoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#780190}
-
Henrik Boström authored
The ThermalResource reports kOveruse or kUnderuse every 10 seconds while it has a registered listener and the DeviceThermalMeasurement is known. Because OnThermalMeasurement() only happens when the thermal state changes, repeated kOveruse is needed to adapt multiple steps. Based on [1] and manual observations, we do not want to adapt if thermals are kNominal or kFair so we map these to kUnderuse. But if thermals are kSerious or kCritical this is a strong signal from the OS that "corrective action" or "immediate corrective action" is needed. It can easily take a minute before the thermal state changes after load distribution has changed, so the effects of ThermalResource is likely to result in either maximally adapted or not adapted at all. The repeated interval of 10 seconds was somewhat arbitrarily chosen but was chosen as a tradeoff between giving the OS time to measure the new load and not making the resource too spammy. This CL implements and tests the resource, but does not wire it up to the webrtc::PeerConnection or RTCPeerConnectionHandler::OnThermalMeasurement(). This will be done in a separate CL. [1] https://developer.apple.com/library/archive/documentation/Performance/Conceptual/power_efficiency_guidelines_osx/RespondToThermalStateChanges.html Bug: chromium:1094844 Change-Id: I7dee0842ba786babbacb7e9d19acb58a1c9daa05 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2247769Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Mirko Bonadei <mbonadei@chromium.org> Commit-Queue: Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#780189}
-
Sophey Dong authored
Bug: 1079467 Change-Id: I492cd2771ddac88710a9219b8335eafdccf707c1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2246146 Commit-Queue: Sophey Dong <sophey@chromium.org> Reviewed-by:
Tanya Gupta <tgupta@chromium.org> Reviewed-by:
Lijin Shen <lazzzis@google.com> Reviewed-by:
Kyle Milka <kmilka@chromium.org> Cr-Commit-Position: refs/heads/master@{#780188}
-
Robbie Gibson authored
This adds CacheStorage blocking to the list of other storage methods blocked as part of the cookie blocking process on iOS. To more closely emulate desktop, it returns a rejected promise with the same error message. Bug: 1085123 Change-Id: If6ceed88e601b594c74c2580c26f8a183c9fba7f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2247883Reviewed-by:
dpapad <dpapad@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Robbie Gibson <rkgibson@google.com> Cr-Commit-Position: refs/heads/master@{#780187}
-
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-linux-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: I372835fba2738adaef15f3324a471870c338649d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253211Reviewed-by:
-
calamity authored
This reverts commit f248d2a2. Reason for revert: Suspected of causing failures for AuraWindowVideoCaptureDeviceBrowserTest.* https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/19215 Original change's description: > When Serializing the CopyOutputRequest use the threadpool for expensive copies. > > Currently when Chrome requests a screenshot we send a CopyOutputRequest > from the browser. However while taking this screenshot we need to > transfer it over a mojo pipe. This serialization takes 100-200 ms an > example traces in the linked bug on the VizCompositor thread and then > another 100-200 ms to deserialize it on the CrBrowserMain. This can > block high priority tasks which delay input handling causing visible > jank to users. > > We fix this by doing the expensive copies on the ThreadPool, most other > copy tasks only take a couple ms so this removes most of the pain > relatively easily. > > Tests had to be updated to know to wait for > CopyOutputResult::SendResult to complete async. > > > Bug: internal b/152858656 > Change-Id: I114d65894c80742309bc2a02fdd2a0a7ad56dae7 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2201762 > Commit-Queue: Stephen Nusko <nuskos@chromium.org> > Reviewed-by: Eric Seckler <eseckler@chromium.org> > Reviewed-by: vmpstr <vmpstr@chromium.org> > Reviewed-by: Mitsuru Oshima (slow:gardening) <oshima@chromium.org> > Reviewed-by: Chris Palmer <palmer@chromium.org> > Reviewed-by: Stephen Nusko <nuskos@chromium.org> > Auto-Submit: Stephen Nusko <nuskos@chromium.org> > Cr-Commit-Position: refs/heads/master@{#779930} TBR=palmer@chromium.org,oshima@chromium.org,vmpstr@chromium.org,eseckler@chromium.org,nuskos@chromium.org Change-Id: I7c3a2011728b442580dd2c2da4816c0426649a4b No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: internal b/152858656 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253678Reviewed-by:
-
Chromium WPT Sync authored
Using wpt-import in Chromium c6db4598. 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: hbos@chromium.org, hta@chromium.org: external/wpt/webrtc NOAUTOREVERT=true TBR=robertma@google.com No-Export: true Change-Id: Ib68cda916467447de5f5371f59c757b42e312a45 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2252881Reviewed-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@{#780184}
-
Thanh Nguyen authored
If a token is skipped during matching process, consider it as no match Bug: 990684 Change-Id: Ib9850afdf542c48d37202ddb085f39c6fa711fbb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249264 Commit-Queue: Thanh Nguyen <thanhdng@chromium.org> Reviewed-by:
-
Hiroki Sato authored
CL:2244656 resolved old dependencies and specific include rule for this directory is no longer needed. Test: No pre-upload warnings. build passes. Change-Id: Id70a2cb1886bc2e881b4896fc7b04efbdf74b002 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2249262 Auto-Submit: Hiroki Sato <hirokisato@chromium.org> Commit-Queue: Sara Kato <sarakato@chromium.org> Reviewed-by:
Sara Kato <sarakato@chromium.org> Cr-Commit-Position: refs/heads/master@{#780182}
-
John Abd-El-Malek authored
Bug: 1096390 Change-Id: I83d9062895cdc29c6f21b815105123b376472085 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2251054 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#780181}
-
chromium-autoroll authored
https://chromium.googlesource.com/openscreen.git/+log/1cdd473e5bca..ec44eb332577 2020-06-19 rwkeane@google.com mDNS: Add new OPT record type If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/openscreen-chromium Please CC jophba@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Bug: None Tbr: jophba@chromium.org Change-Id: I68e242aed351d24fd3716c3dbc20218727877919 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253519Reviewed-by:
-
Joel Hockey authored
Change-Id: Ieab59bdd9f9568977b570aa108b50759f881a787 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2251572 Commit-Queue: Joel Hockey <joelhockey@chromium.org> Commit-Queue: Daniel Ng <danielng@google.com> Auto-Submit: Joel Hockey <joelhockey@chromium.org> Reviewed-by:
Daniel Ng <danielng@google.com> Cr-Commit-Position: refs/heads/master@{#780179}
-
Joel Hockey authored
Bug: b/158509828 Change-Id: Ic900dd5381396f7fc6e93c7e77ddf506f2eacfa7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2251863 Auto-Submit: Joel Hockey <joelhockey@chromium.org> Commit-Queue: Daniel Ng <danielng@google.com> Reviewed-by:
-
