Refactor ZLib and Verbatim encoders.

remoting/host/capturer_linux.cc

Index: remoting/host/capturer_linux.cc
diff --git a/remoting/host/capturer_linux.cc b/remoting/host/capturer_linux.cc
index 70c956d32e41484c332685d6ccaa4c9e75809b9f..f18af5a3f56d5f48119e6349233bcfd8efbd1d8d 100644
--- a/remoting/host/capturer_linux.cc
+++ b/remoting/host/capturer_linux.cc
@@ -210,7 +210,7 @@ void CapturerLinuxPimpl::CalculateInvalidRects() {
         gfx::Rect damage_rect(event->area.x, event->area.y, event->area.width,
                               event->area.height);
         invalid_rects.insert(damage_rect);
-        VLOG(3) << "Damage receved for rect at ("
+        LOG(WARNING) << "Damage receved for rect at ("

[garykac, 2010/12/02 19:51:31]

received

                 << damage_rect.x() << "," << damage_rect.y() << ") size ("
                 << damage_rect.width() << "," << damage_rect.height() << ")";
       }
@@ -221,7 +221,7 @@ void CapturerLinuxPimpl::CalculateInvalidRects() {
 
   if (capture_fullscreen_) {
     capturer_->InvalidateFullScreen();
-    capture_fullscreen_ = false;
+    //    capture_fullscreen_ = false;

[Alpha Left Google, 2010/12/02 19:39:55]

commented out code?

[garykac, 2010/12/02 19:51:31]

Why is this commented out?  Does it need a TODO?

[Sergey Ulanov, 2010/12/03 00:17:21]

This wasn't meant to be part of this CL (Linux capturer is currently broken, so
I had to comment this line to workaround it). Reverted now

   } else {
     capturer_->InvalidateRects(invalid_rects);
   }