Comment #47 on issue 3 by bugdro...@chromium.org: This is a test http://code.google.com/p/chromium/issues/detail?id=3
The following revision refers to this bug: http://src.chromium.org/viewvc/chrome?view=rev&revision=33265 ------------------------------------------------------------------------ r33265 | m...@chromium.org | 2009-11-29 19:30:18 -0800 (Sun, 29 Nov 2009) | 29 lines Changed paths: M http://src.chromium.org/viewvc/chrome/branches/249/src/chrome/common/plugin_messages.h?r1=33265&r2=33264 M http://src.chromium.org/viewvc/chrome/branches/249/src/chrome/common/plugin_messages_internal.h?r1=33265&r2=33264 M http://src.chromium.org/viewvc/chrome/branches/249/src/chrome/plugin/webplugin_delegate_stub.cc?r1=33265&r2=33264 M http://src.chromium.org/viewvc/chrome/branches/249/src/chrome/plugin/webplugin_proxy.cc?r1=33265&r2=33264 M http://src.chromium.org/viewvc/chrome/branches/249/src/chrome/plugin/webplugin_proxy.h?r1=33265&r2=33264 M http://src.chromium.org/viewvc/chrome/branches/249/src/chrome/renderer/webplugin_delegate_proxy.cc?r1=33265&r2=33264 M http://src.chromium.org/viewvc/chrome/branches/249/src/chrome/renderer/webplugin_delegate_proxy.h?r1=33265&r2=33264 Merge r33264 from the trunk to the 249 branch. Add acknowledgement messages for PluginMsg_UpdateGeometry. On the Mac, use these acknowledgements to know when it's safe to dump old TransportDIBs in the renderer process. The Mac TransportDIB implementation uses base::SharedMemory, which cannot be disposed of if an inflight UpdateGeometry message refers to the shared memory file descriptor. BUG=27510, 26754 TEST=1. From bug 25710: a. Visit http://www.dkmsoftware.com/Yubotu.htm b. Click "Play Now" c. Resize vigorously. Expect: no sad plugin icon. 2. Test case from bug 26754 (affected machines only): a. Visit http://news.google.com/ b. Click the [+] to the left of a YouTube link. On affected machines, you'll get a sad plugin icon. c. Click the [+] to the left of a different YouTube link. Expect: no crash. 3. Test case from bug 26754 comment 9 (affected machines only): a. Have lots of bookmarks (import Safari defaults) b. Rightclick on bookmark bar, and choose "Open All Bookmarks" Expect: no crash. This change may not actually fix the third test case. Review URL: http://codereview.chromium.org/417005 tbr=m...@chromium.org Review URL: http://codereview.chromium.org/449007 ------------------------------------------------------------------------ -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings -- Automated mail from issue updates at http://crbug.com/ Subscription options: http://groups.google.com/group/chromium-bugs