superreview requested: [Bug 439601] use of wbr tag by YahooGroups corrupts text in simple HTML view : [Attachment 325496] patch for review

alta88 <alta88@gmail.com> has asked neil@parkwaycc.co.uk <neil@httl.net> for
superreview:
Bug 439601: use of wbr tag by YahooGroups corrupts text in simple HTML view
https://bugzilla.mozilla.org/show_bug.cgi?id=439601

Attachment 325496: patch for review
https://bugzilla.mozilla.org/attachment.cgi?id=325496&action=edit

------- Additional Comments from alta88 <alta88@gmail.com>
sr? neil, please checkin while you're at it..
0
bugzilla
6/18/2008 3:09:10 PM
mozilla.dev.super-review 29307 articles. 3 followers. Post Follow

0 Replies
734 Views

Similar Articles

[PageSpeed] 8
Get it on Google Play
Get it on Apple App Store

Reply:

Similar Artilces:

superreview granted: [Bug 439601] use of wbr tag by YahooGroups corrupts text in simple HTML view : [Attachment 325496] patch for review
neil@parkwaycc.co.uk <neil@httl.net> has granted alta88 <alta88@gmail.com>'s request for superreview: Bug 439601: use of wbr tag by YahooGroups corrupts text in simple HTML view https://bugzilla.mozilla.org/show_bug.cgi?id=439601 Attachment 325496: patch for review https://bugzilla.mozilla.org/attachment.cgi?id=325496&action=edit ...

superreview requested: [Bug 313217] document.open('replace') now opens text/plain document where it used to open text/html : [Attachment 200309] Patch to make a type of "replace" mean HTML but not re
Boris Zbarsky <bzbarsky@mit.edu> has asked Johnny Stenback <jst@mozilla.org> for superreview: Bug 313217: document.open('replace') now opens text/plain document where it used to open text/html https://bugzilla.mozilla.org/show_bug.cgi?id=313217 Attachment 200309: Patch to make a type of "replace" mean HTML but not replace https://bugzilla.mozilla.org/attachment.cgi?id=200309&action=edit ------- Additional Comments from Boris Zbarsky <bzbarsky@mit.edu> Just treat "replace" in the first slot as if nothing were set. ...

superreview requested: [Bug 393002] "Server not found" message for view-source window doesn't display server name : [ Attachment 292031] patch v2: address issue raised by super-review
Anant Narayanan <anant@kix.in> has asked Boris Zbarsky (reviews very slow until spring) <bzbarsky@mit.edu> for superreview: Bug 393002: "Server not found" message for view-source window doesn't display server name https://bugzilla.mozilla.org/show_bug.cgi?id=393002 Attachment 292031: patch v2: address issue raised by super-review https://bugzilla.mozilla.org/attachment.cgi?id=292031&action=edit ...

superreview requested: [Bug 249903] nsGNOMERegistry::HandlerExists uses gconf client after g_object_unref()ing it : [Attachment 152362] updated patch as requested by the reviewer
Christian Persch <chpe@gnome.org> has asked Boris Zbarsky (out of town June 14 -- July 11, no email then) <bzbarsky@mit.edu> for superreview: Bug 249903: nsGNOMERegistry::HandlerExists uses gconf client after g_object_unref()ing it http://bugzilla.mozilla.org/show_bug.cgi?id=249903 Attachment 152362: updated patch as requested by the reviewer http://bugzilla.mozilla.org/attachment.cgi?id=152362&action=edit ------- Additional Comments from Christian Persch <chpe@gnome.org> Carrying forward bryner's r+, and asking for sr ...

superreview requested: [Bug 26622] nsAutoString use as member variables should be reviewed : [Attachment 172796] mailnews patch
Mikael Parknert <mikael@parknert.se> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 26622: nsAutoString use as member variables should be reviewed https://bugzilla.mozilla.org/show_bug.cgi?id=26622 Attachment 172796: mailnews patch https://bugzilla.mozilla.org/attachment.cgi?id=172796&action=edit ...

