Close shm in image data destructor

Close shm in image data destructor
TEST=try
BUG=http://code.google.com/p/chromium/issues/detail?id=120728

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

[brettw, 2012-03-28 20:20:55]

lgtm

[dmichael (off chromium), 2012-03-28 20:21:59]

lgtm

Is there any reasonable way to test this (possibly in a future CL)?

[nfullagar, 2012-03-28 20:25:44]

ppapi_browser/ppb_image_data has some tests; can add more in another CL.

[commit-bot: I haz the power, 2012-03-28 20:33:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/nfullagar@google.com/9875027/3002

[dmichael (off chromium), 2012-03-28 20:33:59]

If it's not too hard to test particularly that we're not leaking handles (or
whatever way you would detect this bug in a test), that would be nice for a
future CL (not necessary here). When we eventually switch proxies, we want to
make sure we don't regress on this kind of stuff.

Still lgtm

[commit-bot: I haz the power, 2012-03-29 01:40:10]

Change committed as 129550