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)

PowerPC
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Brade, Assigned: Brade)

References

Details

Attachments

(2 files, 5 obsolete files)

sync up with newer point along 1.0 branch
Attached patch patch 1 of 2 to land (obsolete) — Splinter Review
This is 1 of 2 diffs that need to land.
Attached patch patch 2 of 2 to land (obsolete) — Splinter Review
This is 2 of 2 diffs that need to land.
Keywords: patch
patch keyword? brade, do you not have check-in privs?
Keywords: review
Status: NEW → ASSIGNED
Keywords: patch, review
Attachment #94510 - Attachment is obsolete: true
Attachment #94511 - Attachment is obsolete: true
bryner: what commands did you use to generate this diff?  It looks like it's
missing some files (such as nsIPlaintextEditor.idl).
cvs diff -N from the mozilla directory.
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?)
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.
Comment on attachment 95300 [details]
diff of CHIMERA_M1_0_BRANCH against CHIMERA_M1_0_BASE (gzipped)

obsoleting
Attachment #95300 - Attachment is obsolete: true
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.
Attachment #95044 - Attachment is obsolete: true
Attachment #95045 - Attachment is obsolete: true
Attached patch new diff, part 2Splinter Review
This diff includes the two additional patches from bug 141900.
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
Depends on: 163543
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.

Attachment

General

Created:
Updated:
Size: