diff options
| author | sail@chromium.org <sail@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-09-01 03:37:20 +0000 |
|---|---|---|
| committer | sail@chromium.org <sail@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-09-01 03:37:20 +0000 |
| commit | 0f3a2d136bbe988ad5fa8b5585d2d1b6dfd40aff (patch) | |
| tree | 330624241ff080d3f750c2f021137498e97c5f3b /chrome_frame/plugin_url_request.cc | |
| parent | 50b0139750b7892d0d39a1cc0da93619cbf2ecd1 (diff) | |
| download | chromium_src-0f3a2d136bbe988ad5fa8b5585d2d1b6dfd40aff.zip chromium_src-0f3a2d136bbe988ad5fa8b5585d2d1b6dfd40aff.tar.gz chromium_src-0f3a2d136bbe988ad5fa8b5585d2d1b6dfd40aff.tar.bz2 | |
Pepper Flash Mac: Fix crash when playing video on MB retina
When playing video on Mac Book retina devices Chrome would crash.
The problem was that we were using the page scale factor when drawing bitmaps from the plugin. Thus when drawing the bitmap we would dereference memory past the end of the bitmap buffer and crash.
To fix this we now pass the plugin's scale factor instead of the page's scale factor.
BUG=141541
TEST=Note, this crash only happens if accelerated compositing is disabled. This is most common on MB retina devices running 10.7 since the device is on the GPU black list.
To test this I ran Chrome on a Mac Book retina on 10.8 with --disable-accelerated-compositing. I verified that without this patch Chrome crashes and with this patch the video plays fine. Also tested that everything works correctly on an external monitor with a 1x scale factor.
Also, to reproduce this crash you also need this patch:
https://chromiumcodereview.appspot.com/10879050
That patch got reverted due to this crash. Once the fix is checked in I'll reland that patch.
Review URL: https://chromiumcodereview.appspot.com/10905036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154593 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome_frame/plugin_url_request.cc')
0 files changed, 0 insertions, 0 deletions