superreview requested: [Bug 26622] nsAutoString use as member variables should be reviewed : [Attachment 206922] mailnews patch v2
Mark Banner <bugzilla@standard8.demon.co.uk> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 26622: nsAutoString use as member variables should be reviewed https://bugzilla.mozilla.org/show_bug.cgi?id=26622 Attachment 206922: mailnews patch v2 https://bugzilla.mozilla.org/attachment.cgi?id=206922&action=edit ------- Additional Comments from Mark Banner <bugzilla@standard8.demon.co.uk> As Mikael is not currently working on Mozilla, I've updated his patch which had (r+ and sr+) with respect to Neil's comments and produced this new o...

superreview requested: [Bug 26622] nsAutoString use as member variables should be reviewed : [Attachment 207854] mailnews patch v4
Mark Banner <bugzilla@standard8.demon.co.uk> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 26622: nsAutoString use as member variables should be reviewed https://bugzilla.mozilla.org/show_bug.cgi?id=26622 Attachment 207854: mailnews patch v4 https://bugzilla.mozilla.org/attachment.cgi?id=207854&action=edit ------- Additional Comments from Mark Banner <bugzilla@standard8.demon.co.uk> Requesting sr, I'll address Neil's review comments on check in. ...

superreview requested: [Bug 130728] messageBody.css should use namespaces correctly : [Attachment 159996] Patch to address review comments
Justin Wood (Callek on irc) <116057@bacon.qcc.mass.edu> has asked neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> for superreview: Bug 130728: messageBody.css should use namespaces correctly https://bugzilla.mozilla.org/show_bug.cgi?id=130728 Attachment 159996: Patch to address review comments https://bugzilla.mozilla.org/attachment.cgi?id=159996&action=edit ...

superreview requested: [Bug 295715] HTML table should use summary attribute for accessible name : [Attachment 184685] Simple patch
Aaron Leventhal <aaronleventhal@moonset.net> has asked neil@parkwaycc.co.uk <neil.parkwaycc.co.uk@myrealbox.com> for superreview: Bug 295715: HTML table should use summary attribute for accessible name https://bugzilla.mozilla.org/show_bug.cgi?id=295715 Attachment 184685: Simple patch https://bugzilla.mozilla.org/attachment.cgi?id=184685&action=edit ...

superreview requested: [Bug 295401] Allow DHTML accessibility to be used in text/html : [Attachment 184437] If text/html, allow wairole to automatically expand to the WAI role namespace
Aaron Leventhal <aaronleventhal@moonset.net> has asked Johnny Stenback <jst@mozilla.org> for superreview: Bug 295401: Allow DHTML accessibility to be used in text/html https://bugzilla.mozilla.org/show_bug.cgi?id=295401 Attachment 184437: If text/html, allow wairole to automatically expand to the WAI role namespace https://bugzilla.mozilla.org/attachment.cgi?id=184437&action=edit ...

superreview requested: [Bug 258769] Try using nsString more in addrbook to reduce conversions : [Attachment 158668] patch adressing review comments
Mikael Parknert <mikael@parknert.se> has asked David Baron <dbaron@dbaron.org> for superreview: Bug 258769: Try using nsString more in addrbook to reduce conversions http://bugzilla.mozilla.org/show_bug.cgi?id=258769 Attachment 158668: patch adressing review comments http://bugzilla.mozilla.org/attachment.cgi?id=158668&action=edit ...

superreview granted: [Bug 227986] EmbedPrompter should use GtkComboBox on gtk+ 2.4 : [Attachment 137398] updated patch with the changes requested by the reviewer
Christopher Blizzard <blizzard@mozilla.org> has granted Christopher Blizzard <blizzard@mozilla.org>'s request for superreview: Bug 227986: EmbedPrompter should use GtkComboBox on gtk+ 2.4 http://bugzilla.mozilla.org/show_bug.cgi?id=227986 Attachment 137398: updated patch with the changes requested by the reviewer http://bugzilla.mozilla.org/attachment.cgi?id=137398&action=edit ...

