Closed Bug 1145208 Opened 9 years ago Closed 9 years ago

[e10s] Update tests in browser/base/content/tests/plugins

Categories

(Firefox :: General, defect)

x86_64
All
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1129040
Tracking Status
e10s - ---

People

(Reporter: jimm, Assigned: jimm)

Details

Attachments

(1 file, 2 obsolete files)

working on this currently, I'm about 30% of the way through these tests.
Attached patch wip (obsolete) — Splinter Review
Blocks: e10s-tests
Attached patch wip (obsolete) — Splinter Review
Current state:

browser.ini is up to date and identifies the tests I've fixed vs. those that still have problems.

Currently finishing up browser_pluginnotification.js, so don't run that one.

Also, in some of the bugxxxxx tests, I kept the little while loop for running across test functions. I ended up completely removing that and converting to tasks about half way through, so I need to go back and edit the older tests to bring them in line.

Also I need to go through plugins head.js, there's a lot of cruft there. For example, we have helpers that have promiseXXX, waitForXXX, and asyncXXX. I'd like to clean this up a bit after I'm through.
Attachment #8580116 - Attachment is obsolete: true
tracking-e10s: --- → +
Blocks: 1148827
Attached patch patchSplinter Review
Mostly up to date, now just looking for stability issues.
Attachment #8582550 - Attachment is obsolete: true
No longer blocks: e10s-tests, 1148541, 1148827
Status: NEW → RESOLVED
Closed: 9 years ago
No longer depends on: 1129040
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: