DXVA: Avoid infinite recursion

I'm guessing this shouldn't happen, but apparently it did anyways.
Let's have a max number of retries...

BUG=715707
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2846343002
Cr-Commit-Position: refs/heads/master@{#468561}

media/gpu/dxva_video_decode_accelerator_win.cc

@@ -1778,6 +1778,10 @@ void DXVAVideoDecodeAccelerator::DoDecode(const gfx::ColorSpace& color_space) {
   if (D3D11Device())
     g_last_device_removed_reason = D3D11Device()->GetDeviceRemovedReason();
 
+  base::win::ScopedComPtr<IMFSample> output_sample;
+  int retries = 10;
+  while (true) {
+    output_sample.Reset();
     MFT_OUTPUT_DATA_BUFFER output_data_buffer = {0};
     DWORD status = 0;
     HRESULT hr;
@@ -1793,7 +1797,6 @@ void DXVAVideoDecodeAccelerator::DoDecode(const gfx::ColorSpace& color_space) {
       DVLOG(1) << "Got events from ProcessOuput, but discarding";
       events->Release();
     }
-  base::win::ScopedComPtr<IMFSample> output_sample;
     output_sample.Attach(output_data_buffer.pSample);
     if (FAILED(hr)) {
       // A stream change needs further ProcessInput calls to get back decoder
@@ -1807,9 +1810,14 @@ void DXVAVideoDecodeAccelerator::DoDecode(const gfx::ColorSpace& color_space) {
           NOTREACHED() << "Failed to set decoder output media type to NV12";
           SetState(kStopped);
         } else {
-        DVLOG(1) << "Received output format change from the decoder."
-                    " Recursively invoking DoDecode";
-        DoDecode(color_space);
+          if (retries-- > 0) {
+            DVLOG(1) << "Received format change from the decoder, retrying.";
+            continue;  // Retry
+          } else {
+            RETURN_AND_NOTIFY_ON_FAILURE(
+                false, "Received too many format changes from decoder.",
+                PLATFORM_FAILURE, );
+          }
         }
         return;
       } else if (hr == MF_E_TRANSFORM_NEED_MORE_INPUT) {
@@ -1822,6 +1830,9 @@ void DXVAVideoDecodeAccelerator::DoDecode(const gfx::ColorSpace& color_space) {
         return;
       }
     }
+
+    break;  // No more retries needed.
+  }
   TRACE_EVENT_ASYNC_END0("gpu", "DXVAVideoDecodeAccelerator.Decoding", this);
 
   TRACE_COUNTER1("DXVA Decoding", "TotalPacketsBeforeDecode",