superreview requested: [Bug 280713] ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly : [Attachment 173127] patch v1

Blake Kaplan <mrbkap@gmail.com> has asked Boris Zbarsky <bzbarsky@mit.edu> for
superreview:
Bug 280713: ContinueInterruptedParsing doesn't set the current context's
scanner's mIncremental member correctly
https://bugzilla.mozilla.org/show_bug.cgi?id=280713

Attachment 173127: patch v1
https://bugzilla.mozilla.org/attachment.cgi?id=173127&action=edit
0
bugzilla
2/7/2005 8:30:37 PM
netscape.mozilla.reviewers 29156 articles. 0 followers. Follow

0 Replies
626 Views

Similar Articles

[PageSpeed] 42

Reply:

Similar Artilces:

superreview granted: [Bug 280713] ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly : [Attachment 173127] patch v1
Boris Zbarsky <bzbarsky@mit.edu> has granted Blake Kaplan <mrbkap@gmail.com>'s request for superreview: Bug 280713: ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly https://bugzilla.mozilla.org/show_bug.cgi?id=280713 Attachment 173127: patch v1 https://bugzilla.mozilla.org/attachment.cgi?id=173127&action=edit ------- Additional Comments from Boris Zbarsky <bzbarsky@mit.edu> sr=bzbarsky, but it sounds like some cleanup of incremental vs final chunk is in order.... ...

superreview requested: [Bug 280713] ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly : [Attachment 173997] updated to review comments
Blake Kaplan <mrbkap@gmail.com> has asked Boris Zbarsky <bzbarsky@mit.edu> for superreview: Bug 280713: ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly https://bugzilla.mozilla.org/show_bug.cgi?id=280713 Attachment 173997: updated to review comments https://bugzilla.mozilla.org/attachment.cgi?id=173997&action=edit ...

superreview granted: [Bug 280713] ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly : [Attachment 173997] updated to review comments
Boris Zbarsky <bzbarsky@mit.edu> has granted Blake Kaplan <mrbkap@gmail.com>'s request for superreview: Bug 280713: ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly https://bugzilla.mozilla.org/show_bug.cgi?id=280713 Attachment 173997: updated to review comments https://bugzilla.mozilla.org/attachment.cgi?id=173997&action=edit ------- Additional Comments from Boris Zbarsky <bzbarsky@mit.edu> sr=bzbarsky. Land this either _really_ early in 1.8b2 or in 1.9a.... ...

superreview requested: [Bug 122238] input type="image" doesn't send x/y (image doesn't exist; width and height not set) : [Attachment 139545] Same as jkeiser's patch, really...
Boris Zbarsky <bz-vacation@mit.edu> has asked Johnny Stenback <jst@mozilla.jstenback.com> for superreview: Bug 122238: input type="image" doesn't send x/y (image doesn't exist; width and height not set) http://bugzilla.mozilla.org/show_bug.cgi?id=122238 Attachment 139545: Same as jkeiser's patch, really... http://bugzilla.mozilla.org/attachment.cgi?id=139545&action=edit ...

superreview granted: [Bug 122238] input type="image" doesn't send x/y (image doesn't exist; width and height not set) : [Attachment 139545] Same as jkeiser's patch, really...
Johnny Stenback <jst@mozilla.jstenback.com> has granted Boris Zbarsky <bz-vacation@mit.edu>'s request for superreview: Bug 122238: input type="image" doesn't send x/y (image doesn't exist; width and height not set) http://bugzilla.mozilla.org/show_bug.cgi?id=122238 Attachment 139545: Same as jkeiser's patch, really... http://bugzilla.mozilla.org/attachment.cgi?id=139545&action=edit ------- Additional Comments from Johnny Stenback <jst@mozilla.jstenback.com> + if (!name.IsEmpty()) { + aFormSubmission->AddNameValuePair(thi...

superreview requested: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 163330] corrects neil's comments
Jean-Marc Desperrier <jmdesp@alussinan.org> has asked neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 163330: corrects neil's comments https://bugzilla.mozilla.org/attachment.cgi?id=163330&action=edit ...

superreview requested: [Bug 271959] Mozilla's About page doesn't know how to spell its own name in its license : [Attachment 167171] patch
Alex Vincent <ajvincent@gmail.com> has asked Boris Zbarsky (out of town Dec 10 -- Jan 2) <bzbarsky@mit.edu> for superreview: Bug 271959: Mozilla's About page doesn't know how to spell its own name in its license https://bugzilla.mozilla.org/show_bug.cgi?id=271959 Attachment 167171: patch https://bugzilla.mozilla.org/attachment.cgi?id=167171&action=edit ------- Additional Comments from Alex Vincent <ajvincent@gmail.com> This may or may not require strong review; it's a comment-only change, really a nitpick. ...

