1357582 - Convert PRCList usage in PluginModuleParent to mozilla::LinkedList

Status: RESOLVED WONTFIX

(Core Graveyard :: Plug-ins, enhancement)

Description

[Eric Rahm [:erahm]]

We would like to update PRCList usage in nsPluginHost to use mozilla::LinkedList.

- Convert |sModuleListHead| [1], to be a mozilla::LinkedList.
- Update |PluginModuleMapping| [2] to inherit from mozilla::LinkedListElement
- Update PRCList calls to LinkedList equivalents:
  - remove PR_INIT_CLIST
  - PR_INSERT_BEFORE -> LinkedList::insertBack
  - PR_INSERT_AFTER/PR_APPEND_LINK -> LinkedList::insertFront
  - remove PR_REMOVE_LINK 
  - PR_CLIST_IS_EMPTY -> LinkedList::isEmpty
  - convert do/while loops w/ PR_LIST_HEAD/PR_NEXT_LINK -> range-based for (auto elm : list)

[1] http://searchfox.org/mozilla-central/rev/214345204f1e7d97abb571b7992b6deedb5ff98f/dom/plugins/ipc/PluginModuleParent.cpp#354
[2] http://searchfox.org/mozilla-central/rev/214345204f1e7d97abb571b7992b6deedb5ff98f/dom/plugins/ipc/PluginModuleParent.cpp#216

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

(In reply to Eric Rahm [:erahm] from comment #0)
> We would like to update PRCList usage in nsPluginHost to use
> mozilla::LinkedList.
> 
> - Convert |sModuleListHead| [1], to be a mozilla::LinkedList.
> - Update |PluginModuleMapping| [2] to inherit from mozilla::LinkedListElement

Be advised that this code is being removed in bug 1352575.

Comment 2

[Benjamin Smedberg]

Let's not leave this bug here.