479879 - ExtendedSplitView autosaving is broken

Status: RESOLVED FIXED

(Camino Graveyard :: General, defect)

Description

[Stuart Morgan]

We're missing a formatter string in the method that returns the name, so different ESVs will stomp each other. I haven't checked if the Cert ESV (the only one other than the location+search field) autosaves, this may not be doing anything bad now, but should be fixed.

Comment 1

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

Attachment: Fix-ish

(In reply to comment #0)
> I haven't checked if the Cert ESV (the
> only one other than the location+search field) autosaves,

It does autosave[1].  It doesn't appear to stomp because the location+search does not autosave[2] (instead, there we save a width under its own key[3]).

This patch fixes autosaving the name with the "ExtendedSplitView Sizes" key for the I-can't-believe-we-really-named-it-that "CategoriesSplitter" (which also gets a new, more descriptive name, since we've never really been saving it).

However, this patch doesn't fix writing values for the two frames under a  "NSSplitView Subview Frames (null)" key; I'm not sure how to fix that.

[1] http://mxr.mozilla.org/camino/source/camino/src/security/CertificatesWindowController.mm#430
[2] http://mxr.mozilla.org/camino/source/camino/src/browser/BrowserWindowController.mm#954
[3] http://mxr.mozilla.org/camino/source/camino/src/browser/BrowserWindowController.mm#807

Comment 2

[Stuart Morgan]

Comment on attachment 490441 [details] [diff] [review]
Fix-ish

sr=smorgan

Comment 3

[Smokey Ardisson (offline for a while; not following bugs - do not email)]

http://hg.mozilla.org/camino/rev/b22b4a5ed489

(In reply to comment #1)
> However, this patch doesn't fix writing values for the two frames under a 
> "NSSplitView Subview Frames (null)" key; I'm not sure how to fix that.

Filed bug 615766 for that.