Bug 1570592 - Remove AsyncImagePipelineManager::GetNextExternalImageId() r=nical

Differential Revision: https://phabricator.services.mozilla.com/D40168

--HG--
extra : moz-landing-system : lando

gfx/layers/wr/AsyncImagePipelineManager.cpp

@@ -87,15 +87,6 @@ bool AsyncImagePipelineManager::GetAndResetWillGenerateFrame(
   return ret;
 }
 
-wr::ExternalImageId AsyncImagePipelineManager::GetNextExternalImageId() {
-  static uint32_t sNextId = 0;
-  ++sNextId;
-  MOZ_RELEASE_ASSERT(sNextId != UINT32_MAX);
-  // gecko allocates external image id as (IdNamespace:32bit +
-  // ResourceId:32bit). And AsyncImagePipelineManager uses IdNamespace = 0.
-  return wr::ToExternalImageId((uint64_t)sNextId);
-}
-
 void AsyncImagePipelineManager::AddPipeline(const wr::PipelineId& aPipelineId,
                                             WebRenderBridgeParent* aWrBridge) {
   if (mDestroyed) {

gfx/layers/wr/AsyncImagePipelineManager.h

@@ -120,8 +120,6 @@ class AsyncImagePipelineManager final {
   void SetWillGenerateFrame(wr::RenderRoot aRenderRoot);
   bool GetAndResetWillGenerateFrame(wr::RenderRoot aRenderRoot);
 
-  wr::ExternalImageId GetNextExternalImageId();
-
  private:
   void ProcessPipelineRendered(const wr::PipelineId& aPipelineId,
                                const wr::Epoch& aEpoch,