Remove ColorBehavior argument to Image::imageForCurrentFrame

third_party/WebKit/Source/core/html/ImageData.cpp

 /*
  * Copyright (C) 2008 Apple 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:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 16 matching lines @@
  */
 
 #include "core/html/ImageData.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/V8Uint8ClampedArray.h"
 #include "core/dom/ExceptionCode.h"
 #include "core/frame/ImageBitmap.h"
 #include "core/imagebitmap/ImageBitmapOptions.h"
 #include "platform/RuntimeEnabledFeatures.h"
+#include "platform/graphics/ColorBehavior.h"

[Justin Novosad, 2017/03/03 19:34:57]

Is this because it used to be indirectly included?

[ccameron, 2017/03/03 20:13:21]

Yes, exactly.

 
 namespace blink {
 
 bool ImageData::validateConstructorArguments(const unsigned& paramFlags,
                                              const IntSize* size,
                                              const unsigned& width,
                                              const unsigned& height,
                                              const DOMArrayBufferView* data,
                                              const String* colorSpace,
                                              ExceptionState* exceptionState,
@@ skipping 347 matching lines @@
     : m_size(size),
       m_colorSpace(getImageDataColorSpace(colorSpaceName)),
       m_data(byteArray) {
   DCHECK_GE(size.width(), 0);
   DCHECK_GE(size.height(), 0);
   SECURITY_CHECK(static_cast<unsigned>(size.width() * size.height() * 4) <=
                  m_data->length());
 }
 
 }  // namespace blink