Closed Bug 1142790 Opened 8 years ago Closed 8 years ago

Add a test for passing Request objects to DOM cache matchAll method

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: ehsan.akhgari, Assigned: ehsan.akhgari)

References

Details

Attachments

(1 file, 1 obsolete file)

No description provided.
Assignee: nobody → ehsan
Blocks: cachetests
Comment on attachment 8576985 [details] [diff] [review]
Add a test for passing Request objects to DOM cache matchAll method

Review of attachment 8576985 [details] [diff] [review]:
-----------------------------------------------------------------

All the comments from bug 1142333 apply here as well.  We need to check the various combinations of values in the CacheQueryOptions that can be passed to MatchAll(), etc.
Attachment #8576985 - Flags: review?(bkelly)
Comment on attachment 8576985 [details] [diff] [review]
Add a test for passing Request objects to DOM cache matchAll method

We're going to do follow-ups for missing stuff.
Attachment #8576985 - Flags: review+
Attachment #8576985 - Attachment is obsolete: true
Attachment #8577350 - Flags: review?(bkelly)
Attachment #8577350 - Flags: review?(bkelly) → review+
Sorry relanded since I have a ton of stuff built on top of this:

https://hg.mozilla.org/integration/mozilla-inbound/rev/80a2308b5cc8

I filed bug 1143222 to investigate and disabled these two tests for now.
Flags: needinfo?(ehsan)
https://hg.mozilla.org/mozilla-central/rev/80a2308b5cc8
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.