Specify the rect invalidated by SetLayerTransform.

Specify the rect invalidated by SetLayerTransform.

When SetLayerTransform was introduced, instead of specifying the rect
that is invalidated, the no_update_visible flag was manually set to
false to indicate that the flush callback needs to be executed. Since
nothing was invalidated, we don't get calls to ViewInitiatedPaint,
which delays the callback.

By setting the op_rect like the other queued commands do, the
invalidations and callback scheduling are done correctly.

BUG=605379
Committed: https://crrev.com/9fb5007ba0a99358e8418f0c2b3791705006a7c0
Cr-Commit-Position: refs/heads/master@{#427459}

[Kevin McNee - google account, 2016-10-24 15:45:26]

Description was changed from

==========
Specify the rect invalidated by SetLayerTransform.

When SetLayerTransform was introduced, instead of specifying the rect
that is invalidated, the no_update_visible flag was manually set to
false to indicate that the flush callback needs to be executed. Since
nothing was invalidated, we don't get calls to ViewInitiatedPaint,
which delays the callback.

By setting the op_rect like the other queued commands do, the
invalidations and callback scheduling are done correctly.

BUG=605379
==========

to

==========
Specify the rect invalidated by SetLayerTransform.

When SetLayerTransform was introduced, instead of specifying the rect
that is invalidated, the no_update_visible flag was manually set to
false to indicate that the flush callback needs to be executed. Since
nothing was invalidated, we don't get calls to ViewInitiatedPaint,
which delays the callback.

By setting the op_rect like the other queued commands do, the
invalidations and callback scheduling are done correctly.

BUG=605379
==========

[Kevin McNee - google account, 2016-10-24 15:45:26]

mcnee@google.com changed reviewers:
+ bokan@chromium.org, raymes@chromium.org, wjmaclean@chromium.org

[Kevin McNee - google account, 2016-10-24 15:51:06]

raymes: Hello. Could you take a look at this change? There was a problem with
the implementation of SetLayerTransform when it was first added to PPAPI
(https://codereview.chromium.org/1881603002/) which caused delays in flush ack
callbacks.

[wjmaclean_google.com, 2016-10-24 16:44:22]

On 2016/10/24 15:51:06, Kevin McNee wrote:
> raymes: Hello. Could you take a look at this change? There was a problem with
> the implementation of SetLayerTransform when it was first added to PPAPI
> (https://codereview.chromium.org/1881603002/) which caused delays in flush ack
> callbacks.

I'm fine with this ... lgtm.

[bokan, 2016-10-24 17:03:28]

seems reasonable to me, non-OWNER lgtm

[raymes, 2016-10-25 03:08:43]

raymes@chromium.org changed reviewers:
+ piman@chromium.org

[raymes, 2016-10-25 03:08:43]

I defer to piman who originally reviewed this change.

[piman, 2016-10-25 19:14:19]

lgtm

[Kevin McNee - google account, 2016-10-25 19:49:02]

The CQ bit was checked by mcnee@google.com

[commit-bot: I haz the power, 2016-10-25 19:49:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-25 20:49:50]

Description was changed from

==========
Specify the rect invalidated by SetLayerTransform.

When SetLayerTransform was introduced, instead of specifying the rect
that is invalidated, the no_update_visible flag was manually set to
false to indicate that the flush callback needs to be executed. Since
nothing was invalidated, we don't get calls to ViewInitiatedPaint,
which delays the callback.

By setting the op_rect like the other queued commands do, the
invalidations and callback scheduling are done correctly.

BUG=605379
==========

to

==========
Specify the rect invalidated by SetLayerTransform.

When SetLayerTransform was introduced, instead of specifying the rect
that is invalidated, the no_update_visible flag was manually set to
false to indicate that the flush callback needs to be executed. Since
nothing was invalidated, we don't get calls to ViewInitiatedPaint,
which delays the callback.

By setting the op_rect like the other queued commands do, the
invalidations and callback scheduling are done correctly.

BUG=605379
==========

[commit-bot: I haz the power, 2016-10-25 20:49:51]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-25 20:51:49]

Description was changed from

==========
Specify the rect invalidated by SetLayerTransform.

When SetLayerTransform was introduced, instead of specifying the rect
that is invalidated, the no_update_visible flag was manually set to
false to indicate that the flush callback needs to be executed. Since
nothing was invalidated, we don't get calls to ViewInitiatedPaint,
which delays the callback.

By setting the op_rect like the other queued commands do, the
invalidations and callback scheduling are done correctly.

BUG=605379
==========

to

==========
Specify the rect invalidated by SetLayerTransform.

When SetLayerTransform was introduced, instead of specifying the rect
that is invalidated, the no_update_visible flag was manually set to
false to indicate that the flush callback needs to be executed. Since
nothing was invalidated, we don't get calls to ViewInitiatedPaint,
which delays the callback.

By setting the op_rect like the other queued commands do, the
invalidations and callback scheduling are done correctly.

BUG=605379

Committed: https://crrev.com/9fb5007ba0a99358e8418f0c2b3791705006a7c0
Cr-Commit-Position: refs/heads/master@{#427459}
==========

[commit-bot: I haz the power, 2016-10-25 20:51:50]

Patchset 1 (id:??) landed as
https://crrev.com/9fb5007ba0a99358e8418f0c2b3791705006a7c0
Cr-Commit-Position: refs/heads/master@{#427459}