Add missing DCHECK to validate HR.

This issue was found by a linter.

The value of 'hr' is ignored. Probably a nice idea to validate the CreateInstance success.

R=amit@chromium.org,tommi@chromium.org
BUG=

Review URL: https://chromiumcodereview.appspot.com/21948002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@218662 0039d316-1c4b-4281-b951-d872f2087c98

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 (FAILED(hr))
+    return hr;
   hr = CallbackImpl::OnStopBinding(hresult, error);
   ReleaseBind();
   return hr;

chrome_frame/urlmon_moniker.cc

@@ -243,6 +243,7 @@ HRESULT MonikerPatch::BindToStorage(IMoniker_BindToStorage_Fn original,
   CComObject<BSCBStorageBind>* callback = NULL;
   if (ShouldWrapCallback(me, iid, bind_ctx)) {
     hr = CComObject<BSCBStorageBind>::CreateInstance(&callback);
+    DCHECK(SUCCEEDED(hr));
     auto_release_callback = callback;
     DCHECK_EQ(callback->m_dwRef, 1);
     hr = callback->Initialize(me, bind_ctx);