803438 - Convert the preflight cache to use mozilla/LinkedList.h rather than prclist.h

Status: RESOLVED FIXED

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

Description

[Jeff Walden [:Waldo]]

Attachment: Patch

prclist.h is grody macros, LinkedList.h is the new hotness (and still type-safe, and moreover full of delicious sanity-checking assertions).  And prclist.h includes prtypes.h, which is on ehsan's hit list.  :-)

I'm reasonably certain, given locally running some of content/ mochitests, that this should be good, but I pushed to try just to be safe:

https://tbpl.mozilla.org/?tree=Try&rev=1b1890873105

I don't really plan on doing conversions beyond this one, but one good example seems worthwhile so that each individual transition can be a good first (or maybe second) bug.   Also it seems worthwhile in order to consider whether the LinkedList interface is as good/complete as it could be.  Just doing this one conversion points out one, maybe two, maybe three, maybe even four issues worth considering.  This is the basic patch, and I'll file a followup to discuss those other issues separately.

Comment 1

[Justin Lebar (not reading bugmail)]

Comment on attachment 673104 [details] [diff] [review]
Patch

Looks good. My only complaint is the 4 lines of context.  git bz [1] will ensure you always get 8 lines.

[1] https://github.com/jlebar/moz-git-tools

Comment 2

[Jeff Walden [:Waldo]]

But that would require using git.  And [diff] unified = 8 would work just as well there, if I didn't intentionally have unified = 3.  :-P

https://hg.mozilla.org/integration/mozilla-inbound/rev/499ca9ee03bc

Comment 3

[Justin Lebar (not reading bugmail)]

> But that would require using git

Ah, well, if you're on hg, you have no excuse.  :)

> And [diff] unified = 8 would work just as well there, if I didn't intentionally have unified = 3.  
> :-P

I'm not sure what you mean, but I don't believe you can make git diff default to -U8.  You have to create an alias, e.g.

    [alias]
    dif = diff -U8 --patience -M -C

Comment 4

[Jeff Walden [:Waldo]]

Er, sorry, that might have been clearer without "there" -- the syntax bits were referring to ~/.hgrc.

Comment 5

[Jeff Walden [:Waldo]]

...wait.  Did I just miss a chance to snark about hg being better than git?  I am falling down on the job!  I should be ashamed to show my face in public.  ;-)

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 673104 [details] [diff] [review]
Patch

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

::: content/base/src/nsCrossSiteListenerProxy.cpp
@@ +219,4 @@
>      // If that didn't remove anything then kick out the least recently used
>      // entry.
>      if (mTable.Count() == PREFLIGHT_CACHE_SIZE) {
> +      CacheEntry* lruEntry = static_cast<CacheEntry*>(mList.popLast());

Do you still need that cast?

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/499ca9ee03bc