corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1736266 - Remove unused DidCompositeObserver. r=sotaro

Browse Source 
This was added in bug 1320864 and looks like it was used for plugins.

Differential Revision: https://phabricator.services.mozilla.com/D128735
This commit is contained in:
Jeff Muizelaar
2021-10-19 00:55:30 +00:00
parent 5a7124f219
commit e9fd7de01c
2 changed files with 0 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
gfx/layers/wr/WebRenderLayerManager.cpp
View File

@@ -602,11 +602,6 @@ void WebRenderLayerManager::DidComposite(
mTransactionIdAllocator->NotifyTransactionCompleted(aTransactionId); mTransactionIdAllocator->NotifyTransactionCompleted(aTransactionId);
} }
} }
// These observers fire whether or not we were in a transaction.
for (size_t i = 0; i < mDidCompositeObservers.Length(); i++) {
mDidCompositeObservers[i]->DidComposite();
}
} }
void WebRenderLayerManager::ClearCachedResources() { void WebRenderLayerManager::ClearCachedResources() {
@@ -673,18 +668,6 @@ TransactionId WebRenderLayerManager::GetLastTransactionId() {
return mLatestTransactionId; return mLatestTransactionId;
} }
void WebRenderLayerManager::AddDidCompositeObserver(
DidCompositeObserver* aObserver) {
if (!mDidCompositeObservers.Contains(aObserver)) {
mDidCompositeObservers.AppendElement(aObserver);
}
}
void WebRenderLayerManager::RemoveDidCompositeObserver(
DidCompositeObserver* aObserver) {
mDidCompositeObservers.RemoveElement(aObserver);
}
void WebRenderLayerManager::FlushRendering(wr::RenderReasons aReasons) { void WebRenderLayerManager::FlushRendering(wr::RenderReasons aReasons) {
CompositorBridgeChild* cBridge = GetCompositorBridgeChild(); CompositorBridgeChild* cBridge = GetCompositorBridgeChild();
if (!cBridge) { if (!cBridge) {

9
gfx/layers/wr/WebRenderLayerManager.h
View File

@@ -54,10 +54,6 @@ class WebRenderBridgeChild;
class WebRenderParentCommand; class WebRenderParentCommand;
class TransactionIdAllocator; class TransactionIdAllocator;
class LayerUserData; class LayerUserData;
class DidCompositeObserver {
public:
virtual void DidComposite() = 0;
};
class WebRenderLayerManager final : public WindowRenderer { class WebRenderLayerManager final : public WindowRenderer {
typedef nsTArray<RefPtr<Layer>> LayerRefArray; typedef nsTArray<RefPtr<Layer>> LayerRefArray;
@@ -113,9 +109,6 @@ class WebRenderLayerManager final : public WindowRenderer {
void SetTransactionIdAllocator(TransactionIdAllocator* aAllocator); void SetTransactionIdAllocator(TransactionIdAllocator* aAllocator);
TransactionId GetLastTransactionId(); TransactionId GetLastTransactionId();
void AddDidCompositeObserver(DidCompositeObserver* aObserver);
void RemoveDidCompositeObserver(DidCompositeObserver* aObserver);
void FlushRendering(wr::RenderReasons aReasons) override; void FlushRendering(wr::RenderReasons aReasons) override;
void WaitOnTransactionProcessed() override; void WaitOnTransactionProcessed() override;
@@ -234,8 +227,6 @@ class WebRenderLayerManager final : public WindowRenderer {
RefPtr<TransactionIdAllocator> mTransactionIdAllocator; RefPtr<TransactionIdAllocator> mTransactionIdAllocator;
TransactionId mLatestTransactionId; TransactionId mLatestTransactionId;
nsTArray<DidCompositeObserver*> mDidCompositeObservers;
gfx::UserData mUserData; gfx::UserData mUserData;
// This holds the scroll data that we need to send to the compositor for // This holds the scroll data that we need to send to the compositor for