707052 - regression in patch for bug 90268

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Josh Aas]

Attachment: local test case

Chris Pearce found a regression in my patch for bug 90268. I want to give his regression its own bug for tracking purposes.

I'm attaching a zip file with a local test case, but there is also a URL for it.

To reproduce problem with patches in bug 90268:

    Click "Create windowless plugin"
    Click "Add plugin to doc". Observe plugin working.
    Click "Remove plugin".
    Click "Add plugin to doc". Observe plugin not working.

Comment 1

[Josh Aas]

Attachment: flawed automated test v1.0

I set out to make an automated test and I succeeded, sort of. It depends on how long the stream takes to load, which means it involves some nasty timeouts that I'm not comfortable checking in. The timeouts approximate the time between clicking buttons in the test case Chris provided.

Comment 2

[Josh Aas]

This regression is fixed in the latest version of the patch.