Comment #2 on issue 25710 by bugdro...@chromium.org: Clickint ona  link  
puts a blue outline across it
http://code.google.com/p/chromium/issues/detail?id=25710

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=33264

------------------------------------------------------------------------
r33264 | m...@chromium.org | 2009-11-29 19:27:28 -0800 (Sun, 29 Nov 2009) |  
24 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/plugin_messages.h?r1=33264&r2=33263
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/common/plugin_messages_internal.h?r1=33264&r2=33263
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/webplugin_delegate_stub.cc?r1=33264&r2=33263
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/webplugin_proxy.cc?r1=33264&r2=33263
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/plugin/webplugin_proxy.h?r1=33264&r2=33263
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/webplugin_delegate_proxy.cc?r1=33264&r2=33263
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/renderer/webplugin_delegate_proxy.h?r1=33264&r2=33263

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 in-flight  
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 plug-in 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 plug-in 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. Right-click 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
------------------------------------------------------------------------


--
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

Reply via email to