My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 81880: CallOnMainThread must pass PP_ERROR_ABORTED to the callback on surf-away/reload
4 people starred this issue and may be notified of changes. Back to list
 
Reported by pol...@google.com, May 6, 2011
bsy:

currently, if CallOnMainThread is invoked and the user surfs away, the timeout may cause the callback to fire after surfing away, and the callback is always invoked with the user specified result code.  so, there's no way for the callback function to know if an error condition occurred (e.g., aborted callback) and the plugin object etc had been destroyed.  by setting the timeout to a value that allows me to surf away to a page where there's no pepper plugin, i could get my code to cause an aw snap.

viettrungluu:

CallOnMainThread should have the same semantics as all the other
callbacks (see the comment in webkit/plugins/ppapi/callbacks.h).

May 23, 2011
#1 jeffr...@chromium.org
(No comment was entered for this change.)
Labels: Feature-Plugins-Pepper
May 26, 2011
#2 stuartmorgan@chromium.org
(No comment was entered for this change.)
Labels: -Feature-Plugins
Jun 23, 2011
#3 darin@chromium.org
(No comment was entered for this change.)
Status: Assigned
Labels: Mstone-14
Jul 27, 2011
#4 kerz@google.com
Punting out non-critical bugs.  Please move back to 14 if you believe this was done in error.
Labels: -Mstone-14 Mstone-15 MovedFrom-14
Sep 8, 2011
#5 kar...@google.com
moving all non-essential bugs from 15 to 16. please feel free to move back if this was an error and your bug is a release blocker.
Labels: Mstone-16 bulkmove MovedFrom15
Oct 24, 2011
#6 lafo...@google.com
(No comment was entered for this change.)
Labels: -Mstone-16 MovedFrom-16 Mstone-17
Dec 19, 2011
#7 kerz@google.com
Moving bugs marked as Available but not blockers from M17 to M18.  Please move back if you think this is a blocker, and add the ReleaseBlock-Stable label.  If you're able.
Labels: -Mstone-17 Mstone-18 MovedFrom-17
Feb 6, 2012
#8 cstefan...@google.com
Bulk edit: M18 branch point was a week ago. Please give an update/ETA if the fix should be merged into M18 - otherwise adjust the milestone.
Feb 7, 2012
#9 kar...@google.com
(No comment was entered for this change.)
Labels: MovedFrom18 Mstone-19
Mar 22, 2012
#10 viettrun...@chromium.org
(No comment was entered for this change.)
Labels: -Mstone-19 Mstone-20
Apr 27, 2012
#11 kerz@google.com
These bugs have hit their move limit.  Please re-target as appropriate.
Labels: -Mstone-20 MovedFrom-20
Jan 12, 2013
#12 viettrun...@chromium.org
(No comment was entered for this change.)
Status: Available
Owner: ---
Mar 10, 2013
#13 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-Internals -Feature-Plugins-Pepper Cr-Content-Plugins-Pepper Cr-Internals
Apr 5, 2013
#14 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: Cr-Blink
Apr 5, 2013
#15 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: Cr-Internals-Plugins
Apr 5, 2013
#16 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Cr-Content-Plugins-Pepper Cr-Internals-Plugins-Pepper
Sign in to add a comment

Powered by Google Project Hosting