Closed Bug 1368029 Opened 3 years ago Closed 3 years ago
Remove LRU policy support in Process
This code is not currently used. I suggest to remove it. I'm planning to use partially the ProcessPriorityManager, but the LRU should not be involved.
Comment on attachment 8871694 [details] [diff] [review] lru_1.patch LGTM with just one nit, remove the related bits from HAL too: https://dxr.mozilla.org/mozilla-central/rev/f7adbf457ee20eeffde72694e0d17d73616e3cfd/hal/sandbox/SandboxHal.cpp#352 https://dxr.mozilla.org/mozilla-central/rev/f7adbf457ee20eeffde72694e0d17d73616e3cfd/hal/fallback/FallbackProcessPriority.cpp#14 https://dxr.mozilla.org/mozilla-central/rev/f7adbf457ee20eeffde72694e0d17d73616e3cfd/hal/Hal.h#442 https://dxr.mozilla.org/mozilla-central/rev/f7adbf457ee20eeffde72694e0d17d73616e3cfd/hal/Hal.cpp#854
Attachment #8871694 - Flags: review?(gsvelto) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/cb62a593770c Remove LRU priority support from ProcessPriorityManager, r=gsvelto
You need to log in before you can comment on or make changes to this bug.