588871 - Remove ConsoleStorage methods used for log message storage

Status: RESOLVED DUPLICATE of bug 703235

(DevTools :: General, defect, P2)

Description

[David Dahl :ddahl]

We are moving all storage over to the ConsoleStorageService now, so we can use the ConsoleStorage object for config/filtering prefs only now - as well as other things that might come up in the future.

We should also rename this object. Let the bikeshedding begin!

Comment 1

[Kevin Dangoor]

ConsoleConfig?

Whatever, we need not spend much time on the naming given that it's just an internal component for the Web Console.

Comment 2

[David Dahl :ddahl]

Attachment: v 0.1 Code removal Patch

This is a followup to all of the ConsoleStorageService bugs. This is purely code removal and will affect perf and memory use in a GOOD way:)

Comment 3

[Johnathan Nightingale [:johnath]]

Comment on attachment 499160 [details] [diff] [review]
v 0.1 Code removal Patch

Can't approve without review.

Comment 4

[Dietrich Ayala (:dietrich)]

Is this rotted or still needing review?

Comment 5

[David Dahl :ddahl]

Comment on attachment 499160 [details] [diff] [review]
v 0.1 Code removal Patch

I will need to unbitrot this and remove some clutter

Comment 6

[Mihai Sucan [:msucan]]

This is a console cleanup bug. Some day we should complete this patch. The ConsoleStorage object in HUDService is not very much needed, actually. We can handle prefs in a better way.

Comment 7

[Sonny Piers [:sonny]]

This will be fixed by my patch on bug 703235 .