Closed Bug 1321225 Opened 8 years ago Closed 7 years ago

Remove legacy generator from layout/.

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla53
Tracking Flags:
Tracking Status
firefox53 --- fixed

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

Remove legacy generator from layout/.
6.11 KB, patch
jrmuizel
: review+
Details | Diff | Splinter Review 
Just replaced legacy generator with ES6 generator.

some notes:

* layout/base/tests/bug583889_inner1.html
  ES6 generator doesn't have |send|, but |next| receives a value
  also ES6 generator doesn't throw StopIteration, so we can just remove final yield

* layout/generic/crashtests/383089-1.html
  ES6 generator doesn't throw StopIteration, but returns object with {done, value} properties
Attachment #8815640 - Flags: review?(jmuizelaar)
Attachment #8815640 - Attachment is patch: true
Attachment #8815640 - Attachment mime type: text/x-patch → text/plain
Attachment #8815640 - Flags: review?(jmuizelaar) → review+
https://hg.mozilla.org/mozilla-central/rev/61c633e7330a
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox53: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: