Remove various non-standard things from tests

RESOLVED FIXED in Firefox 37

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: evilpie, Assigned: evilpie)

Tracking

unspecified
Firefox 37
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 3 obsolete attachments)

(Assignee)

Description

4 years ago
While working on different iterations of my patch for bug 783829, I identified various tests that could be updated. I should probably not waste those patches.
(Assignee)

Comment 1

4 years ago
Created attachment 8535590 [details] [diff] [review]
fix-browser-tests
(Assignee)

Comment 2

4 years ago
Created attachment 8535591 [details] [diff] [review]
fix-tests
(Assignee)

Updated

4 years ago
Attachment #8535590 - Attachment is patch: true
(Assignee)

Updated

4 years ago
Attachment #8535591 - Attachment is patch: true
(Assignee)

Comment 3

4 years ago
Created attachment 8543544 [details] [diff] [review]
Cleanup FormHistory a bit

If you look athte whole file it looks like it expects the object for "for each" to have a length property. So it's probably just an array.
Attachment #8543544 - Flags: review?(mhammond)
(Assignee)

Comment 4

4 years ago
Created attachment 8543545 [details] [diff] [review]
Remove Iterator() from some tests
Attachment #8543545 - Flags: review?(bugs)
(Assignee)

Updated

4 years ago
Attachment #8535590 - Attachment is obsolete: true
(Assignee)

Updated

4 years ago
Attachment #8535591 - Attachment is obsolete: true

Updated

4 years ago
Attachment #8543544 - Flags: review?(mhammond) → review+

Comment 5

4 years ago
Comment on attachment 8543545 [details] [diff] [review]
Remove Iterator() from some tests

> function runNextSpinTest() {
>-  try {
>-    var [index, test] = spinTests.next();
>-    test();
>-  } catch(e) {
>-    if (e == StopIteration) {
>-      SimpleTest.finish();
>-      return; // We're all done
>-    }
>-    throw e;
>+  var test = spinTests.shift();
>+  if (!test) {
>+    SimpleTest.finish();
>+    return;
>   }
>+  SimpleTest.executeSoon(test);
I don't understand why you need asynchronousness  here.
Doesn't the old code just run sync or am I missing something.


Please fix or explain and re-ask review.
Attachment #8543545 - Flags: review?(bugs) → review-
(Assignee)

Comment 6

4 years ago
Created attachment 8545974 [details] [diff] [review]
v2 - Remove Iterator() from some tests

Fixed.
Attachment #8543545 - Attachment is obsolete: true
Attachment #8545974 - Flags: review?(bugs)

Updated

4 years ago
Attachment #8545974 - Flags: review?(bugs) → review+
https://hg.mozilla.org/mozilla-central/rev/c3179d05230b
https://hg.mozilla.org/mozilla-central/rev/6429d270c004
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 37
You need to log in before you can comment on or make changes to this bug.