Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Issue 9959078: Revert 128179 - Make sure the plugin scriptable object is released before NPP_Destroy. (Closed)

Created:
8 years, 8 months ago by Wez
Modified:
8 years, 8 months ago
Reviewers:
Wez
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam
Visibility:
Public.

Description

Revert 128179 - Make sure the plugin scriptable object is released before NPP_Destroy. We're temporarily reverting this change to gather more data on its impact on plugin crash rates. When the we tear down a plugin instance the plugin process first invokes NPP_Destroy, and then tears down the IPC channel to the renderer, to give NPP_Destroy a chance to do last-minute scripting. When the IPC channel for the last instance is torn down we also clean up the IPC channels and stubs for any plugin-side NPObjects that remain. We suspect that some plugins implement the scriptable object as part of the plugin instance, rather than independently ref-counted, so that our releasing the object after NPP_Destroy actually triggers the plugin process to crash. This CL tears down the stub for the plugin's scriptable object before we call NPP_Destroy. As per crbug.com/119414, we will remove this code if it doesn't significantly impact crashes. BUG=101968 Review URL: http://codereview.chromium.org/9817023 TBR=cpu@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=130199

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -22 lines) Patch
M content/common/npobject_stub.h View 2 chunks +2 lines, -3 lines 0 comments Download
M content/plugin/webplugin_delegate_stub.h View 3 chunks +1 line, -3 lines 0 comments Download
M content/plugin/webplugin_delegate_stub.cc View 4 chunks +7 lines, -16 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Wez
8 years, 8 months ago (2012-04-02 20:33:49 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698