Closed
Bug 161668
Opened 22 years ago
Closed 22 years ago
sync up with newer point along 1.0 branch
Categories
(Camino Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Brade, Assigned: Brade)
References
Details
Attachments
(2 files, 5 obsolete files)
395.00 KB,
patch
|
Details | Diff | Splinter Review | |
320.04 KB,
patch
|
Details | Diff | Splinter Review |
sync up with newer point along 1.0 branch
Assignee | ||
Comment 1•22 years ago
|
||
This is 1 of 2 diffs that need to land.
Assignee | ||
Comment 2•22 years ago
|
||
This is 2 of 2 diffs that need to land.
patch keyword? brade, do you not have check-in privs?
Keywords: review
Assignee | ||
Updated•22 years ago
|
Comment 4•22 years ago
|
||
Attachment #94510 -
Attachment is obsolete: true
Updated•22 years ago
|
Attachment #94511 -
Attachment is obsolete: true
Comment 5•22 years ago
|
||
Assignee | ||
Comment 6•22 years ago
|
||
bryner: what commands did you use to generate this diff? It looks like it's missing some files (such as nsIPlaintextEditor.idl).
Comment 7•22 years ago
|
||
cvs diff -N from the mozilla directory.
Comment 8•22 years ago
|
||
I think you're referring to some of the files checked in as a fix for bug 141900. I talked to kin about this; only attachment 87306 [details] [diff] [review] has been checked in on the 1.0 branch. He said that the async reflow/paint/scroll patch was causing some problems with the caret jumping back to the beginning of the text field, so I wasn't sure if we wanted to re-land this fix or not. It sounds like maybe we want to re-land it anyway. (native textfields, anyone?)
Comment 9•22 years ago
|
||
Comment 10•22 years ago
|
||
bnesse, would you be able to sanity-check the plugin parts of this patch? There were a few places where merge conflicts happened and I'm not 100% sure I merged correctly. Thanks.
Assignee | ||
Comment 11•22 years ago
|
||
Comment on attachment 95300 [details]
diff of CHIMERA_M1_0_BRANCH against CHIMERA_M1_0_BASE (gzipped)
obsoleting
Attachment #95300 -
Attachment is obsolete: true
Assignee | ||
Comment 12•22 years ago
|
||
I'm ok with the current (non-obsolete) patches to be landed on the branch assuming the configure files are also checked in as well as the patches from bug 141900 which were inadvertently removed from the diffs. We need to coordinate the landing with mcafee so he can get a tinderbox working on the CHIMERA_M1_0_1_BRANCH before we land the rest; Chris--can you get that started? The only other issue is that these diffs are now out-of-date with regards to the current tip of the chimera branch; we'll need to do another round of diffs before we can actually have everyone use it.
Comment 13•22 years ago
|
||
Updated•22 years ago
|
Attachment #95044 -
Attachment is obsolete: true
Attachment #95045 -
Attachment is obsolete: true
Comment 14•22 years ago
|
||
This diff includes the two additional patches from bug 141900.
Assignee | ||
Comment 15•22 years ago
|
||
I've looked over the above diffs. They look much the same as the diffs I have generated except I used -u3 or -u4 so I can't directly compare them as easily that way. r=brade
Assignee | ||
Comment 16•22 years ago
|
||
this is done
Status: ASSIGNED → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•