Comment #47 on issue 2 by [email protected]: Testing if chromium id
works
http://code.google.com/p/chromium/issues/detail?id=2
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=33265
------------------------------------------------------------------------
r33265 | [email protected] | 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
[email protected]
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
