cc: Do not request redraw on commit when impl-side painting

cc/layer_tree_host_unittest_scroll.cc

Index: cc/layer_tree_host_unittest_scroll.cc
diff --git a/cc/layer_tree_host_unittest_scroll.cc b/cc/layer_tree_host_unittest_scroll.cc
index 5c30a22de8b5405d1d71ab8b25c4ca2574c72032..c7d6bbea70e4e35148aa52768a901a42a376decd 100644
--- a/cc/layer_tree_host_unittest_scroll.cc
+++ b/cc/layer_tree_host_unittest_scroll.cc
@@ -558,6 +558,7 @@ class ImplSidePaintingScrollTestSimple : public ImplSidePaintingScrollTest {
           EXPECT_VECTOR_EQ(root->scrollDelta(), impl_thread_scroll1_);
           EXPECT_VECTOR_EQ(root->sentScrollDelta(), gfx::Vector2d());
           postSetNeedsCommitToMainThread();
+          impl->setNeedsRedraw();

[nduca, 2013/01/15 01:54:54]

any idea why this one happened?

[enne (OOO), 2013/01/15 01:58:35]

Yeah, I talked with Brian about this.  Previously this test assumed there'd be a
redraw post commit that would get kicked off to try to draw the old main frame
again while there was a pending tree.

That said, I wonder if this is a race between the commit and the redraw, where
it could try to redraw and rather than getting into the else block it re-does
the first part of this if block.

I wonder if something like commitComplete should instead set the redraw in this
test.

[brianderson, 2013/01/15 02:01:56]

Will have commit complete trigger setNeedsRedraw once.

         } else {
           can_activate_ = true;
           ASSERT_TRUE(pending_root);