Refactoring ImageBuffer to decouple it from Canvas2DLayerBridge

Source/core/platform/graphics/Canvas2DImageBufferSurface.h

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 10 matching lines @@
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#ifndef HTMLSrcsetParser_h
-#define HTMLSrcsetParser_h
+#ifndef Canvas2DImageBufferSurface_h
+#define Canvas2DImageBufferSurface_h
 
-#include "wtf/text/WTFString.h"
+#include "core/platform/graphics/Canvas2DLayerBridge.h"
+#include "platform/graphics/ImageBufferSurface.h"
 
 namespace WebCore {
 
-class ImageCandidate {
+// This shim necessary because ImageBufferSurfaces are not allowed to be RefCounted
+class Canvas2DImageBufferSurface : public ImageBufferSurface {
 public:
-    ImageCandidate()
-        : m_scaleFactor(1.0)
+    Canvas2DImageBufferSurface(const IntSize& size, OpacityMode opacityMode, float resolutionScale, int msaaSampleCount)
+        : ImageBufferSurface(size, opacityMode, resolutionScale)
+        , m_layerBridge(Canvas2DLayerBridge::create(size, opacityMode, msaaSampleCount))
     {
+        clear();
     }
 
-    ImageCandidate(const String& source, unsigned start, unsigned length, float scaleFactor)
-        : m_string(source.createView(start, length))
-        , m_scaleFactor(scaleFactor)
-    {
-    }
+    virtual ~Canvas2DImageBufferSurface() { m_layerBridge->destroy(); }
 
-    String toString() const
-    {
-        return m_string.toString();
-    }
-
-    inline float scaleFactor() const
-    {
-        return m_scaleFactor;
-    }
-
-    inline bool isEmpty() const
-    {
-        return m_string.isEmpty();
-    }
+    // ImageBufferSurface implementation
+    virtual void aboutToUse() OVERRIDE { m_layerBridge->aboutToUse(); }
+    virtual SkCanvas* canvas() const OVERRIDE { return m_layerBridge->canvas(); }
+    virtual bool isValid() const OVERRIDE { return m_layerBridge && m_layerBridge->isValid(); }
+    virtual blink::WebLayer* layer() const OVERRIDE { return m_layerBridge->layer(); }
+    virtual Platform3DObject getBackingTexture() const OVERRIDE { return m_layerBridge->getBackingTexture(); }
+    virtual bool isAccelerated() const { return m_layerBridge->isAccelerated(); }
 
 private:
-    StringView m_string;
-    float m_scaleFactor;
+    RefPtr<Canvas2DLayerBridge> m_layerBridge;
 };
 
-ImageCandidate bestFitSourceForSrcsetAttribute(float deviceScaleFactor, const String& srcsetAttribute);
-
-ImageCandidate bestFitSourceForImageAttributes(float deviceScaleFactor, const String& srcAttribute, const String& srcsetAttribute);
-
-String bestFitSourceForImageAttributes(float deviceScaleFactor, const String& srcAttribute, ImageCandidate& srcsetImageCandidate);
-
 }
 
 #endif