superreview granted: [Bug 249903] nsGNOMERegistry::HandlerExists uses gconf client after g_object_unref()ing it : [Attachment 152362] updated patch as requested by the reviewer
Darin Fisher (IBM) <darin@meer.net> has granted Christian Persch <chpe@gnome.org>'s request for superreview: Bug 249903: nsGNOMERegistry::HandlerExists uses gconf client after g_object_unref()ing it http://bugzilla.mozilla.org/show_bug.cgi?id=249903 Attachment 152362: updated patch as requested by the reviewer http://bugzilla.mozilla.org/attachment.cgi?id=152362&action=edit ------- Additional Comments from Darin Fisher (IBM) <darin@meer.net> >Index: nsGNOMERegistry.cpp >+ return isEnabled ? PR_TRUE : PR_FALSE; return isEnabled != FALSE; ...

superreview requested: [Bug 224053] Simple HTML (sanitizer) message view disallows numbered list (OL) with non-default start : [Attachment 134410] Patch
Malcolm Rowe <malcolm-bmo@farside.org.uk> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 224053: Simple HTML (sanitizer) message view disallows numbered list (OL) with non-default start http://bugzilla.mozilla.org/show_bug.cgi?id=224053 Attachment 134410: Patch http://bugzilla.mozilla.org/attachment.cgi?id=134410&action=edit ...

superreview requested: [Bug 58305] Find in page ignores text fields - does not search form textarea : [Attachment 145985] patch
rbs@maths.uq.edu.au has asked Simon Fraser <sfraser@aol.net> for superreview: Bug 58305: Find in page ignores text fields - does not search form textarea http://bugzilla.mozilla.org/show_bug.cgi?id=58305 Attachment 145985: patch - updated per review comments http://bugzilla.mozilla.org/attachment.cgi?id=145985&action=edit ...

superreview requested: [Bug 214839] Need shared PLDHashTableOps for [const] char *key use-cases : [Attachment 129077] diff -w version of last patch (review this)
Brendan Eich <brendan@mozilla.org> has asked David Baron <dbaron@dbaron.org> for superreview: Bug 214839: Need shared PLDHashTableOps for [const] char *key use-cases http://bugzilla.mozilla.org/show_bug.cgi?id=214839 Attachment 129077: diff -w version of last patch (review this) http://bugzilla.mozilla.org/attachment.cgi?id=129077&action=edit ------- Additional Comments from Brendan Eich <brendan@mozilla.org> Looking for reviews of relevant files by all on the cc: list, and I'd like to get this in for 1.5b, which means in the next day or two, realistical...

superreview requested: [Bug 57342] Add "View as Text/HTML/..." option for unknown mime content-type : [Attachment 176937] backend patch, v4
Christian Biesinger (:bi) <cbiesinger@gmx.at> has asked Darin Fisher <darin@meer.net> for superreview: Bug 57342: Add "View as Text/HTML/..." option for unknown mime content-type https://bugzilla.mozilla.org/show_bug.cgi?id=57342 Attachment 176937: backend patch, v4 https://bugzilla.mozilla.org/attachment.cgi?id=176937&action=edit ------- Additional Comments from Christian Biesinger (:bi) <cbiesinger@gmx.at> transferring r=bz this includes the following change for onStopRequest/NS_BINDING_REDIRECTED: (the if for the else is if (!oldListener)) ---...

superreview requested: [Bug 58305] Find in page ignores text fields - does not search form textarea : [Attachment 145985] patch
rbs@maths.uq.edu.au has asked Johnny Stenback <jst@mozilla.jstenback.com> for superreview: Bug 58305: Find in page ignores text fields - does not search form textarea http://bugzilla.mozilla.org/show_bug.cgi?id=58305 Attachment 145985: patch - updated per review comments http://bugzilla.mozilla.org/attachment.cgi?id=145985&action=edit ...

