Plumb through cropped output size for VideoFrame

cc/layer_tree_host_impl_unittest.cc

 // Copyright 2011 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 "config.h"
 
 #include "cc/layer_tree_host_impl.h"
 
 #include "base/command_line.h"
 #include "base/hash_tables.h"
@@ skipping 2520 matching lines @@
 private:
     base::hash_set<unsigned> m_allocatedTextureIds;
 };
 
 // Fake video frame that represents a 4x4 YUV video frame.
 class FakeVideoFrame: public WebVideoFrame {
 public:
     FakeVideoFrame() : m_textureId(0) { memset(m_data, 0x80, sizeof(m_data)); }
     virtual ~FakeVideoFrame() { }
     virtual Format format() const { return m_textureId ? FormatNativeTexture : FormatYV12; }
-    virtual unsigned width() const { return 4; }
-    virtual unsigned height() const { return 4; }
     virtual unsigned planes() const { return m_textureId ? 0 : 3; }
-    virtual int stride(unsigned plane) const { return 4; }
     virtual const void* data(unsigned plane) const { return m_data; }
     virtual unsigned textureId() const { return m_textureId; }
     virtual unsigned textureTarget() const { return m_textureId ? GL_TEXTURE_2D : 0; }
+    virtual unsigned visibleX() const { return 0; }
+    virtual unsigned visibleY() const { return 0; }
+    virtual unsigned visibleWidth() const { return 4; }
+    virtual unsigned visibleHeight() const { return 4; }
+    virtual unsigned textureWidth() const { return 4; }
+    virtual unsigned textureHeight() const { return 4; }
 

[Ami GONE FROM CHROMIUM, 2012/10/25 00:43:07]

y u no test the interesting case of non-zero offset and non-1.0 scaling?

[sheu, 2012/10/25 01:53:54]

Done.

     void setTextureId(unsigned id) { m_textureId = id; }
 
 private:
     char m_data[16];
     unsigned m_textureId;
 };
 
 // Fake video frame provider that always provides the same FakeVideoFrame.
 class FakeVideoFrameProvider: public WebVideoFrameProvider {
 public:
@@ skipping 1816 matching lines @@
         verifyRenderPassTestData(removeRenderPassesCases[testCaseIndex], testData);
         testCaseIndex++;
     }
 }
 
 INSTANTIATE_TEST_CASE_P(LayerTreeHostImplTests,
                         LayerTreeHostImplTest,
                         ::testing::Values(false, true));
 
 } // namespace