[Pepper][Compositor] Fix the compositor stops updating texture issue.

[Pepper][Compositor] Fix the compositor stops updating texture issue.

BUG=417408
Committed: https://crrev.com/b25533eb65de20237dff5ea50606e3339e9bd9ca
Cr-Commit-Position: refs/heads/master@{#296952}

[Peng, 2014-09-25 20:22:11]

penghuang@chromium.org changed reviewers:
+ danakj@chromium.org

[Peng, 2014-09-25 20:22:34]

penghuang@chromium.org changed reviewers:
+ dmichael@chromium.org

[Peng, 2014-09-25 20:23:02]

On 2014/09/25 20:22:34, Peng wrote:
> mailto:penghuang@chromium.org changed reviewers:
> + mailto:dmichael@chromium.org

Hi Dana & David, PTAL. Thanks.

[danakj, 2014-09-25 20:25:20]

LGTM

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

https://codereview.chromium.org/601293003/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_compositor_host.cc:262:
texture_layer->SetNeedsDisplay();
Can you add a TODO that you should get a damage region from the application and
propagate that here?

https://codereview.chromium.org/601293003/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_compositor_host.cc:294:
image_layer->SetNeedsDisplay();
Can you add a TODO that you should get a damage region from the application and
propagate that here?

[Peng, 2014-09-25 20:37:52]

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

https://codereview.chromium.org/601293003/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_compositor_host.cc:262:
texture_layer->SetNeedsDisplay();
On 2014/09/25 20:25:20, danakj wrote:
> Can you add a TODO that you should get a damage region from the application
and
> propagate that here?

Done.

https://codereview.chromium.org/601293003/diff/1/content/renderer/pepper/pepp...
content/renderer/pepper/pepper_compositor_host.cc:294:
image_layer->SetNeedsDisplay();
On 2014/09/25 20:25:20, danakj wrote:
> Can you add a TODO that you should get a damage region from the application
and
> propagate that here?

Done.

[dmichael (off chromium), 2014-09-26 15:12:00]

OWNERS lgtm

[Peng, 2014-09-26 15:16:06]

The CQ bit was checked by penghuang@chromium.org

[commit-bot: I haz the power, 2014-09-26 15:16:54]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/601293003/20001

[commit-bot: I haz the power, 2014-09-26 15:48:54]

Committed patchset #2 (id:20001) as a036abd0bc8b465ded64490602f41b870354958e

[commit-bot: I haz the power, 2014-09-26 15:49:56]

Patchset 2 (id:??) landed as
https://crrev.com/b25533eb65de20237dff5ea50606e3339e9bd9ca
Cr-Commit-Position: refs/heads/master@{#296952}