Closed
Bug 493546
Opened 16 years ago
Closed 16 years ago
Enable building of toolkit/components/cookie for Thunderbird
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla1.9.2a1
People
(Reporter: standard8, Assigned: standard8)
References
Details
(Keywords: verified1.9.1.1, Whiteboard: [tb3needs])
Attachments
(1 file)
690 bytes,
patch
|
ted
:
review+
dmosedale
:
superreview+
samuel.sidler+old
:
approval1.9.1.1+
|
Details | Diff | Splinter Review |
In bug 492279 Thunderbird is looking at improving its cookie policy for TB 3.
As part of this work, we're looking at relying on the core cookie policy rather than our own specific override.
Using the core policy means that a user could change network.cookie.lifetimePolicy to prompt if we should allow cookies for a site or not.
This means we'll need the cookie dialog chrome which exists in toolkit/components/cookie, if we don't have this, no prompt happens and Couldn't convert chrome URL: chrome://cookie/content/cookieAcceptDialog.xul is dumped to the error console.
Therefore I'd like to include that in Thunderbird's build as a) it does not add significantly to the build size, b) it is one less ifdefed directory within toolkit, c) we already include the locale files for it, d) extensions may want to allow the user to enable it. Even if we don't end up using it, I think reducing ifdefs is a good idea.
Requesting r and sr as although I know we just need build review, I'd like to get Dan's approval on including it in the build.
I have tested that an appropriately patched TB brings up the dialog correctly.
I'm looking to get this into 1.9.1 branch, especially as its no-change for Firefox (expect build order).
Attachment #378047 -
Flags: superreview?(dmose)
Attachment #378047 -
Flags: review?(ted.mielczarek)
Assignee | ||
Updated•16 years ago
|
Whiteboard: [tb3needs]
Comment 1•16 years ago
|
||
Comment on attachment 378047 [details] [diff] [review]
The fix
sr=dmose
Attachment #378047 -
Flags: superreview?(dmose) → superreview+
Updated•16 years ago
|
Attachment #378047 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 2•16 years ago
|
||
I'm looking to get permission to land this patch on trunk which is restricted at the moment, irc seems to be quiet wrt drivers so asking on the bug. Here's the answers to the various questions I expect:
- This patch very slightly modifies the order of which Firefox builds directories in toolkit/components.
- This patch passed on try server builds on 18th May. The only failures in tests were failures that occurred on the tree at the same time.
- At some stage soon I'm expecting that Thunderbird will want this patch on the 1.9.1 branch. I'm currently waiting reviews on bug 492279 (which is a TB blocker), so I can't specify how long that is but it could be in the next week or two.
- If I get reviews before the next Thunderbird beta, I'd want to get this patch in as well, I can only do that if this has landed on trunk.
- I have no idea how much longer the Firefox restriction and lock down will be.
Whiteboard: [tb3needs] → [tb3needs][needs approval to land on trunk]
Assignee | ||
Comment 3•16 years ago
|
||
Checked in to trunk: http://hg.mozilla.org/mozilla-central/rev/a12ef932037a
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [tb3needs][needs approval to land on trunk] → [tb3needs][backing before requesting approval]
Assignee | ||
Updated•16 years ago
|
Whiteboard: [tb3needs][backing before requesting approval] → [tb3needs][baking before requesting approval]
Assignee | ||
Comment 4•16 years ago
|
||
No approval flags for 1.9.1.1 yet :-( so wanted will have to do.
TB 3 wants this to enable inclusion of the cookie prompts for its builds. See comment 2 for more info.
Flags: wanted1.9.1.x?
Assignee | ||
Updated•16 years ago
|
Whiteboard: [tb3needs][baking before requesting approval] → [tb3needs]
Assignee | ||
Updated•16 years ago
|
Attachment #378047 -
Flags: approval1.9.1.1?
Assignee | ||
Comment 5•16 years ago
|
||
Comment on attachment 378047 [details] [diff] [review]
The fix
Requesting branch approval please see previous comments on this bug for reasons why we need/want this and it doesn't really affect ff.
Assignee | ||
Updated•16 years ago
|
Flags: wanted1.9.1.x?
Assignee | ||
Updated•16 years ago
|
Whiteboard: [tb3needs] → [tb3needs][waiting for 1911 approval]
Comment 6•16 years ago
|
||
Comment on attachment 378047 [details] [diff] [review]
The fix
Approved for 1.9.1.1. a=ss for release-drivers
Attachment #378047 -
Flags: approval1.9.1.1? → approval1.9.1.1+
Updated•16 years ago
|
Whiteboard: [tb3needs][waiting for 1911 approval] → [tb3needs][needs checkin]
Updated•16 years ago
|
Keywords: checkin-needed
Target Milestone: mozilla1.9.1 → mozilla1.9.2a1
Assignee | ||
Comment 7•16 years ago
|
||
I can handle my own checkins, I'll do this tomorrow morning UTC.
Keywords: checkin-needed
Assignee | ||
Comment 8•16 years ago
|
||
Checked in to 1.9.1: http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a6f2586c52dc
Keywords: fixed1.9.1.1
Whiteboard: [tb3needs][needs checkin] → [tb3needs]
Comment 9•16 years ago
|
||
Mark: can you verify that this is fixed in latest-mozilla1.9.1 nightly or better yet the 3.5.1 release candidate: ftp://ftp.mozilla.org/pub/mozilla.org/firefox/nightly/3.5.1-candidates/build1/
Assignee | ||
Comment 10•16 years ago
|
||
Well it is a bit difficult to verify a build config patch, especially one that is enabling stuff for Thunderbird only and not Firefox.
However, I check that the cookie prompt dialog still works in firefox (lifetimePolicy set to 1), and that it also works in Thunderbird. I can't think of anything else to check.
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1.1 → verified1.9.1.1
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•