superreview requested: [Bug 58305] Find in page ignores text fields - does not search form textarea : [Attachment 145985] patch
Simon Fraser <sfraser@aol.net> has asked Simon Fraser <sfraser@aol.net> for superreview: Bug 58305: Find in page ignores text fields - does not search form textarea http://bugzilla.mozilla.org/show_bug.cgi?id=58305 Attachment 145985: patch - updated per review comments http://bugzilla.mozilla.org/attachment.cgi?id=145985&action=edit ------- Additional Comments from Simon Fraser <sfraser@aol.net> Sorry, this is a large patch and I don't have time to superreview. ...

superreview requested: [Bug 307527] Connection timeout with IMAP on dual core systems : [Attachment 208675] Proposed patch: use timeGetTime (whitespace ignored for code review)
Wan-Teh Chang <wtchang@redhat.com> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 307527: Connection timeout with IMAP on dual core systems https://bugzilla.mozilla.org/show_bug.cgi?id=307527 Attachment 208675: Proposed patch: use timeGetTime (whitespace ignored for code review) https://bugzilla.mozilla.org/attachment.cgi?id=208675&action=edit ------- Additional Comments from Wan-Teh Chang <wtchang@redhat.com> The original code uses QueryPerformanceCounter if it works, and falls back on GetTickCount. This patch essentially has two ch...

superreview requested: [Bug 250386] Camino should look for user defined earchURLList.plist first before using the default list. : [Attachment 163203] Updated patch addressing review comments
Ludovic Hirlimann <qa-mozilla@hirlimann.net> has asked Mike Pinkerton <pinkerton@aol.net> for superreview: Bug 250386: Camino should look for user defined earchURLList.plist first before using the default list. https://bugzilla.mozilla.org/show_bug.cgi?id=250386 Attachment 163203: Updated patch addressing review comments https://bugzilla.mozilla.org/attachment.cgi?id=163203&action=edit ------- Additional Comments from Ludovic Hirlimann <qa-mozilla@hirlimann.net> asking for sr ...

superreview granted: [Bug 313217] document.open('replace') now opens text/plain document where it used to open text/html : [Attachment 200309] Patch to make a type of "replace" mean HTML but not repl
Johnny Stenback <jst@mozilla.org> has granted Boris Zbarsky <bzbarsky@mit.edu>'s request for superreview: Bug 313217: document.open('replace') now opens text/plain document where it used to open text/html https://bugzilla.mozilla.org/show_bug.cgi?id=313217 Attachment 200309: Patch to make a type of "replace" mean HTML but not replace https://bugzilla.mozilla.org/attachment.cgi?id=200309&action=edit ------- Additional Comments from Johnny Stenback <jst@mozilla.org> r+sr=jst ...

superreview requested: [Bug 410215] GTK's .xpm decoding is weird, stop using XPM as the default window icon : [Attachment 295160] Patch v. 1 for review
Justin Dolske <dolske@mozilla.com> has asked Robert O'Callahan (:roc) <roc@ocallahan.org> for superreview: Bug 410215: GTK's .xpm decoding is weird, stop using XPM as the default window icon https://bugzilla.mozilla.org/show_bug.cgi?id=410215 Attachment 295160: Patch v.1 for review https://bugzilla.mozilla.org/attachment.cgi?id=295160&action=edit ------- Additional Comments from Justin Dolske <dolske@mozilla.com> This changes the icon setting code to prefer a PNG, and if no PNG is found it falls back to looking for an XPM. This allow us to start switc...

superreview requested: [Bug 224053] Simple HTML (sanitizer) message view disallows numbered list (OL) with non-default start : [Attachment 134465] Patch v2
Malcolm Rowe <malcolm-bmo@farside.org.uk> has asked David Bienvenu <bienvenu@nventure.com> for superreview: Bug 224053: Simple HTML (sanitizer) message view disallows numbered list (OL) with non-default start http://bugzilla.mozilla.org/show_bug.cgi?id=224053 Attachment 134465: Patch v2 - allow type on LI http://bugzilla.mozilla.org/attachment.cgi?id=134465&action=edit ...

Web resources about - superreview requested: [Bug 439601] use of wbr tag by YahooGroups corrupts text in simple HTML view : [Attachment 325496] patch for review - mozilla.dev.super-review

Resources last updated: 2/28/2016 8:03:47 PM