Several minor improvements of DirectX capturer

webrtc/modules/desktop_capture/win/dxgi_output_duplicator.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 129 matching lines @@
     return false;
   }
 
   // We need to merge updated region with the one from last frame, since current
   // frame contains the content one frame before. Note, this is for double
   // buffering implementation, as what we have in ScreenCapturerWinDirectx. If
   // a consumer uses single buffering, we should clear context->updated_region
   // after it has been merged to updated_region.
   DesktopRegion updated_region;
   updated_region.Swap(&context->updated_region);
-  if (error.Error() == S_OK && frame_info.AccumulatedFrames > 0) {
+  if (error.Error() == S_OK &&
+      frame_info.AccumulatedFrames > 0 &&
+      resource) {
     DetectUpdatedRegion(frame_info, offset, &context->updated_region);
     if (!texture_->CopyFrom(frame_info, resource.Get(),
                             context->updated_region)) {
       return false;
     }
     SpreadContextChange(context);
     updated_region.AddRegion(context->updated_region);
 
     const DesktopFrame& source = texture_->AsDesktopFrame();
     for (DesktopRegion::Iterator it(updated_region); !it.IsAtEnd();
@@ skipping 56 matching lines @@
 
   if (metadata.capacity() < frame_info.TotalMetadataBufferSize) {
     metadata.clear();  // Avoid data copy
     metadata.reserve(frame_info.TotalMetadataBufferSize);
   }
 
   UINT buff_size = 0;
   DXGI_OUTDUPL_MOVE_RECT* move_rects =
       reinterpret_cast<DXGI_OUTDUPL_MOVE_RECT*>(metadata.data());
   size_t move_rects_count = 0;
-  _com_error error = _com_error(duplication_->GetFrameMoveRects(
-      static_cast<UINT>(metadata.capacity()), move_rects, &buff_size));
+  _com_error error = duplication_->GetFrameMoveRects(
+      static_cast<UINT>(metadata.capacity()), move_rects, &buff_size);
   if (error.Error() != S_OK) {
     LOG(LS_ERROR) << "Failed to get move rectangles, error "
                   << error.ErrorMessage() << ", code " << error.Error();
     return false;
   }
   move_rects_count = buff_size / sizeof(DXGI_OUTDUPL_MOVE_RECT);
 
   RECT* dirty_rects = reinterpret_cast<RECT*>(metadata.data() + buff_size);
   size_t dirty_rects_count = 0;
-  error = _com_error(duplication_->GetFrameDirtyRects(
+  error = duplication_->GetFrameDirtyRects(
       static_cast<UINT>(metadata.capacity()) - buff_size, dirty_rects,
-      &buff_size));
+      &buff_size);
   if (error.Error() != S_OK) {
     LOG(LS_ERROR) << "Failed to get dirty rectangles, error "
                   << error.ErrorMessage() << ", code " << error.Error();
     return false;
   }
   dirty_rects_count = buff_size / sizeof(RECT);
 
   while (move_rects_count > 0) {
     updated_region->AddRect(DesktopRect::MakeXYWH(
         move_rects->SourcePoint.x, move_rects->SourcePoint.y,
@@ skipping 57 matching lines @@
 }
 
 DesktopRect DxgiOutputDuplicator::TargetRect(DesktopRect rect,
                                              DesktopVector offset) {
   rect = SourceRect(rect);
   rect.Translate(offset);
   return rect;
 }
 
 }  // namespace webrtc