1006566 - Unify the common DataStore tests between the main thread and the worker thread.

Status: RESOLVED FIXED

(Core :: DOM: Workers, defect)

Description

[Gene Lian (I already quit Mozilla)]

After fixing bug 1006562:

  is(cursor.store, gStore, "Cursor.store is the store");

we can then unify the common part between:

1. file_basic.html and file_basic_worker.js
2. file_sync.html and file_sync_worker.js

Comment 1

[Gene Lian (I already quit Mozilla)]

Attachment: Part 1, make up missing tests for workers

To make no difference for the tests between the main thread and the worker, so that we can refactorize them later.

Comment 2

[Gene Lian (I already quit Mozilla)]

Attachment: Part 2, unify the common parts

This is the main fix. In the worker, we can use the dedicated importScripts(...) to import the needed JS file.

Comment 3

[Gene Lian (I already quit Mozilla)]

Attachment: Part 3, minor alignment

I didn't align the codes in the previous patches for your convenience of reviews, which are eventually aligned in this patch. ;)

Comment 4

[Gene Lian (I already quit Mozilla)]

Full try: https://tbpl.mozilla.org/?tree=Try&rev=132224b232f4

Comment 5

[Gene Lian (I already quit Mozilla)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2a78eb9c84fc
https://hg.mozilla.org/integration/mozilla-inbound/rev/217a9ba98fc8
https://hg.mozilla.org/integration/mozilla-inbound/rev/c0f7841fbe96

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/2a78eb9c84fc
https://hg.mozilla.org/mozilla-central/rev/217a9ba98fc8
https://hg.mozilla.org/mozilla-central/rev/c0f7841fbe96