Consolidate Windows bitmap and DC code

skia/ext/skia_utils_win.cc

 // Copyright (c) 2006-2008 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 "skia/ext/skia_utils_win.h"
 
 #include <stddef.h>
 #include <windows.h>
 
+#include "base/debug/gdi_debug_util_win.h"
+#include "base/win/win_util.h"
 #include "third_party/skia/include/core/SkRect.h"
+#include "third_party/skia/include/core/SkSurface.h"
 #include "third_party/skia/include/core/SkTypes.h"
 #include "third_party/skia/include/effects/SkGradientShader.h"
 
 namespace {
 
 static_assert(offsetof(RECT, left) == offsetof(SkIRect, fLeft), "o1");
 static_assert(offsetof(RECT, top) == offsetof(SkIRect, fTop), "o2");
 static_assert(offsetof(RECT, right) == offsetof(SkIRect, fRight), "o3");
 static_assert(offsetof(RECT, bottom) == offsetof(SkIRect, fBottom), "o4");
 static_assert(sizeof(RECT().left) == sizeof(SkIRect().fLeft), "o5");
 static_assert(sizeof(RECT().top) == sizeof(SkIRect().fTop), "o6");
 static_assert(sizeof(RECT().right) == sizeof(SkIRect().fRight), "o7");
 static_assert(sizeof(RECT().bottom) == sizeof(SkIRect().fBottom), "o8");
 static_assert(sizeof(RECT) == sizeof(SkIRect), "o9");
 
+HBITMAP CreateHBitmap(int width, int height) {
+  // CreateDIBSection appears to get unhappy if we create an empty bitmap, so
+  // just create a minimal bitmap.
+  if ((width == 0) || (height == 0)) {
+    width = 1;
+    height = 1;
+  }
+
+  BITMAPINFOHEADER hdr = {0};
+  hdr.biSize = sizeof(BITMAPINFOHEADER);
+  hdr.biWidth = width;
+  hdr.biHeight = -height;  // minus means top-down bitmap
+  hdr.biPlanes = 1;
+  hdr.biBitCount = 32;
+  hdr.biCompression = BI_RGB;  // no compression
+  hdr.biSizeImage = 0;
+  hdr.biXPelsPerMeter = 1;
+  hdr.biYPelsPerMeter = 1;
+  hdr.biClrUsed = 0;
+  hdr.biClrImportant = 0;
+
+  HBITMAP hbitmap = CreateDIBSection(nullptr, reinterpret_cast<BITMAPINFO*>(&hdr),

[Peter Kasting, 2016/02/16 21:12:10]

Nit: 80 columns

[tomhudson, 2016/06/21 22:20:22]

Done.

+                                     0, nullptr, nullptr, 0);
+
+#if !defined(_WIN64)
+  // If CreateDIBSection() failed, try to get some useful

[Peter Kasting, 2016/02/16 21:12:10]

Nit: Wrap as late as possible

[tomhudson, 2016/06/21 22:20:22]

Done.

+  // information out before we crash for post-mortem analysis.
+  if (!hbitmap)
+    base::debug::GDIBitmapAllocFailure(&hdr, nullptr);
+#endif
+
+  return hbitmap;
+}
+
+
 }  // namespace
-
 namespace skia {
 
 POINT SkPointToPOINT(const SkPoint& point) {
   POINT win_point = {
       SkScalarRoundToInt(point.fX), SkScalarRoundToInt(point.fY)
   };
   return win_point;
 }
 
 SkRect RECTToSkRect(const RECT& rect) {
@@ skipping 28 matching lines @@
   // and arcs themselves fully respect the device context's world-to-device
   // transformation.
   BOOL res = SetGraphicsMode(context, GM_ADVANCED);
   SkASSERT(res != 0);
 
   // Enables dithering.
   res = SetStretchBltMode(context, HALFTONE);
   SkASSERT(res != 0);
   // As per SetStretchBltMode() documentation, SetBrushOrgEx() must be called
   // right after.
-  res = SetBrushOrgEx(context, 0, 0, NULL);
+  res = SetBrushOrgEx(context, 0, 0, nullptr);
   SkASSERT(res != 0);
 
   // Sets up default orientation.
   res = SetArcDirection(context, AD_CLOCKWISE);
   SkASSERT(res != 0);
 
   // Sets up default colors.
   res = SetBkColor(context, RGB(255, 255, 255));
   SkASSERT(res != CLR_INVALID);
   res = SetTextColor(context, RGB(0, 0, 0));
   SkASSERT(res != CLR_INVALID);
   res = SetDCBrushColor(context, RGB(255, 255, 255));
   SkASSERT(res != CLR_INVALID);
   res = SetDCPenColor(context, RGB(0, 0, 0));
   SkASSERT(res != CLR_INVALID);
 
   // Sets up default transparency.
   res = SetBkMode(context, OPAQUE);
   SkASSERT(res != 0);
   res = SetROP2(context, R2_COPYPEN);
   SkASSERT(res != 0);
 }
 
+#if 0
+namespace {
+
+bool PrepareAllocation(HDC context, SkImageInfo* size, BITMAP* backing) {
+  HBITMAP backing_handle =
+      static_cast<HBITMAP>(GetCurrentObject(context, OBJ_BITMAP));
+  if (GetObject(backing_handle, sizeof(*backing),
+                             backing) != sizeof(*backing))
+    return false;
+  *size = SkImageInfo::MakeN32Premul(backing->bmWidth,
+                                     backing->bmHeight);
+  return true;
+}
+
+}
+
+SkSurface* MapPlatformSurface(HDC context) {
+  SkImageInfo size;
+  Bitmap backing;
+  if (!PrepareAllocation(HDC, &size, &backing))
+    return nullptr;
+  return SkSurface::NewRasterDirect(size, backing.bmBits,
+                                    backing.bmWidthBytes);
+}
+  
+SkBitmap MapPlatformBitmap(HDC context) {
+  SkImageInfo size;
+  Bitmap backing;
+  SkBitmap bitmap;
+  if (PrepareAllocation(HDC, &size, &backing))
+    bitmap.installPixels(size, backing.bmBits, size.minRowBytes());
+  return bitmap;
+}
+#endif
+
+SkSurface* MapPlatformSurface(HDC context) {
+  HBITMAP backing_handle =
+      static_cast<HBITMAP>(GetCurrentObject(context, OBJ_BITMAP));
+  BITMAP backing;
+  if (GetObject(backing_handle, sizeof(backing),
+                &backing) != sizeof(backing))
+    return nullptr;
+  return SkSurface::NewRasterDirect(
+      SkImageInfo::MakeN32Premul(backing.bmWidth, backing.bmHeight),
+      backing.bmBits, backing.bmWidthBytes);
+}
+
+SkBitmap MapPlatformBitmap(HDC context) {
+  HBITMAP backing_handle =
+    static_cast<HBITMAP>(GetCurrentObject(context, OBJ_BITMAP));
+  BITMAP backing;
+  SkBitmap bitmap;
+  if (GetObject(backing_handle, sizeof(backing),
+                &backing) == sizeof(backing)) {
+    SkImageInfo size = SkImageInfo::MakeN32Premul(backing.bmWidth,
+                                                  backing.bmHeight);
+    bitmap.installPixels(size, backing.bmBits, size.minRowBytes());
+  }
+  return bitmap;
+}
+
+HDC CreateOffscreenSurface(int width, int height) {
+  HBITMAP bitmap = nullptr;
+
+  // If this process doesn't have access to GDI, we'll have to use a shared
+  // memory segment instead.
+  if (!base::win::IsUser32AndGdi32Available())
+    return nullptr;
+
+  bitmap = CreateHBitmap(width, height);
+  if (!bitmap)
+    return nullptr;
+
+  HDC hdc = CreateCompatibleDC(nullptr);
+  SkASSERT(hdc);
+  InitializeDC(hdc);
+  HRGN clip = CreateRectRgn(0, 0, width, height);
+  int result = SelectClipRgn(hdc, clip);
+  SkASSERT(result != ERROR);
+  result = DeleteObject(clip);
+  SkASSERT(result);
+
+  SelectObject(hdc, bitmap);
+
+  // need to clean up by calling DeleteDC(hdc) else we leak!
+
+  return hdc;
+}
+
 }  // namespace skia