Move screen capturers from remoting/capturer to media/video/capturer/screen

media/video/capture/screen/screen_capturer_linux.cc

 // Copyright (c) 2012 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 "remoting/capturer/video_frame_capturer.h"
+#include "media/video/capture/screen/screen_capturer.h"
 
 #include <X11/Xlib.h>
 #include <X11/Xutil.h>
 #include <X11/extensions/Xdamage.h>
 #include <X11/extensions/Xfixes.h>
 
 #include <set>
 
 #include "base/basictypes.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/stl_util.h"
 #include "base/time.h"
-#include "remoting/capturer/capture_data.h"
-#include "remoting/capturer/differ.h"
-#include "remoting/capturer/linux/x_server_pixel_buffer.h"
-#include "remoting/capturer/mouse_cursor_shape.h"
-#include "remoting/capturer/video_frame.h"
-#include "remoting/capturer/video_frame_capturer_helper.h"
-#include "remoting/capturer/video_frame_queue.h"
+#include "media/video/capture/screen/differ.h"
+#include "media/video/capture/screen/linux/x_server_pixel_buffer.h"
+#include "media/video/capture/screen/mouse_cursor_shape.h"
+#include "media/video/capture/screen/screen_capture_data.h"
+#include "media/video/capture/screen/screen_capture_frame.h"
+#include "media/video/capture/screen/screen_capture_frame_queue.h"
+#include "media/video/capture/screen/screen_capturer_helper.h"
 
