This version addresses some popup sizing bugs, and also a few other issues I
ran into when debugging Blake's problems:
* The standalone popup needs a max width of 800px for Chrome compatibility,
which is wider than our default max width.
* I added a flex attribute to our browser so that it fills the entire space
of the slide-in panel. This is only necessary for browsers with content
that is shorter than the height of the panel when it gets its desired
width, but becomes longer when it doesn't, so it didn't show up in my
initial tests.
* I also added an extra pixel to the width calculations, since I noticed that
a lot of single lines of text were unexpectedly wrapping without it. I'll
look into this more in a follow-up bug.
I also added some comments, and renamed a couple of variables, where things
seemed unclear.
The test changes are mostly just updates to older browser action tests to use
newer helpers, rather than ad-hoc events, to open/close/click the widgets. A
few tests also needed updates to explicitly close the panel when they were
done with it.
--HG--
extra : commitid : BhHv1aZSrBL
extra : rebase_source : 6fe0069ebd2f94b0ffd02ad757a799fbdff9226e
extra : source : 628af985c7eba7e4665a1b62f9a60f6a6fd39822
This brings our behavior closer in line with Chrome's.
--HG--
extra : commitid : DaDofvmQEZW
extra : rebase_source : cbeda3889362511c5002f2710c063a2c83047d3c
extra : source : 7ad8b56958c54787313ecbe60eb1e8fbafb2f376
This version addresses some popup sizing bugs, and also a few other issues I
ran into when debugging Blake's problems:
* The standalone popup needs a max width of 800px for Chrome compatibility,
which is wider than our default max width.
* I added a flex attribute to our browser so that it fills the entire space
of the slide-in panel. This is only necessary for browsers with content
that is shorter than the height of the panel when it gets its desired
width, but becomes longer when it doesn't, so it didn't show up in my
initial tests.
* I also added an extra pixel to the width calculations, since I noticed that
a lot of single lines of text were unexpectedly wrapping without it. I'll
look into this more in a follow-up bug.
I also added some comments, and renamed a couple of variables, where things
seemed unclear.
The test changes are mostly just updates to older browser action tests to use
newer helpers, rather than ad-hoc events, to open/close/click the widgets. A
few tests also needed updates to explicitly close the panel when they were
done with it.
--HG--
extra : commitid : 9OyRnaMTqn8
extra : rebase_source : efd28a49ca5961e2cf545420a81bd19b796d8957
This brings our behavior closer in line with Chrome's.
--HG--
extra : commitid : 7X5knx0CtGm
extra : rebase_source : b301e9c568df0edb313b10cf06e2f7a9c23a6e78
Bug 1121291 made @hidden persist but we are reverting that change so we need to remove the values so the password column doesn't always show by default.
--HG--
extra : commitid : 7WHEU1OHPgl
extra : rebase_source : fb7f791197c61b67fe50280bb5ce4371dc95caad
Nothing in mozilla-central uses the NS_APP_BOOKMARKS_50_FILE/BMarks key except
tests. It's also not been very useful since the switch to places in ... Firefox
3? so even if addons use it, it's not doing them much good.
BookmarkHTMLUtils.defaultPath returns the same thing anyways for those that
really do insist on getting the equivalent thing (but I'd argue
BookmarkHTMLUtils.defaultPath, as well as the browser.bookmarks.file pref could
just go away as well).
Seamonkey does use NS_APP_BOOKMARKS_50_FILE/BMarks, but they really only need
the #define, which they can add in their source.