superreview requested: [Bug 346119] Code added in bug 338122 doesn't actually enforce document equality : [Attachment 231004] update prescontext's mDocument when new shell is set.
Smaug <Olli.Pettay@gmail.com> has asked Boris Zbarsky <bzbarsky@mit.edu> for superreview: Bug 346119: Code added in bug 338122 doesn't actually enforce document equality https://bugzilla.mozilla.org/show_bug.cgi?id=346119 Attachment 231004: update prescontext's mDocument when new shell is set. https://bugzilla.mozilla.org/attachment.cgi?id=231004&action=edit ------- Additional Comments from Smaug <Olli.Pettay@gmail.com> This still ensures that nsPresContext::Document() never returns nsnull. ...

superreview requested: [Bug 407538] newURI("http://foo.com:80").port is 80, should be -1 (can set a URI's port to it's default value by setting .spec) : [Attachment 292260] patch
Gavin Sharp <gavin.sharp@gmail.com> has asked Christian :Biesinger <cbiesinger@gmx.at> for superreview: Bug 407538: newURI("http://foo.com:80").port is 80, should be -1 (can set a URI's port to it's default value by setting .spec) https://bugzilla.mozilla.org/show_bug.cgi?id=407538 Attachment 292260: patch https://bugzilla.mozilla.org/attachment.cgi?id=292260&action=edit ------- Additional Comments from Gavin Sharp <gavin.sharp@gmail.com> I'm not sure about the XXX comment in the test - seems like we should probably fix that, too, but I&...

superreview requested: [Bug 123006] Keyword substitution with %s doesn't escape characters : [Attachment 162990] nit corrected mozilla patch
Jean-Marc Desperrier <jmdesp@alussinan.org> has asked neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> for superreview: Bug 123006: Keyword substitution with %s doesn't escape characters https://bugzilla.mozilla.org/show_bug.cgi?id=123006 Attachment 162990: nit corrected mozilla patch https://bugzilla.mozilla.org/attachment.cgi?id=162990&action=edit ...

superreview requested: [Bug 238733] Components.classes['@mozilla.org/wallet/wallet-service;1'] has no properties in contentAreaClick.js : [Attachment 144811] don't call the function if it's going to
timeless@myrealbox.com (working) <timeless@bemail.org> has asked Boris Zbarsky <bzbarsky@mit.edu> for superreview: Bug 238733: Components.classes['@mozilla.org/wallet/wallet-service;1'] has no properties in contentAreaClick.js https://bugzilla.mozilla.org/show_bug.cgi?id=238733 Attachment 144811: don't call the function if it's going to fail https://bugzilla.mozilla.org/attachment.cgi?id=144811&action=edit ...

superreview requested: [Bug 288638] get rid of 'character' member (of MozXftLangGroup) that's not used any more : [Attachment 179274] patch
Jungshik Shin <jshin1987@gmail.com> has asked David Baron <dbaron@mozilla.org> for superreview: Bug 288638: get rid of 'character' member (of MozXftLangGroup) that's not used any more https://bugzilla.mozilla.org/show_bug.cgi?id=288638 Attachment 179274: patch https://bugzilla.mozilla.org/attachment.cgi?id=179274&action=edit ...

superreview granted: [Bug 279533] Update Camino's font preferences code per bug 95227 (serif/sans-serif set is now per language) / serif / sans-serif pref ui doesn't work : [Attachment 174475] patch
Simon Fraser <sfraser_bugs@smfr.org> has granted Asaf Romano <bugs.mano@mail-central.com>'s request for superreview: Bug 279533: Update Camino's font preferences code per bug 95227 (serif/sans-serif set is now per language) / serif / sans-serif pref ui doesn't work https://bugzilla.mozilla.org/show_bug.cgi?id=279533 Attachment 174475: patch v1 https://bugzilla.mozilla.org/attachment.cgi?id=174475&action=edit ...

superreview requested: [Bug 279533] Update Camino's font preferences code per bug 95227 (serif/sans-serif set is now per language) / serif / sans-serif pref ui doesn't work : [Attachment 174475] patc
Asaf Romano <bugs.mano@mail-central.com> has asked Mike Pinkerton <pinkerton@aol.net> for superreview: Bug 279533: Update Camino's font preferences code per bug 95227 (serif/sans-serif set is now per language) / serif / sans-serif pref ui doesn't work https://bugzilla.mozilla.org/show_bug.cgi?id=279533 Attachment 174475: patch v1 https://bugzilla.mozilla.org/attachment.cgi?id=174475&action=edit ...

Web resources about - superreview requested: [Bug 280713] ContinueInterruptedParsing doesn't set the current context's scanner's mIncremental member correctly : [Attachment 173127] patch v1 - netscape.mozilla.reviewers

Resources last updated: 12/6/2015 2:10:16 PM