-namespace remoting {
+namespace media {
 
 namespace {
 
 // Default to false, since many systems have broken XDamage support - see
 // http://crbug.com/73423.
 static bool g_should_use_x_damage = false;
 
 static bool ShouldUseXDamage() {
   return g_should_use_x_damage;
 }
 
 // A class representing a full-frame pixel buffer.
-class VideoFrameLinux : public VideoFrame {
+class ScreenCaptureFrameLinux : public ScreenCaptureFrame {
  public:
-  explicit VideoFrameLinux(const SkISize& window_size);
-  virtual ~VideoFrameLinux();
+  explicit ScreenCaptureFrameLinux(const SkISize& window_size);
+  virtual ~ScreenCaptureFrameLinux();
 
  private:
   // Allocated pixel buffer.
   scoped_array<uint8> data_;
 
-  DISALLOW_COPY_AND_ASSIGN(VideoFrameLinux);
+  DISALLOW_COPY_AND_ASSIGN(ScreenCaptureFrameLinux);
 };
 
 // A class to perform video frame capturing for Linux.
-class VideoFrameCapturerLinux : public VideoFrameCapturer {
+class ScreenCapturerLinux : public ScreenCapturer {
  public:
-  VideoFrameCapturerLinux();
-  virtual ~VideoFrameCapturerLinux();
+  ScreenCapturerLinux();
+  virtual ~ScreenCapturerLinux();
 
   bool Init();  // TODO(ajwong): Do we really want this to be synchronous?
 
   // Capturer interface.
   virtual void Start(Delegate* delegate) OVERRIDE;
   virtual void Stop() OVERRIDE;
   virtual void InvalidateRegion(const SkRegion& invalid_region) OVERRIDE;
   virtual void CaptureFrame() OVERRIDE;
 
  private:
   void InitXDamage();
 
   // Read and handle all currently-pending XEvents.
   // In the DAMAGE case, process the XDamage events and store the resulting
-  // damage rectangles in the VideoFrameCapturerHelper.
+  // damage rectangles in the ScreenCapturerHelper.
   // In all cases, call ScreenConfigurationChanged() in response to any
   // ConfigNotify events.
   void ProcessPendingXEvents();
 
   // Capture the cursor image and notify the delegate if it was captured.
   void CaptureCursor();
 
-  // Capture screen pixels, and return the data in a new CaptureData object,
-  // to be freed by the caller.
-  // In the DAMAGE case, the VideoFrameCapturerHelper already holds the list of
-  // invalid rectangles from ProcessPendingXEvents().
-  // In the non-DAMAGE case, this captures the whole screen, then calculates
-  // some invalid rectangles that include any differences between this and the
-  // previous capture.
-  scoped_refptr<CaptureData> CaptureScreen();
+  // Capture screen pixels, and return the data in a new ScreenCaptureData
+  // object, to be freed by the caller. In the DAMAGE case, the
+  // ScreenCapturerHelper already holds the list of invalid rectangles from
+  // ProcessPendingXEvents(). In the non-DAMAGE case, this captures the whole
+  // screen, then calculates some invalid rectangles that include any
+  // differences between this and the previous capture.
+  scoped_refptr<ScreenCaptureData> CaptureScreen();
 
   // Called when the screen configuration is changed. |root_window_size|
   // specifies the most recent size of the root window.
   void ScreenConfigurationChanged(const SkISize& root_window_size);
 
   // Synchronize the current buffer with |last_buffer_|, by copying pixels from
   // the area of |last_invalid_rects|.
   // Note this only works on the assumption that kNumBuffers == 2, as
   // |last_invalid_rects| holds the differences from the previous buffer and
   // the one prior to that (which will then be the current buffer).
   void SynchronizeFrame();
 
   void DeinitXlib();
 
   // Capture a rectangle from |x_server_pixel_buffer_|, and copy the data into
   // |capture_data|.
-  void CaptureRect(const SkIRect& rect, CaptureData* capture_data);
+  void CaptureRect(const SkIRect& rect, ScreenCaptureData* capture_data);
 
   // We expose two forms of blitting to handle variations in the pixel format.
   // In FastBlit, the operation is effectively a memcpy.
-  void FastBlit(uint8* image, const SkIRect& rect, CaptureData* capture_data);
-  void SlowBlit(uint8* image, const SkIRect& rect, CaptureData* capture_data);
+  void FastBlit(uint8* image,
+                const SkIRect& rect,
+                ScreenCaptureData* capture_data);
+  void SlowBlit(uint8* image,
+                const SkIRect& rect,
+                ScreenCaptureData* capture_data);
 
   Delegate* delegate_;
 
   // X11 graphics context.
   Display* display_;
   GC gc_;
   Window root_window_;
 
   // Last known dimensions of the root window.
   SkISize root_window_size_;
 
   // XFixes.
   bool has_xfixes_;
   int xfixes_event_base_;
   int xfixes_error_base_;
 
   // XDamage information.
   bool use_damage_;
   Damage damage_handle_;
   int damage_event_base_;
   int damage_error_base_;
   XserverRegion damage_region_;
 
   // Access to the X Server's pixel buffer.
   XServerPixelBuffer x_server_pixel_buffer_;
 
   // A thread-safe list of invalid rectangles, and the size of the most
   // recently captured screen.
-  VideoFrameCapturerHelper helper_;
+  ScreenCapturerHelper helper_;
 
   // Queue of the frames buffers.
-  VideoFrameQueue queue_;
+  ScreenCaptureFrameQueue queue_;
 
   // Invalid region from the previous capture. This is used to synchronize the
   // current with the last buffer used.
   SkRegion last_invalid_region_;
 
   // |Differ| for use when polling for changes.
   scoped_ptr<Differ> differ_;
 
-  DISALLOW_COPY_AND_ASSIGN(VideoFrameCapturerLinux);
+  DISALLOW_COPY_AND_ASSIGN(ScreenCapturerLinux);
 };
 
-VideoFrameLinux::VideoFrameLinux(const SkISize& window_size) {
-  set_bytes_per_row(window_size.width() * CaptureData::kBytesPerPixel);
+ScreenCaptureFrameLinux::ScreenCaptureFrameLinux(const SkISize& window_size) {
+  set_bytes_per_row(window_size.width() * ScreenCaptureData::kBytesPerPixel);
   set_dimensions(window_size);
 
   size_t buffer_size = bytes_per_row() * window_size.height();
   data_.reset(new uint8[buffer_size]);
   set_pixels(data_.get());
 }
 
-VideoFrameLinux::~VideoFrameLinux() {
+ScreenCaptureFrameLinux::~ScreenCaptureFrameLinux() {
 }
 
-VideoFrameCapturerLinux::VideoFrameCapturerLinux()
+ScreenCapturerLinux::ScreenCapturerLinux()
     : delegate_(NULL),
       display_(NULL),
       gc_(NULL),
       root_window_(BadValue),
       root_window_size_(SkISize::Make(0, 0)),
       has_xfixes_(false),
       xfixes_event_base_(-1),
       xfixes_error_base_(-1),
       use_damage_(false),
       damage_handle_(0),
       damage_event_base_(-1),
       damage_error_base_(-1),
       damage_region_(0) {
   helper_.SetLogGridSize(4);
 }
 
-VideoFrameCapturerLinux::~VideoFrameCapturerLinux() {
+ScreenCapturerLinux::~ScreenCapturerLinux() {
   DeinitXlib();
 }
 
-bool VideoFrameCapturerLinux::Init() {
+bool ScreenCapturerLinux::Init() {
   // TODO(ajwong): We should specify the display string we are attaching to
   // in the constructor.
   display_ = XOpenDisplay(NULL);
   if (!display_) {
     LOG(ERROR) << "Unable to open display";
     return false;
   }
 
   root_window_ = RootWindow(display_, DefaultScreen(display_));
   if (root_window_ == BadValue) {
@@ skipping 30 matching lines @@
                             XFixesDisplayCursorNotifyMask);
   }
 
   if (ShouldUseXDamage()) {
     InitXDamage();
   }
 
   return true;
 }
 
-void VideoFrameCapturerLinux::InitXDamage() {
+void ScreenCapturerLinux::InitXDamage() {
   // Our use of XDamage requires XFixes.
   if (!has_xfixes_) {
     return;
   }
 
   // Check for XDamage extension.
   if (!XDamageQueryExtension(display_, &damage_event_base_,
                              &damage_error_base_)) {
     LOG(INFO) << "X server does not support XDamage.";
     return;
@@ skipping 17 matching lines @@
   if (!damage_region_) {
     XDamageDestroy(display_, damage_handle_);
     LOG(ERROR) << "Unable to create XFixes region.";
     return;
   }
 
   use_damage_ = true;
   LOG(INFO) << "Using XDamage extension.";
 }
 
-void VideoFrameCapturerLinux::Start(Delegate* delegate) {
+void ScreenCapturerLinux::Start(Delegate* delegate) {
   DCHECK(delegate_ == NULL);
 
   delegate_ = delegate;
 }
 
-void VideoFrameCapturerLinux::Stop() {
+void ScreenCapturerLinux::Stop() {
 }
 
-void VideoFrameCapturerLinux::InvalidateRegion(const SkRegion& invalid_region) {
+void ScreenCapturerLinux::InvalidateRegion(const SkRegion& invalid_region) {
   helper_.InvalidateRegion(invalid_region);
 }
 
-void VideoFrameCapturerLinux::CaptureFrame() {
+void ScreenCapturerLinux::CaptureFrame() {
   base::Time capture_start_time = base::Time::Now();
 
   // Process XEvents for XDamage and cursor shape tracking.
   ProcessPendingXEvents();
 
   // If the current buffer is from an older generation then allocate a new one.
   // Note that we can't reallocate other buffers at this point, since the caller
   // may still be reading from them.
   if (queue_.current_frame_needs_update()) {
-    scoped_ptr<VideoFrameLinux> buffer(new VideoFrameLinux(
+    scoped_ptr<ScreenCaptureFrameLinux> buffer(new ScreenCaptureFrameLinux(
         root_window_size_));
-    queue_.ReplaceCurrentFrame(buffer.PassAs<VideoFrame>());
+    queue_.ReplaceCurrentFrame(buffer.PassAs<ScreenCaptureFrame>());
   }
 
   // Refresh the Differ helper used by CaptureFrame(), if needed.
-  const VideoFrame* current_buffer = queue_.current_frame();
+  const ScreenCaptureFrame* current_buffer = queue_.current_frame();
   if (!use_damage_ && (
       !differ_.get() ||
       (differ_->width() != current_buffer->dimensions().width()) ||
       (differ_->height() != current_buffer->dimensions().height()) ||
       (differ_->bytes_per_row() != current_buffer->bytes_per_row()))) {
     differ_.reset(new Differ(current_buffer->dimensions().width(),
                              current_buffer->dimensions().height(),
-                             CaptureData::kBytesPerPixel,
+                             ScreenCaptureData::kBytesPerPixel,
                              current_buffer->bytes_per_row()));
   }
 
-  scoped_refptr<CaptureData> capture_data(CaptureScreen());
+  scoped_refptr<ScreenCaptureData> capture_data(CaptureScreen());
 
   // Swap the current & previous buffers ready for the next capture.
   last_invalid_region_ = capture_data->dirty_region();
 
   queue_.DoneWithCurrentFrame();
 
   capture_data->set_capture_time_ms(
       (base::Time::Now() - capture_start_time).InMillisecondsRoundedUp());
   delegate_->OnCaptureCompleted(capture_data);
 }
 
-void VideoFrameCapturerLinux::ProcessPendingXEvents() {
+void ScreenCapturerLinux::ProcessPendingXEvents() {
   // Find the number of events that are outstanding "now."  We don't just loop
   // on XPending because we want to guarantee this terminates.
   int events_to_process = XPending(display_);
   XEvent e;
 
   for (int i = 0; i < events_to_process; i++) {
     XNextEvent(display_, &e);
     if (use_damage_ && (e.type == damage_event_base_ + XDamageNotify)) {
       XDamageNotifyEvent* event = reinterpret_cast<XDamageNotifyEvent*>(&e);
       DCHECK(event->level == XDamageReportNonEmpty);
     } else if (e.type == ConfigureNotify) {
       const XConfigureEvent& event = e.xconfigure;
       ScreenConfigurationChanged(SkISize::Make(event.width, event.height));
     } else if (has_xfixes_ &&
                e.type == xfixes_event_base_ + XFixesCursorNotify) {
       XFixesCursorNotifyEvent* cne;
       cne = reinterpret_cast<XFixesCursorNotifyEvent*>(&e);
       if (cne->subtype == XFixesDisplayCursorNotify) {
         CaptureCursor();
       }
     } else {
       LOG(WARNING) << "Got unknown event type: " << e.type;
     }
   }
 }
 
-void VideoFrameCapturerLinux::CaptureCursor() {
+void ScreenCapturerLinux::CaptureCursor() {
   DCHECK(has_xfixes_);
 
   XFixesCursorImage* img = XFixesGetCursorImage(display_);
   if (!img) {
     return;
   }
 
   scoped_ptr<MouseCursorShape> cursor(new MouseCursorShape());
   cursor->size.set(img->width, img->height);
   cursor->hotspot.set(img->xhot, img->yhot);
 
   int total_bytes = cursor->size.width() * cursor->size.height() *
-      CaptureData::kBytesPerPixel;
+      ScreenCaptureData::kBytesPerPixel;
   cursor->data.resize(total_bytes);
 
   // Xlib stores 32-bit data in longs, even if longs are 64-bits long.
   unsigned long* src = img->pixels;
   uint32* dst = reinterpret_cast<uint32*>(string_as_array(&cursor->data));
   uint32* dst_end = dst + (img->width * img->height);
   while (dst < dst_end) {
     *dst++ = static_cast<uint32>(*src++);
   }
   XFree(img);
 
   delegate_->OnCursorShapeChanged(cursor.Pass());
 }
 
-scoped_refptr<CaptureData> VideoFrameCapturerLinux::CaptureScreen() {
-  VideoFrame* frame = queue_.current_frame();
-  scoped_refptr<CaptureData> capture_data(new CaptureData(
+scoped_refptr<ScreenCaptureData> ScreenCapturerLinux::CaptureScreen() {
+  ScreenCaptureFrame* frame = queue_.current_frame();
+  scoped_refptr<ScreenCaptureData> capture_data(new ScreenCaptureData(
       frame->pixels(), frame->bytes_per_row(), frame->dimensions()));
 
   // Pass the screen size to the helper, so it can clip the invalid region if it
   // expands that region to a grid.
   helper_.set_size_most_recent(capture_data->size());
 
   // In the DAMAGE case, ensure the frame is up-to-date with the previous frame
   // if any.  If there isn't a previous frame, that means a screen-resolution
   // change occurred, and |invalid_rects| will be updated to include the whole
   // screen.
@@ skipping 48 matching lines @@
       // full-screen notification after a screen-resolution change, so
       // this is done here.
       invalid_region.op(screen_rect, SkRegion::kUnion_Op);
     }
   }
 
   capture_data->mutable_dirty_region() = invalid_region;
   return capture_data;
 }
 
-void VideoFrameCapturerLinux::ScreenConfigurationChanged(
+void ScreenCapturerLinux::ScreenConfigurationChanged(
     const SkISize& root_window_size) {
   root_window_size_ = root_window_size;
 
   // Make sure the frame buffers will be reallocated.
   queue_.SetAllFramesNeedUpdate();
 
   helper_.ClearInvalidRegion();
   x_server_pixel_buffer_.Init(display_, root_window_size_);
 }
 
-void VideoFrameCapturerLinux::SynchronizeFrame() {
+void ScreenCapturerLinux::SynchronizeFrame() {
   // Synchronize the current buffer with the previous one since we do not
   // capture the entire desktop. Note that encoder may be reading from the
   // previous buffer at this time so thread access complaints are false
   // positives.
 
   // TODO(hclam): We can reduce the amount of copying here by subtracting
   // |capturer_helper_|s region from |last_invalid_region_|.
   // http://crbug.com/92354
   DCHECK(queue_.previous_frame());
 
-  VideoFrame* current = queue_.current_frame();
-  VideoFrame* last = queue_.previous_frame();
+  ScreenCaptureFrame* current = queue_.current_frame();
+  ScreenCaptureFrame* last = queue_.previous_frame();
   DCHECK_NE(current, last);
   for (SkRegion::Iterator it(last_invalid_region_); !it.done(); it.next()) {
     const SkIRect& r = it.rect();
     int offset = r.fTop * current->bytes_per_row() +
-        r.fLeft * CaptureData::kBytesPerPixel;
+        r.fLeft * ScreenCaptureData::kBytesPerPixel;
     for (int i = 0; i < r.height(); ++i) {
       memcpy(current->pixels() + offset, last->pixels() + offset,
-             r.width() * CaptureData::kBytesPerPixel);
-      offset += current->dimensions().width() * CaptureData::kBytesPerPixel;
+             r.width() * ScreenCaptureData::kBytesPerPixel);
+      offset += current->dimensions().width() *
+          ScreenCaptureData::kBytesPerPixel;
     }
   }
 }
 
-void VideoFrameCapturerLinux::DeinitXlib() {
+void ScreenCapturerLinux::DeinitXlib() {
   if (gc_) {
     XFreeGC(display_, gc_);
     gc_ = NULL;
   }
 
   x_server_pixel_buffer_.Release();
 
   if (display_) {
     if (damage_handle_)
       XDamageDestroy(display_, damage_handle_);
     if (damage_region_)
       XFixesDestroyRegion(display_, damage_region_);
     XCloseDisplay(display_);
     display_ = NULL;
     damage_handle_ = 0;
     damage_region_ = 0;
   }
 }
 
-void VideoFrameCapturerLinux::CaptureRect(const SkIRect& rect,
-                                          CaptureData* capture_data) {
+void ScreenCapturerLinux::CaptureRect(const SkIRect& rect,
+                                      ScreenCaptureData* capture_data) {
   uint8* image = x_server_pixel_buffer_.CaptureRect(rect);
   int depth = x_server_pixel_buffer_.GetDepth();
   int bpp = x_server_pixel_buffer_.GetBitsPerPixel();
   bool is_rgb = x_server_pixel_buffer_.IsRgb();
   if ((depth == 24 || depth == 32) && bpp == 32 && is_rgb) {
     DVLOG(3) << "Fast blitting";
     FastBlit(image, rect, capture_data);
   } else {
     DVLOG(3) << "Slow blitting";
     SlowBlit(image, rect, capture_data);
   }
 }
 
-void VideoFrameCapturerLinux::FastBlit(uint8* image, const SkIRect& rect,
-                                       CaptureData* capture_data) {
+void ScreenCapturerLinux::FastBlit(uint8* image, const SkIRect& rect,
+                                   ScreenCaptureData* capture_data) {
   uint8* src_pos = image;
   int src_stride = x_server_pixel_buffer_.GetStride();
   int dst_x = rect.fLeft, dst_y = rect.fTop;
 
   uint8* dst_pos = capture_data->data() + capture_data->stride() * dst_y;
-  dst_pos += dst_x * CaptureData::kBytesPerPixel;
+  dst_pos += dst_x * ScreenCaptureData::kBytesPerPixel;
 
   int height = rect.height();
-  int row_bytes = rect.width() * CaptureData::kBytesPerPixel;
+  int row_bytes = rect.width() * ScreenCaptureData::kBytesPerPixel;
   for (int y = 0; y < height; ++y) {
     memcpy(dst_pos, src_pos, row_bytes);
     src_pos += src_stride;
     dst_pos += capture_data->stride();
   }
 }
 
-void VideoFrameCapturerLinux::SlowBlit(uint8* image, const SkIRect& rect,
-                                       CaptureData* capture_data) {
+void ScreenCapturerLinux::SlowBlit(uint8* image, const SkIRect& rect,
+                                   ScreenCaptureData* capture_data) {
   int src_stride = x_server_pixel_buffer_.GetStride();
   int dst_x = rect.fLeft, dst_y = rect.fTop;
   int width = rect.width(), height = rect.height();
 
   unsigned int red_mask = x_server_pixel_buffer_.GetRedMask();
   unsigned int blue_mask = x_server_pixel_buffer_.GetBlueMask();
   unsigned int green_mask = x_server_pixel_buffer_.GetGreenMask();
   unsigned int red_shift = x_server_pixel_buffer_.GetRedShift();
   unsigned int blue_shift = x_server_pixel_buffer_.GetBlueShift();
   unsigned int green_shift = x_server_pixel_buffer_.GetGreenShift();
 
   unsigned int max_red = red_mask >> red_shift;
   unsigned int max_blue = blue_mask >> blue_shift;
   unsigned int max_green = green_mask >> green_shift;
 
   unsigned int bits_per_pixel = x_server_pixel_buffer_.GetBitsPerPixel();
 
   uint8* dst_pos = capture_data->data() + capture_data->stride() * dst_y;
   uint8* src_pos = image;
-  dst_pos += dst_x * CaptureData::kBytesPerPixel;
+  dst_pos += dst_x * ScreenCaptureData::kBytesPerPixel;
   // TODO(hclam): Optimize, perhaps using MMX code or by converting to
   // YUV directly
   for (int y = 0; y < height; y++) {
     uint32_t* dst_pos_32 = reinterpret_cast<uint32_t*>(dst_pos);
     uint32_t* src_pos_32 = reinterpret_cast<uint32_t*>(src_pos);
     uint16_t* src_pos_16 = reinterpret_cast<uint16_t*>(src_pos);
     for (int x = 0; x < width; x++) {
       // Dereference through an appropriately-aligned pointer.
       uint32_t pixel;
       if (bits_per_pixel == 32)
         pixel = src_pos_32[x];
       else if (bits_per_pixel == 16)
         pixel = src_pos_16[x];
       else
         pixel = src_pos[x];
       uint32_t r = (((pixel & red_mask) >> red_shift) * 255) / max_red;
       uint32_t b = (((pixel & blue_mask) >> blue_shift) * 255) / max_blue;
       uint32_t g = (((pixel & green_mask) >> green_shift) * 255) / max_green;
       // Write as 32-bit RGB.
       dst_pos_32[x] = r << 16 | g << 8 | b;
     }
     dst_pos += capture_data->stride();
     src_pos += src_stride;
   }
 }
 
 }  // namespace
 
 // static
-scoped_ptr<VideoFrameCapturer> VideoFrameCapturer::Create() {
-  scoped_ptr<VideoFrameCapturerLinux> capturer(new VideoFrameCapturerLinux());
+scoped_ptr<ScreenCapturer> ScreenCapturer::Create() {
+  scoped_ptr<ScreenCapturerLinux> capturer(new ScreenCapturerLinux());
   if (!capturer->Init())
     capturer.reset();
-  return capturer.PassAs<VideoFrameCapturer>();
+  return capturer.PassAs<ScreenCapturer>();
 }
 
 // static
-scoped_ptr<VideoFrameCapturer> VideoFrameCapturer::CreateWithFactory(
+scoped_ptr<ScreenCapturer> ScreenCapturer::CreateWithFactory(
     SharedBufferFactory* shared_buffer_factory) {
   NOTIMPLEMENTED();
-  return scoped_ptr<VideoFrameCapturer>();
+  return scoped_ptr<ScreenCapturer>();
 }
 
 // static
-void VideoFrameCapturer::EnableXDamage(bool enable) {
+void ScreenCapturer::EnableXDamage(bool enable) {
   g_should_use_x_damage = enable;
 }
 
-}  // namespace remoting
+}  // namespace media