Allow ImageBitmaps derived from HTMLImageElements to be evicted from RAM.

Source/core/page/ImageBitmapTest.cpp

 /*
  * 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 11 matching lines @@
  * 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.
  */
 
 #include "config.h"
-#include "core/animation/AnimatableNeutral.h"
+#include "core/page/ImageBitmap.h"
 
-#include "core/animation/AnimatableUnknown.h"
-#include "core/css/CSSArrayFunctionValue.h"
+#include "core/dom/Document.h"
+#include "core/html/HTMLImageElement.h"
+#include "core/loader/cache/CachedImage.h"
+#include "core/platform/graphics/BitmapImage.h"
 
 #include <gtest/gtest.h>
 
-using namespace WebCore;
+namespace WebCore {
 
-namespace {
+class ImageBitmapTest : public ::testing::Test {
+};
 
-TEST(AnimatableNeutral, Create)
+// Verifies that the cached image help by an ImageBitmap is the same as the

[Justin Novosad, 2013/08/08 13:18:28]

help -> held

+// one held by the HTMLImageElement.
+TEST_F(ImageBitmapTest, CachedImageConsistency)
 {
-    EXPECT_TRUE(AnimatableValue::neutralValue());
+    RefPtr<HTMLImageElement> imageElement = HTMLImageElement::create(Document::create().get());
+    CachedImage* cachedImage = new CachedImage(BitmapImage::create().get());

[Justin Novosad, 2013/08/08 13:18:28]

I think this code leaks the cachedImage. Should put it in an OwnPtr.

Also, you should call the version of ImageBitmap::create that takes a
NativeImageSkia, so that we actually have something to crop from.

+    imageElement->setCachedImage(cachedImage);
+    RefPtr<ImageBitmap> imageBitmap = ImageBitmap::create(imageElement.get(), IntRect(0, 0, 50, 50));

[Justin Novosad, 2013/08/08 13:18:28]

Would be good to test different scenarios here:
crop rect == image rect.
crop rect included in image rect
crop rect outside of image rect (In this case perhaps
imageBitmap->bitmapImage().get() == 0?)
crop rect intersecting image rect

+    ASSERT_EQ(imageBitmap->bitmapImage().get(), cachedImage->image());
 }
 
-TEST(AnimatableNeutral, Add)
-{
-    RefPtr<CSSValue> cssValue = CSSArrayFunctionValue::create();
-    RefPtr<AnimatableValue> animatableUnknown = AnimatableUnknown::create(cssValue);
-
-    EXPECT_EQ(cssValue, AnimatableValue::add(animatableUnknown.get(), AnimatableValue::neutralValue())->toCSSValue());
-    EXPECT_EQ(cssValue, AnimatableValue::add(AnimatableValue::neutralValue(), animatableUnknown.get())->toCSSValue());
-}
-
-}
+} // namespace