[Blink] Move unused_resources_reclaim_timer_
into CRPSharedImage
Cleanup from moving resource recycling into CanvasResourceProviderSI. Bug: 352263194 Change-Id: I6111d6cc4ef5886c0df153ddf5d4b3268df77762 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6440824 Reviewed-by: Vasiliy Telezhnikov <vasilyt@chromium.org> Commit-Queue: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/main@{#1444554}
This commit is contained in:

committed by
Chromium LUCI CQ

parent
1d44e395c7
commit
331c55a13c
third_party/blink/renderer/platform/graphics
@ -310,6 +310,9 @@ class CanvasResourceProviderSharedImage : public CanvasResourceProvider,
|
||||
}
|
||||
return !canvas_resources_.empty();
|
||||
}
|
||||
bool unused_resources_reclaim_timer_is_running_for_testing() const override {
|
||||
return unused_resources_reclaim_timer_.IsRunning();
|
||||
}
|
||||
scoped_refptr<gpu::ClientSharedImage>
|
||||
GetBackingClientSharedImageForExternalWrite(
|
||||
gpu::SyncToken* internal_access_sync_token,
|
||||
@ -950,6 +953,7 @@ class CanvasResourceProviderSharedImage : public CanvasResourceProvider,
|
||||
// When and if |resource_recycling_enabled_| is false, |canvas_resources_|
|
||||
// will only hold one resource at most.
|
||||
WTF::Vector<UnusedResource> canvas_resources_;
|
||||
base::OneShotTimer unused_resources_reclaim_timer_;
|
||||
bool resource_recycling_enabled_ = true;
|
||||
|
||||
// `raster_context_provider_` holds a reference on the shared
|
||||
|
@ -337,8 +337,8 @@ class PLATFORM_EXPORT CanvasResourceProvider
|
||||
gpu::SyncToken& completion_sync_token);
|
||||
|
||||
virtual bool HasUnusedResourcesForTesting() const { return false; }
|
||||
bool unused_resources_reclaim_timer_is_running_for_testing() const {
|
||||
return unused_resources_reclaim_timer_.IsRunning();
|
||||
virtual bool unused_resources_reclaim_timer_is_running_for_testing() const {
|
||||
return false;
|
||||
}
|
||||
constexpr static base::TimeDelta kUnusedResourceExpirationTime =
|
||||
base::Seconds(5);
|
||||
@ -397,7 +397,6 @@ class PLATFORM_EXPORT CanvasResourceProvider
|
||||
// CanvasResourceProviderSharedImage.
|
||||
int num_inflight_resources_ = 0;
|
||||
int max_inflight_resources_ = 0;
|
||||
base::OneShotTimer unused_resources_reclaim_timer_;
|
||||
|
||||
private:
|
||||
friend class FlushForImageListener;
|
||||
|
Reference in New Issue
Block a user