Add drawImage() originClean() getSecurityOrigin() to OffscreenCanvas

third_party/WebKit/Source/core/offscreencanvas/OffscreenCanvas.cpp

 // Copyright 2015 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 "core/offscreencanvas/OffscreenCanvas.h"
 
 #include "core/dom/ExceptionCode.h"
 #include "core/html/canvas/CanvasContextCreationAttributes.h"
 #include "core/html/canvas/CanvasRenderingContext.h"
 #include "core/html/canvas/CanvasRenderingContextFactory.h"
 #include "wtf/MathExtras.h"
 
 namespace blink {
 
-OffscreenCanvas::OffscreenCanvas(const IntSize& size)
-    : m_size(size)
+OffscreenCanvas::OffscreenCanvas(ExecutionContext* executionContext, const IntSize& size)
+    : m_executionContext(executionContext)

[Justin Novosad, 2016/04/28 19:38:12]

If haraken thinks this is safe, I am Okay with it, but I am not sure whether it
is okay to keep a long lived reference to the executionContext like this. I
think you should always be getting the execution context from the current script
entry point instead of using a reference that was stored at construction time.

+    , m_size(size)
+    , m_originClean(true)
 {
 }
 
-OffscreenCanvas* OffscreenCanvas::create(unsigned width, unsigned height)
+OffscreenCanvas* OffscreenCanvas::create(ExecutionContext* executionContext, unsigned width, unsigned height)
 {
-    return new OffscreenCanvas(IntSize(clampTo<int>(width), clampTo<int>(height)));
+    return new OffscreenCanvas(executionContext, IntSize(clampTo<int>(width), clampTo<int>(height)));
 }
 
 void OffscreenCanvas::setWidth(unsigned width)
 {
     m_size.setWidth(clampTo<int>(width));
 }
 
 void OffscreenCanvas::setHeight(unsigned height)
 {
     m_size.setHeight(clampTo<int>(height));
@@ skipping 62 matching lines @@
 }
 
 void OffscreenCanvas::registerRenderingContextFactory(PassOwnPtr<CanvasRenderingContextFactory> renderingContextFactory)
 {
     CanvasRenderingContext::ContextType type = renderingContextFactory->getContextType();
     ASSERT(type < CanvasRenderingContext::ContextTypeCount);
     ASSERT(!renderingContextFactories()[type]);
     renderingContextFactories()[type] = std::move(renderingContextFactory);
 }
 
+bool OffscreenCanvas::originClean() const
+{
+    // TODO(crbug.com/607575): Make Settings accessable in worker and use
+    // disableReadingFromCanvas to determine originClean value.
+    return m_originClean;
+}
+
+SecurityOrigin* OffscreenCanvas::getSecurityOrigin() const
+{
+    return getExecutionContext()->getSecurityOrigin();
+}
+
 DEFINE_TRACE(OffscreenCanvas)
 {
     visitor->trace(m_context);
+    visitor->trace(m_executionContext);
 }
 
 } // namespace blink