Add missing DCHECK to validate HR.

chrome_frame/urlmon_bind_status_callback.cc

Index: chrome_frame/urlmon_bind_status_callback.cc
diff --git a/chrome_frame/urlmon_bind_status_callback.cc b/chrome_frame/urlmon_bind_status_callback.cc
index 006e20e923123dc860add889ae9aabaafb974256..bfaced7ff8f38163e6f73e31198b59407c428299 100644
--- a/chrome_frame/urlmon_bind_status_callback.cc
+++ b/chrome_frame/urlmon_bind_status_callback.cc
@@ -320,6 +320,8 @@ STDMETHODIMP BSCBStorageBind::OnStopBinding(HRESULT hresult, LPCWSTR error) {
   ExceptionBarrier barrier;
 
   HRESULT hr = MayPlayBack(BSCF_LASTDATANOTIFICATION);
+  if (hr != S_OK)

[robertshield, 2013/08/02 18:35:42]

FAILED(hr)

[tommi (sloooow) - chröme, 2013/08/04 12:43:12]

No chance of S_FALSE?

[etienneb, 2013/08/05 13:56:26]

I don't think S_FALSE can be here.
At some place, the returns value of MayPlayBack is just ignored.

  if (!is_interesting) {
    // Play back report progress so far.
    MayPlayBack(flags);
    return CallbackImpl::OnDataAvailable(flags, size, format_etc, stgmed);
  }

On 2013/08/04 12:43:12, tommi wrote:

+    return hr;
   hr = CallbackImpl::OnStopBinding(hresult, error);
   ReleaseBind();
   return hr;