[PPAPI] Fix the CommitLayers() will never be completed issue

[PPAPI] Fix the CommitLayers() will never be completed issue

The aync CommitLayers() call will never be completed, if no layers changed since the last call to CommitLayers(). So we have to force a commit for each CommitLayers() call.

BUG=374383
R=bbudge@chromium.org, piman@chromium.org

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

[Peng, 2014-06-18 15:11:14]

Hi Antoine and Bill,

I moved one bug fix from the test CL https://codereview.chromium.org/324983005/
to a new CL, because it is important to our project. I wish it could be landed
before M37 branch day. PTAL.

Thanks,
Peng

[bbudge, 2014-06-18 15:58:24]

lgtm

https://codereview.chromium.org/345443005/diff/1/content/renderer/pepper/pepp...
File content/renderer/pepper/pepper_compositor_host.cc (right):

https://codereview.chromium.org/345443005/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_compositor_host.cc:373: if
(layer_->layer_tree_host())
The comment is hard to understand. Is this what you mean?

// We need to force a commit for each CommitLayers() call, even if no layers
// changed since the last call to CommitLayers(). This is so
ViewInitiatedPaint()
// will always be called.

[Peng, 2014-06-18 16:05:44]

https://codereview.chromium.org/345443005/diff/1/content/renderer/pepper/pepp...
File content/renderer/pepper/pepper_compositor_host.cc (right):

https://codereview.chromium.org/345443005/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_compositor_host.cc:373: if
(layer_->layer_tree_host())
On 2014/06/18 15:58:24, bbudge wrote:
> The comment is hard to understand. Is this what you mean?
> 
> // We need to force a commit for each CommitLayers() call, even if no layers
> // changed since the last call to CommitLayers(). This is so
> ViewInitiatedPaint()
> // will always be called.

Yes. Thanks you. Done

[piman, 2014-06-18 21:43:53]

lgtm

[Peng, 2014-06-18 22:59:37]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2014-06-18 23:00:45]

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

[Peng, 2014-06-19 01:24:08]

Committed patchset #2 manually as r278224 (presubmit successful).