If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Backout new drag for selection handling code from Mozilla8 and reland for Mozilla9

ASSIGNED
Assigned to

Status

()

Core
Selection
ASSIGNED
6 years ago
4 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

(Blocks: 6 bugs)

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 550011 [details] [diff] [review]
backout patch

I and smuag discussed this issue.

Now all known regression bugs have patches. However, we need a big change for bug 671187 and bug 672181. Unfortunately, we don't have much time to test it in current cycle. Therefore, we decided that we backout the patches from Mozilla8.

The attached patch backouts all changes. And it will be relanded after 8/16 (mozilla8 will be moved to Aurora).

This bug manages the temporary backing out.

I'll reopen bug 552707 and bug 644621. And I'll stay the state of all regression bugs. I need to keep to work on the regressions because the next cycle is shorter than current cycle and we have all hands meeting at mid September.
Summary: Backout new drag for selection handling code from Mozilla8 → Backout new drag for selection handling code from Mozilla8 and reland for Mozilla9
Do you want to back out before the Aurora merge, or back out from Aurora right after the merge?
(In reply to comment #1)
> Do you want to back out before the Aurora merge, or back out from Aurora
> right after the merge?

Before. Because testers of nightly should test the backed out build.
Blocks: 675456
Comment on attachment 550011 [details] [diff] [review]
backout patch

http://tbpl.mozilla.org/?tree=Try&pusher=masayuki@d-toybox.com&rev=751909a71c17
Attachment #550011 - Flags: review?(Olli.Pettay)

Comment 4

6 years ago
Comment on attachment 550011 [details] [diff] [review]
backout patch

In the commit message please add all the bug numbers this patch is
backing out.

Other option is to back out each bug individually.
I think I'd prefer that.

But either way
rs=me
Attachment #550011 - Flags: review?(Olli.Pettay) → review+
backed-out:
http://hg.mozilla.org/mozilla-central/rev/d32bc792fb5e
smaug:

I'd like to reland the backed out patches with bug 671187 and bug 675614. Would you review them?
bug 672181 is a serious regression of the series of these bugs. It is especially reproduced with the patch for bug 644621 and <input> or <textarea> element since the size of them is typically small.

I'm creating some patches which decelerate auto scroll speed in the bug. At least, we need to finish the bug before reland them.

However, we might have selected wrong approach for bug 644621...

Updated

6 years ago
Target Milestone: mozilla8 → ---
You need to log in before you can comment on or make changes to this bug.