Use webrtc::DesktopCapturer for screen capturer implementation.

media/video/capture/screen/differ.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/video/capture/screen/differ.h"
 
 #include "base/logging.h"
 #include "media/video/capture/screen/differ_block.h"
 
 namespace media {
 
 Differ::Differ(int width, int height, int bpp, int stride) {
   // Dimensions of screen.
   width_ = width;
   height_ = height;
   bytes_per_pixel_ = bpp;
   bytes_per_row_ = stride;
 
   // Calc number of blocks (full and partial) required to cover entire image.
   // One additional row/column is added as a boundary on the right & bottom.
   diff_info_width_ = ((width_ + kBlockSize - 1) / kBlockSize) + 1;
   diff_info_height_ = ((height_ + kBlockSize - 1) / kBlockSize) + 1;
   diff_info_size_ = diff_info_width_ * diff_info_height_ * sizeof(DiffInfo);
   diff_info_.reset(new DiffInfo[diff_info_size_]);
 }
 
 Differ::~Differ() {}
 
 void Differ::CalcDirtyRegion(const void* prev_buffer, const void* curr_buffer,
-                             SkRegion* region) {
+                             webrtc::DesktopRegion* region) {
   if (!region) {

[alexeypa (please no reviews), 2013/05/08 22:24:59]

The whole purpose of this method is to return |region|. The caller should not be
able to pass NULL. Remove this check.

[Sergey Ulanov, 2013/05/09 18:49:02]

Done

     return;
   }
-  region->setEmpty();
 
   if (!prev_buffer || !curr_buffer) {

[alexeypa (please no reviews), 2013/05/08 22:24:59]

It looks like the callers make sure that |prev_buffer| and |curr_buffer| are not
NULL. Remove this check.

[Sergey Ulanov, 2013/05/09 18:49:02]

Done.

     return;
   }
 
   // Identify all the blocks that contain changed pixels.
   MarkDirtyBlocks(prev_buffer, curr_buffer);
 
   // Now that we've identified the blocks that have changed, merge adjacent
   // blocks to minimize the number of rects that we return.
   MergeBlocks(region);
 }
@@ skipping 77 matching lines @@
   int width_bytes = width * bytes_per_pixel_;
   for (int y = 0; y < height; y++) {
     if (memcmp(prev_buffer, curr_buffer, width_bytes) != 0)
       return 1;
     prev_buffer += bytes_per_row_;
     curr_buffer += bytes_per_row_;
   }
   return 0;
 }
 
-void Differ::MergeBlocks(SkRegion* region) {
+void Differ::MergeBlocks(webrtc::DesktopRegion* region) {
   DCHECK(region);

[alexeypa (please no reviews), 2013/05/08 22:24:59]

nit: remove DCHECK. |region->Clear()| will crash if |region| is NULL just fine.

[Sergey Ulanov, 2013/05/09 18:49:02]

Done.

-  region->setEmpty();
+  region->Clear();
 
   uint8* diff_info_row_start = static_cast<uint8*>(diff_info_.get());
   int diff_info_stride = diff_info_width_ * sizeof(DiffInfo);
 
   for (int y = 0; y < diff_info_height_; y++) {
     uint8* diff_info = diff_info_row_start;
     for (int x = 0; x < diff_info_width_; x++) {
       if (*diff_info != 0) {
         // We've found a modified block. Look at blocks to the right and below
         // to group this block with as many others as we can.
@@ skipping 41 matching lines @@
 
         // Add rect to list of dirty rects.
         width *= kBlockSize;
         if (left + width > width_) {
           width = width_ - left;
         }
         height *= kBlockSize;
         if (top + height > height_) {
           height = height_ - top;
         }
-        region->op(SkIRect::MakeXYWH(left, top, width, height),
-                   SkRegion::kUnion_Op);
+        region->AddRect(
+            webrtc::DesktopRect::MakeXYWH(left, top, width, height));
       }
 
       // Increment to next block in this row.
       diff_info++;
     }
 
     // Go to start of next row.
     diff_info_row_start += diff_info_stride;
   }
 }
 
 }  // namespace media