Vulkan: Remove RendererVk::getNextSubmitFence.

src/libANGLE/renderer/vulkan/RendererVk.cpp

@@ -538,10 +538,6 @@ void RendererVk::onDestroy(vk::Context *context)
         std::lock_guard<std::mutex> lock(mFenceRecyclerMutex);
         mFenceRecycler.destroy(mDevice);
     }
-    {
-        std::lock_guard<decltype(mNextSubmitFenceMutex)> lock(mNextSubmitFenceMutex);
-        mNextSubmitFence.reset(mDevice);
-    }
 
     mPipelineCache.destroy(mDevice);
     mSamplerCache.destroy(this);
@@ -2291,30 +2287,6 @@ angle::Result RendererVk::newSharedFence(vk::Context *context,
     return angle::Result::Continue;
 }
 
-// Return a shared fence to be used for the next submit
-// Fence may be shared with a Sync object.
-// reset indicates that nextSubmitFence should be reset before returning. This ensures that the next
-// request for a submit fence gets a fresh fence.
-// TODO: https://issuetracker.google.com/issues/170312581 - move to CommandProcessor as part of
-// fence ownership follow-up task.
-angle::Result RendererVk::getNextSubmitFence(vk::Shared<vk::Fence> *sharedFenceOut, bool reset)
-{
-    std::lock_guard<decltype(mNextSubmitFenceMutex)> lock(mNextSubmitFenceMutex);
-    if (!mNextSubmitFence.isReferenced())
-    {
-        ANGLE_TRY(newSharedFence(&mCommandProcessor, &mNextSubmitFence));
-    }
-
-    ASSERT(!sharedFenceOut->isReferenced());
-    sharedFenceOut->copy(getDevice(), mNextSubmitFence);
-
-    if (reset)
-    {
-        resetSharedFence(&mNextSubmitFence);
-    }
-    return angle::Result::Continue;
-}
-
 template <VkFormatFeatureFlags VkFormatProperties::*features>
 VkFormatFeatureFlags RendererVk::getFormatFeatureBits(VkFormat format,
                                                       const VkFormatFeatureFlags featureBits) const

src/libANGLE/renderer/vulkan/RendererVk.h

@@ -190,8 +190,6 @@ class RendererVk : angle::NonCopyable
         sharedFenceIn->resetAndRecycle(&mFenceRecycler);
     }
 
-    angle::Result getNextSubmitFence(vk::Shared<vk::Fence> *sharedFenceOut, bool reset);
-
     template <typename... ArgsT>
     void collectGarbageAndReinit(vk::SharedResourceUse *use, ArgsT... garbageIn)
     {
@@ -450,10 +448,6 @@ class RendererVk : angle::NonCopyable
 
     // Async Command Queue
     vk::CommandProcessor mCommandProcessor;
-    // mNextSubmitFence is the fence that's going to be signaled at the next submission.  This is
-    // used to support SyncVk objects, which may outlive the context (as EGLSync objects).
-    vk::Shared<vk::Fence> mNextSubmitFence;
-    std::mutex mNextSubmitFenceMutex;
 
     // track whether we initialized (or released) glslang
     bool mGlslangInitialized;