Fix PluginImageData and mouselock NaCl example:

Fix PluginImageData and mouselock NaCl example:
- Fix the memory leak in PluginImageData.
- Fix the out-of-bound memory access in mouselock NaCl example.

BUG=106779
TEST=mouselock NaCl example won't crash after a while (~30 seconds).

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=113854

[yzshen1, 2011-12-08 23:24:24]

Hi, Dave.

Please take a look. Thanks!

[mball, 2011-12-08 23:36:29]

After this is committed, we should merge the updated example to
https://nativeclient-sdk.googlecode.com/svn/branches/cod/src so that the
pepper_16 bundle picks it up, and also merge the IRT changes to both Chrome 16
and 17.

[yzshen1, 2011-12-08 23:40:14]

On 2011/12/08 23:36:29, mball wrote:
> After this is committed, we should merge the updated example to
> https://nativeclient-sdk.googlecode.com/svn/branches/cod/src so that the
> pepper_16 bundle picks it up, and also merge the IRT changes to both Chrome 16
> and 17.

Thanks, Matthew!

[mball, 2011-12-08 23:59:38]

lgtm

+noelallen, +bradnelson as native_client_sdk owners.

[dmichael (off chromium), 2011-12-09 02:43:43]

LGTM, thanks for tracking that down!

[commit-bot: I haz the power, 2011-12-09 08:10:18]

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

[commit-bot: I haz the power, 2011-12-09 08:10:22]

Presubmit check for 8872045-1 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit Warnings **
License must match:
.*? Copyright \(c\) 2011 The Chromium Authors\. All rights reserved\.\n.*? Use
of this source code is governed by a BSD-style license that can be\n.*? found in
the LICENSE file\.\n
Found a bad license header in these files:
  native_client_sdk/src/examples/mouselock/mouselock.cc \
  ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_image_data.cc \
  ppapi/native_client/src/shared/ppapi_proxy/plugin_ppb_image_data.h

** Presubmit ERRORS **
Missing LGTM from an OWNER for:
native_client_sdk/src/examples/mouselock/mouselock.cc

Missing LGTM from an OWNER for:
native_client_sdk/src/examples/mouselock/mouselock.cc

Presubmit checks took 1.4s to calculate.

[noelallen1, 2011-12-09 18:37:26]

lgtm

[mball, 2011-12-12 20:52:57]

I'll go ahead and merge the changes to the example over to the pepper_16 branch
in the old NaCl SDK repository...

[yzshen1, 2011-12-12 20:56:38]

On 2011/12/12 20:52:57, mball wrote:
> I'll go ahead and merge the changes to the example over to the pepper_16
branch
> in the old NaCl SDK repository...

Thanks!