Closed Bug 695640 Opened 13 years ago Closed 13 years ago

Profile View Source pre chunking and remove it if unnecessary

Categories

(Core :: DOM: HTML Parser, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: hsivonen, Assigned: hsivonen)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

Now that bug 263359 has been fixed, the part of bug 482921 that refixes bug 86355 may be unnecessary. Find out if it is unnecessary and remove if unnecessary.
I profiled View Source on a Hebrew news article and an Arabic news article with and without this patch and there seemed to be no interesting differences. AFAICT, the pre chunking is no longer needed, so let's get rid of it.
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #577611 - Flags: review?(bugs)
Attachment #577611 - Flags: review?(bugs)
Attachment #577611 - Flags: review+
Attachment #577611 - Flags: feedback?(smontagu)
Comment on attachment 577611 [details] [diff] [review]
Remove pre chunking

Review of attachment 577611 [details] [diff] [review]:
-----------------------------------------------------------------

I'm not sure what feedback you want from me here. I'm all for doing this, given that there's no perf impact.
(In reply to Simon Montagu from comment #2)
> I'm all for doing this, given that there's no perf impact.

Great. Landed:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c5d44e6e957b
Backed out on inbound (along with two other bugs from the same push) because of test failures:
https://hg.mozilla.org/integration/mozilla-inbound/rev/20a24dd3f56b

If this wasn't responsible for the failures, it can re-land.
https://hg.mozilla.org/mozilla-central/rev/fcba47973317
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
Comment on attachment 577611 [details] [diff] [review]
Remove pre chunking

(This comment has been posted on all the bugs mentioned in this comment, except bug 710142, so that the release drivers see it regardless of the order in which they process approval requests.)

The new View Source implementation landed before Firefox 10 moved to Aurora. Afterwards, a bunch of regressions were identified. Many of the regression fixes didn't land before Firefox 10 moved to Aurora but they have now been fixed on trunk except for bug 710142.

To avoid shipping with regressions, we either need to land all the regression fixes on Aurora for Firefox 10 (followed by a fix for bug 710142 in Beta if it doesn't make it before Dec 20th) or switch back to the old View Source implementation on Aurora. The new View Source implementation provides much better diagnostics for Web developers than the old View Source implementation.

So I'd like to ask the release drivers to either approve bug 535530, bug 699356, bug 699365, bug 700034, bug 700361, bug 703965, bug 704667 and bug 705473 plus bug 695640, which is a non-regression tweak, or to approve bug 710175 for reverting to the old View Source implementation for Firefox 10.
Attachment #577611 - Flags: feedback?(smontagu) → approval-mozilla-aurora?
Comment on attachment 577611 [details] [diff] [review]
Remove pre chunking

[triage comment]
We decided to back out the new view source parser (bug 710175) rather than take this fixup for Firefox 10.

Denying for Aurora.
Attachment #577611 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora-
Depends on: 1466704
You need to log in before you can comment on or make changes to this bug.