Mac: Kill lots of AcceleratedWidget code (with fire)

content/browser/compositor/software_output_device_mac.mm

Index: content/browser/compositor/software_output_device_mac.mm
diff --git a/content/browser/compositor/software_output_device_mac.mm b/content/browser/compositor/software_output_device_mac.mm
index 5dacbae755c5eaa97a69fb4544135f1a4da80bf2..191a15b86f08a6f3b0d194c8df909b1589b00b12 100644
--- a/content/browser/compositor/software_output_device_mac.mm
+++ b/content/browser/compositor/software_output_device_mac.mm
@@ -6,22 +6,146 @@
 
 #include "content/browser/compositor/software_output_device_mac.h"
 
+#include "third_party/skia/include/core/SkCanvas.h"
 #include "ui/accelerated_widget_mac/accelerated_widget_mac.h"
 #include "ui/compositor/compositor.h"
+#include "ui/gfx/skia_util.h"
 
 namespace content {
 
 SoftwareOutputDeviceMac::SoftwareOutputDeviceMac(ui::Compositor* compositor)
-    : compositor_(compositor) {
-}
+    : compositor_(compositor), scale_factor_(1), current_index_(0) {}
 
 SoftwareOutputDeviceMac::~SoftwareOutputDeviceMac() {
 }
 
+void SoftwareOutputDeviceMac::Resize(const gfx::Size& pixel_size,
+                                     float scale_factor) {
+  if (pixel_size_ == pixel_size && scale_factor_ == scale_factor)
+    return;
+
+  pixel_size_ = pixel_size;
+  scale_factor_ = scale_factor;
+
+  DiscardBackbuffer();
+}
+
+void SoftwareOutputDeviceMac::CopyPreviousBufferDamage(
+    const gfx::Rect& new_damage_rect) {
+  TRACE_EVENT0("browser", "CopyPreviousBufferDamage");
+
+  // The region to copy is the region drawn last frame, minus the region that
+  // is drawn this frame.
+  SkRegion copy_region(
+      SkRegion(gfx::RectToSkIRect(previous_buffer_damage_rect_)));
+  bool copy_region_nonempty = copy_region.op(
+      gfx::RectToSkIRect(new_damage_rect), SkRegion::kDifference_Op);
+  previous_buffer_damage_rect_ = new_damage_rect;
+
+  if (!copy_region_nonempty)
+    return;
+
+  {
+    TRACE_EVENT0("browser", "IOSurfaceLock");
+    IOReturn io_result = IOSurfaceLock(
+        io_surfaces_[!current_index_],
+        kIOSurfaceLockReadOnly | kIOSurfaceLockAvoidSync, nullptr);
+    if (io_result) {
+      LOG(ERROR) << "Failed to lock previous IOSurface " << io_result;

[tapted, 2015/10/22 08:26:49]

nit: DLOG? (i.e. is this likely? There's some resistance to putting error
strings in the release build since it's so unlikely anyone will see them) More
elsewhere.

[ccameron, 2015/10/22 19:26:11]

Done.

+      return;
+    }
+  }
+
+  uint8_t* pixels = reinterpret_cast<uint8_t*>(
+      IOSurfaceGetBaseAddress(io_surfaces_[!current_index_]));
+  size_t pitch = IOSurfaceGetBytesPerRow(io_surfaces_[!current_index_]);
+  size_t bytes_per_element = 4;
+  for (SkRegion::Iterator it(copy_region); !it.done(); it.next()) {
+    const SkIRect& rect = it.rect();
+    canvas_->writePixels(
+        SkImageInfo::MakeN32Premul(rect.width(), rect.height()),
+        pixels + bytes_per_element * rect.x() + pitch * rect.y(), pitch,
+        rect.x(), rect.y());
+  }
+
+  {
+    TRACE_EVENT0("browser", "IOSurfaceUnlock");
+    IOReturn io_result = IOSurfaceUnlock(
+        io_surfaces_[!current_index_],
+        kIOSurfaceLockReadOnly | kIOSurfaceLockAvoidSync, nullptr);
+    if (io_result) {

[tapted, 2015/10/22 08:26:49]

nit: curlies not needed

[ccameron, 2015/10/22 19:26:11]

Done.

+      LOG(ERROR) << "Failed to unlock previous IOSurface " << io_result;
+    }
+  }
+}
+
+SkCanvas* SoftwareOutputDeviceMac::BeginPaint(
+    const gfx::Rect& new_damage_rect) {
+  for (int i = 0; i < 2; ++i) {
+    if (!io_surfaces_[i]) {
+      TRACE_EVENT0("browser", "IOSurfaceCreate");
+      unsigned pixelFormat = 'BGRA';
+      unsigned bytesPerElement = 4;
+      NSDictionary* options = @{
+        (id) kIOSurfaceWidth : @(pixel_size_.width()), (id)
+        kIOSurfaceHeight : @(pixel_size_.height()), (id)
+        kIOSurfacePixelFormat : @(pixelFormat), (id)
+        kIOSurfaceBytesPerElement : @(bytesPerElement),
+      };
+      io_surfaces_[i].reset(IOSurfaceCreate((CFDictionaryRef)options));

[tapted, 2015/10/22 08:26:49]

CFCast<CFDictionaryRef>(options) ? (from foundation_util.h)

[ccameron, 2015/10/22 19:26:11]

Oops -- forgot to remove the C cast. Done.

+    }
+    if (!io_surfaces_[i]) {
+      LOG(ERROR) << "Failed to allocateIOSurface";
+      return nullptr;
+    }
+  }
+
+  {
+    TRACE_EVENT0("browser", "IOSurfaceLock");
+    IOReturn io_result = IOSurfaceLock(io_surfaces_[current_index_],
+                                       kIOSurfaceLockAvoidSync, nullptr);
+    if (io_result) {
+      LOG(ERROR) << "Failed to lock IOSurface " << io_result;
+      return nullptr;
+    }
+  }
+
+  SkPMColor* pixels = reinterpret_cast<SkPMColor*>(
+      IOSurfaceGetBaseAddress(io_surfaces_[current_index_]));
+  size_t pitch = IOSurfaceGetBytesPerRow(io_surfaces_[current_index_]);
+
+  canvas_ = skia::AdoptRef(SkCanvas::NewRasterDirectN32(
+      pixel_size_.width(), pixel_size_.height(), pixels, pitch));
+
+  CopyPreviousBufferDamage(new_damage_rect);
+  return canvas_.get();
+}
+
 void SoftwareOutputDeviceMac::EndPaint() {
   SoftwareOutputDevice::EndPaint();
-  ui::AcceleratedWidgetMacGotSoftwareFrame(
-      compositor_->widget(), scale_factor_, surface_->getCanvas());
+  {
+    TRACE_EVENT0("browser", "IOSurfaceUnlock");
+    IOReturn io_result = IOSurfaceUnlock(io_surfaces_[current_index_],
+                                         kIOSurfaceLockAvoidSync, nullptr);
+    if (io_result) {

[tapted, 2015/10/22 08:26:49]

nit: curlies not needed

[ccameron, 2015/10/22 19:26:11]

Done.

+      LOG(ERROR) << "Failed to unlock IOSurface " << io_result;
+    }
+  }
+
+  canvas_ = nullptr;
+  ui::AcceleratedWidgetMacGotIOSurfaceFrame(compositor_->widget(),
+                                            io_surfaces_[current_index_],
+                                            pixel_size_, scale_factor_, false);
+
+  current_index_ += 1;

[tapted, 2015/10/22 08:26:49]

current_index_ = !current_index?

[ccameron, 2015/10/22 19:26:11]

Done.

+  current_index_ %= 2;
 }
 
+void SoftwareOutputDeviceMac::DiscardBackbuffer() {
+  for (int i = 0; i < 2; ++i)
+    io_surfaces_[i].reset();
+}
+
+void SoftwareOutputDeviceMac::EnsureBackbuffer() {}
+
 }  // namespace content