Of the remaining bugs, most are low-risk and I think can reasonably land
without further approval.
Some, however, have more significant risk associated with them:
<https://bugzilla.mozilla.org/show_bug.cgi?id=516776> (as well as
<https://bugzilla.mozilla.org/show_bug.cgi?id=526918>) is compelling for
our web-integration story but currently has a moderate amount of risk
associated with it.
Keeping <https://bugzilla.mozilla.org/show_bug.cgi?id=526918> in the
tree is high value for the default use case, but if we don't back it
out, there is at least one and possibly multiple other patches we'll
need to take, all to code that is known to be fragile.
For these, I think we're going to need to have a driver call to discuss
in more detail, so that we can come to a decision on what sort of risk
is worth accepting at this stage of the game.
Dan
On 2009-11-06 22:24, Dan Mosedale wrote:
> Keeping <https://bugzilla.mozilla.org/show_bug.cgi?id=526918> in the
> tree
That should be <https://bugzilla.mozilla.org/show_bug.cgi?id=489609>,
correct?
On 07.11.2009 08:18, Nathan Tuggy wrote:
> On 2009-11-06 22:24, Dan Mosedale wrote:
>> Keeping <https://bugzilla.mozilla.org/show_bug.cgi?id=526918> in the
>> tree
> That should be <https://bugzilla.mozilla.org/show_bug.cgi?id=489609>,
> correct?
Yes. This allows subjects to wrap, therefore allowing to see the full
subject all all (with ease) - that's why dmose called it "high-value".
The problem is that we keep running into XUL bugs, and a few of our own.
But I'm trying to fix these. Remaining is:
Bug 525302 has a very small patch (essentially 3 lines), which is fairly
safe. The last 2 lines in the patch also fix bug 526292. It's highly
advisable to get this in ASAP, IMHO.
Bug 526918, which dmose mentioned above, is only about the "All headers"
mode. It's adding some extra space to the top of the header pane. The
good news is that it's fixed on trunk - probably a Gecko bug which has
been fixed on trunk. The bad news is that I don't know yet why it
happened or how to work around it. I'm on it.
Ben