Closed Bug 1370640 Opened 4 years ago Closed 4 years ago

Get rid of ProtoAndIfaceCache::EntrySlotIfExists

Categories

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

53 Branch
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

Details

Attachments

(1 file)

The only remaining consumer doesn't really want to use it.
Depends on: 1370614
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Comment on attachment 8874974 [details] [diff] [review]
Get rid of ProtoAndIfaceCache::EntrySlotIfExists

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

You should probably switch your patch in attachment 8874727 [details] [diff] [review] to HasEntryInSlot :-).
Attachment #8874974 - Flags: review?(peterv) → review+
Comment on attachment 8874974 [details] [diff] [review]
Get rid of ProtoAndIfaceCache::EntrySlotIfExists

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

Actually, we use this in generated code, so how does that work?
Attachment #8874974 - Flags: review+ → review-
It works because this patch is on top of bug 1370614, which replaces all those users (generated, and from attachment 8874727 [details] [diff] [review]) with HasEntryInSlot (and in fact implements it).  See the "depends on" field.  Can I just go with the initial r+?  ;)
Flags: needinfo?(peterv)
Comment on attachment 8874974 [details] [diff] [review]
Get rid of ProtoAndIfaceCache::EntrySlotIfExists

Hmm, ok. It's kind of hard to have to look at dependent bugs when reviewing a patch.
Flags: needinfo?(peterv)
Attachment #8874974 - Flags: review- → review+
Sorry.  I should have made that clearer.  :(
Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/37f50095e77e
Get rid of ProtoAndIfaceCache::EntrySlotIfExists.  r=peterv
https://hg.mozilla.org/mozilla-central/rev/37f50095e77e
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.