Add an "isInsideVSync" check.

Add an "isInsideVSync" check.

Prevent calling postInvalidateOnAnimate inside VSync handler, because
this will cause the invalidation to be one frame delayed.

BUG=394604
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=291525

[hush (inactive), 2014-08-21 21:57:36]

Yaron, can you look at ui/android?

This is a part of the CL from https://codereview.chromium.org/457913002

[hush (inactive), 2014-08-21 21:59:17]

by the way, Bo and I were wondering why VSyncMonitor is not a singleton while it
wraps the Android system's singleton Choreorgrapher

[Yaron, 2014-08-22 01:15:22]

This seems like sievers territory. If he's happy, I can RS

[hush (inactive), 2014-08-22 01:26:06]

On 2014/08/22 01:15:22, Yaron wrote:
> This seems like sievers territory. If he's happy, I can RS

Daniel lgtm'ed the ui/ part in https://codereview.chromium.org/457913002

[Yaron, 2014-08-22 01:28:45]

lgtm

[boliu, 2014-08-22 01:29:17]

aw lgtm

[hush (inactive), 2014-08-22 01:31:02]

The CQ bit was checked by hush@chromium.org

[commit-bot: I haz the power, 2014-08-22 01:32:21]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hush@chromium.org/470523006/1

[commit-bot: I haz the power, 2014-08-22 02:28:58]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[commit-bot: I haz the power, 2014-08-22 02:31:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-22 02:31:11]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[boliu, 2014-08-22 02:39:45]

On 2014/08/22 02:31:11, I haz the power (commit-bot) wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on tryserver.chromium.linux
>
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

Yaron's email is misspelled in ui/android/OWNERS...

Maybe you can correct that in this CL too :p

[boliu, 2014-08-22 03:03:53]

On 2014/08/22 02:39:45, boliu wrote:
> On 2014/08/22 02:31:11, I haz the power (commit-bot) wrote:
> > Try jobs failed on following builders:
> >   chromium_presubmit on tryserver.chromium.linux
> >
>
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
> 
> Yaron's email is misspelled in ui/android/OWNERS...
> 
> Maybe you can correct that in this CL too :p

Ehh, I misread. Not sure why presubmit is not happy

[hush (inactive), 2014-08-22 03:13:43]

The CQ bit was checked by hush@chromium.org

[commit-bot: I haz the power, 2014-08-22 03:13:59]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hush@chromium.org/470523006/1

[commit-bot: I haz the power, 2014-08-22 03:25:01]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[commit-bot: I haz the power, 2014-08-22 03:28:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-22 03:28:09]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[hush (inactive), 2014-08-22 03:32:01]

Hi Jared!
Can you take a look at ui/ part? Thanks!

[jdduke (slow), 2014-08-22 03:35:01]

On 2014/08/22 03:32:01, hush wrote:
> Hi Jared!
> Can you take a look at ui/ part? Thanks!

Rubberstamp lgtm.

[hush (inactive), 2014-08-22 04:10:32]

The CQ bit was checked by hush@chromium.org

[commit-bot: I haz the power, 2014-08-22 04:11:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hush@chromium.org/470523006/1

[commit-bot: I haz the power, 2014-08-22 04:19:08]

FYI, CQ is re-trying this CL (attempt #1).
The failing builders are:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)

[commit-bot: I haz the power, 2014-08-22 05:12:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-08-22 05:12:49]

Try jobs failed on following builders:
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)

[hush (inactive), 2014-08-22 21:05:46]

The CQ bit was checked by hush@chromium.org

[commit-bot: I haz the power, 2014-08-22 21:06:49]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/hush@chromium.org/470523006/1

[commit-bot: I haz the power, 2014-08-22 22:12:18]

Committed patchset #1 (1) as 291525