Sidebar should not open by default in new profiles

VERIFIED FIXED in seamonkey2.1b2

Status

SeaMonkey
Sidebar
--
major
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Blocks: 1 bug, {regression})

Trunk
seamonkey2.1b2
regression
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
I noticed this while working on mochitests.
I confirmed it by creating a new profile.

***

For example, this is causing
{
mochitest-browser-chrome

TEST-UNEXPECTED-FAIL | chrome://mochitests/content/browser/suite/browser/test/browser_bug462289.js | tab key to tab activeElement - Got [object XULElement @ 0x3e0c5e8 (native @ 0x39e3a58)], expected [object XULElement @ 0x544b898 (native @ 0xa583f20)]
}
(Assignee)

Comment 1

7 years ago
Linux and Windows, not OSX:

http://mxr.mozilla.org/comm-central/source/suite/browser/test/Makefile.in#83
{
84 ifneq (cocoa,$(MOZ_WIDGET_TOOLKIT))
85 _BROWSER_FILES += browser_bug462289.js
86 endif
}

*****

Last builds not reporting this bug:

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1295365601.1295368012.28795.gz
Linux comm-central-trunk debug test mochitest-other on 2011/01/18 07:46:41
rev:476b91f50827
moz:141d92d974b4

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1295375700.1295378689.25740.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2011/01/18 10:35:00
rev:476b91f50827
moz:141d92d974b4


First builds reporting this bug:

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1295392267.1295395235.18978.gz
Linux comm-central-trunk debug test mochitest-other on 2011/01/18 15:11:07
rev:44423b415a5f
moz:5337fe8a5bfb

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1295469195.1295474286.18355.gz
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2011/01/19 12:33:15
rev:e9f62190bdf0
moz:844b898f5bc4


Regression timeframes:
http://hg.mozilla.org/comm-central/pushloghtml?fromchange=476b91f50827&tochange=44423b415a5f
KaiRo's push again: omnijar, etc.
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=141d92d974b4&tochange=5337fe8a5bfb
...
blocking-seamonkey2.1: --- → ?
Keywords: regressionwindow-wanted
(Assignee)

Comment 2

7 years ago
This is like bug 627417!

{
/suite/locales/generic/profile/localstore.rdf
    * line 8 -- <RDF:Description about="chrome://navigator/content/navigator.xul#sidebar-box">
}
can't be accessed in omni.jar.
Blocks: 588067
(Assignee)

Updated

7 years ago
Depends on: 627417
(Assignee)

Comment 3

7 years ago
Created attachment 508339 [details] [diff] [review]
(Av1) Keep localstore.rdf out of omni.jar (ftb)
[Checked in: Comment 8]

(Untested, but nothing new.)
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #508339 - Flags: review?(kairo)
(Assignee)

Comment 4

7 years ago
Karsten, I assume we should do the same for
http://mxr.mozilla.org/comm-central/source/suite/locales/generic/profile/panels.rdf.in
but I'm not sure how to check whether it works or not...

Comment 5

7 years ago
Comment on attachment 508339 [details] [diff] [review]
(Av1) Keep localstore.rdf out of omni.jar (ftb)
[Checked in: Comment 8]

We really should be able to do with an empty or non-existing localstore, please make sure to file a bug on that. Still, this workaround is acceptable for the moment.

And I don't understand your comment on panels.rdf, which is here already anyhow.
Attachment #508339 - Flags: review?(kairo) → review+
(Assignee)

Comment 6

7 years ago
(In reply to comment #4)
> http://mxr.mozilla.org/comm-central/source/suite/locales/generic/profile/panels.rdf.in

(In reply to comment #5)
> And I don't understand your comment on panels.rdf, which is here already
> anyhow.

Indeed, correct link:
http://mxr.mozilla.org/comm-central/source/suite/locales/generic/profile/mimeTypes.rdf
(Assignee)

Comment 7

7 years ago
(In reply to comment #5)
> We really should be able to do with an empty or non-existing localstore, please
> make sure to file a bug on that.

Well, I would prefer that you file it, as you can probably give more details than I can about what you're thinking might be possible to do.
(Assignee)

Comment 8

7 years ago
Comment on attachment 508339 [details] [diff] [review]
(Av1) Keep localstore.rdf out of omni.jar (ftb)
[Checked in: Comment 8]

http://hg.mozilla.org/comm-central/rev/c7b112032a41
Attachment #508339 - Attachment description: (Av1) Keep localstore.rdf out of omni.jar (ftb) → (Av1) Keep localstore.rdf out of omni.jar (ftb) [Checked in: Comment 8]
(Assignee)

Updated

7 years ago
Status: ASSIGNED → RESOLVED
blocking-seamonkey2.1: ? → ---
Last Resolved: 7 years ago
Resolution: --- → FIXED

Comment 9

7 years ago
(In reply to comment #7)
> Well, I would prefer that you file it, as you can probably give more details
> than I can about what you're thinking might be possible to do.

All I know about it is that we need to make sure we need to not depend on any contents of localstore.
(Assignee)

Comment 10

7 years ago
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1296496919.1296500862.10776.gz
Linux comm-central-trunk debug test mochitest-other on 2011/01/31 10:01:59

V.Fixed
Status: RESOLVED → VERIFIED
(Assignee)

Comment 11

7 years ago
(In reply to comment #9)
> All I know about it is that we need to make sure we need to not depend on any
> contents of localstore.

Iiuc, bug 559505 should fix this eventually.
You need to log in before you can comment on or make changes to this bug.