Remove isCompositorFramePending plumbing

Remove isCompositorFramePending plumbing

Originally added: https://codereview.chromium.org/139103009.

This is only used for one test, and is somewhat a lie because
WebTestProxy implements its own (very different) animate/composite
scheduling.  Removing this plumbing makes it possible to better clean up
old non-composited paint logic inside of WebTestProxy.

Due to changes elsewhere, this test no longer catches the original
regression.  The scheduleAnimate call inside of RenderLayerCompositor
that was added in the original patch can be removed and the page still
paints properly.  In fact, all scheduleAnimate calls and calls to the
page animator in RLC can be removed and the page still gets updated.

The code in RenderLayerCompositor has changed drastically since m33, and
if possible I'd like to remove this one-off plumbing that is no longer
catching any regressions.

TBR=abarth@chromium.org
BUG=none

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=174200

[ojan, 2014-05-14 23:03:22]

lgtm

I agree. Exposing blink internals like this doesn't make sense for one-offs, as
evidenced by this test no longer testing the original bug.

[commit-bot: I haz the power, 2014-05-14 23:04:04]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/enne@chromium.org/289873002/20001

[enne (OOO), 2014-05-15 00:15:38]

The CQ bit was unchecked by enne@chromium.org

[enne (OOO), 2014-05-15 00:15:51]

abarth: public/web OWNERS?

[enne (OOO), 2014-05-16 17:09:29]

TBRing abarth for public/web as this is just removing a function that no longer
has any callers

[enne (OOO), 2014-05-16 17:09:35]

The CQ bit was checked by enne@chromium.org

[commit-bot: I haz the power, 2014-05-16 17:09:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/enne@chromium.org/289873002/40001

[commit-bot: I haz the power, 2014-05-16 18:38:06]

Change committed as 174200

[chrishtr, 2014-05-19 16:37:04]

lgtm