superreview requested: [Bug 328128] nsIPrefBranch:: prefHasUserValue should document the fact that a pref no longer has a user set value if it is set to its default value

Nickolay_Ponomarev <asqueella@gmail.com> has asked Darin Fisher (Google)
<darin.moz@gmail.com> for superreview:
Bug 328128: nsIPrefBranch::prefHasUserValue should document the fact that a
pref no longer has a user set value if it is set to its default value
https://bugzilla.mozilla.org/show_bug.cgi?id=328128

Attachment 249861: patch
https://bugzilla.mozilla.org/attachment.cgi?id=249861&action=edit
0
bugzilla
12/28/2006 4:08:55 PM
mozilla.dev.super-review 29307 articles. 3 followers. Post Follow

0 Replies
579 Views

Similar Articles

[PageSpeed] 46

Reply:

Similar Artilces:

superreview granted: [Bug 328128] nsIPrefBranch:: prefHasUserValue should document the fact that a pref no longer has a user set value if it is set to its default value
Darin Fisher (Google) <darin.moz@gmail.com> has granted Nickolay_Ponomarev <asqueella@gmail.com>'s request for superreview: Bug 328128: nsIPrefBranch::prefHasUserValue should document the fact that a pref no longer has a user set value if it is set to its default value https://bugzilla.mozilla.org/show_bug.cgi?id=328128 Attachment 249861: patch https://bugzilla.mozilla.org/attachment.cgi?id=249861&action=edit ------- Additional Comments from Darin Fisher (Google) <darin.moz@gmail.com> >Index: modules/libpref/public/nsIPrefBranch.idl >+ * conside...

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 197466] Back button (session history) has 50 entries regardless of setting in prefs.js : [Attachment 305047] Simple patch to follow user prefs (> default) again, init varia
zug_treno@yahoo.com has asked Darin Fisher <darin.moz@gmail.com> for superreview: Bug 197466: Back button (session history) has 50 entries regardless of setting in prefs.js https://bugzilla.mozilla.org/show_bug.cgi?id=197466 Attachment 305047: Simple patch to follow user prefs (> default) again, init variable to 50 https://bugzilla.mozilla.org/attachment.cgi?id=305047&action=edit ------- Additional Comments from zug_treno@yahoo.com Initializes defaultHistoryMaxSize to 50. ...

superreview requested: [Bug 378860] If setAttribute sets an invalid value, the attribute default should be used
Robert Longson <longsonr@gmail.com> has asked Robert O'Callahan (:roc) <roc@ocallahan.org> for superreview: Bug 378860: If setAttribute sets an invalid value, the attribute default should be used https://bugzilla.mozilla.org/show_bug.cgi?id=378860 Attachment 285088: patch https://bugzilla.mozilla.org/attachment.cgi?id=285088&action=edit ...

superreview requested: [Bug 216751] general.config.filename set in user' s prefs.js overrides setting in all.js
David Bienvenu <bienvenu@nventure.com> has asked Scott MacGregor <mscott@mozilla.org> for superreview: Bug 216751: general.config.filename set in user's prefs.js overrides setting in all.js https://bugzilla.mozilla.org/show_bug.cgi?id=216751 Attachment 253396: fix MCD https://bugzilla.mozilla.org/attachment.cgi?id=253396&action=edit ...

superreview requested: [Bug 378860] If setAttribute sets an invalid value, the attribute default should be used #2
Robert Longson <longsonr@gmail.com> has asked Robert O'Callahan (:roc) <roc@ocallahan.org> for superreview: Bug 378860: If setAttribute sets an invalid value, the attribute default should be used https://bugzilla.mozilla.org/show_bug.cgi?id=378860 Attachment 285857: address superreview comments https://bugzilla.mozilla.org/attachment.cgi?id=285857&action=edit ...

superreview requested: [Bug 403480] setting nsIURI:: port to its default value shouldn't be possible
Gavin Sharp <gavin.sharp@gmail.com> has asked Christian :Biesinger <cbiesinger@gmx.at> for superreview: Bug 403480: setting nsIURI::port to its default value shouldn't be possible https://bugzilla.mozilla.org/show_bug.cgi?id=403480 Attachment 288311: patch https://bugzilla.mozilla.org/attachment.cgi?id=288311&action=edit ...

superreview requested: [Bug 216751] general.config.filename set in user's prefs.js overrides setting in all.js : [Attachment 189091] Patch
Frank Wein <bugzilla@mcsmurf.de> has asked Brendan Eich <brendan@mozilla.org> for superreview: Bug 216751: general.config.filename set in user's prefs.js overrides setting in all.js https://bugzilla.mozilla.org/show_bug.cgi?id=216751 Attachment 189091: Patch https://bugzilla.mozilla.org/attachment.cgi?id=189091&action=edit ...

superreview requested: [Bug 219848] Cannot programmatically (JavaScript) set value of HTMLInputElement : [Attachment 134580] Fix for nsDOMClassInfo v1 (diff -w for review only)
Peter Van der Beken <peterv@propagandism.org> has asked Johnny Stenback <jst@mozilla.jstenback.com> for superreview: Bug 219848: Cannot programmatically (JavaScript) set value of HTMLInputElement http://bugzilla.mozilla.org/show_bug.cgi?id=219848 Attachment 134580: Fix for nsDOMClassInfo v1 (diff -w for review only) http://bugzilla.mozilla.org/attachment.cgi?id=134580&action=edit ------- Additional Comments from Peter Van der Beken <peterv@propagandism.org> We don't want to do security checks on defining DOM classes. Access to the DOM classes (eg. aWindo...

superreview requested: [Bug 262624] normal (default value) content in generated text with :after and :before does not override more general rule with content set : [Attachment 161166] Patch
Boris Zbarsky <bzbarsky@mit.edu> has asked David Baron <dbaron@dbaron.org> for superreview: Bug 262624: normal (default value) content in generated text with :after and :before does not override more general rule with content set https://bugzilla.mozilla.org/show_bug.cgi?id=262624 Attachment 161166: Patch https://bugzilla.mozilla.org/attachment.cgi?id=161166&action=edit ------- Additional Comments from Boris Zbarsky <bzbarsky@mit.edu> This just adds "normal" a as a recognized content value and skips content computation altogether if that's the c...

superreview granted: [Bug 407538] newURI("http://foo.com:80").port is 80, should be -1 (can set a URI's port to its default value by setting .spec) : [Attachment 292260] patch
Christian :Biesinger <cbiesinger@gmx.at> has granted Gavin Sharp <gavin.sharp@gmail.com>'s request for superreview: Bug 407538: newURI("http://foo.com:80").port is 80, should be -1 (can set a URI's port to its 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 Christian :Biesinger <cbiesinger@gmx.at> would be good to also test that .spec doesn't include the :80 either + //XXX URL ...

superreview requested: [Bug 319037] Provide default value for mailnews.localizedRe : [Attachment 205555] Allow localizers to specify default value
David Bienvenu <bienvenu@nventure.com> has asked Scott MacGregor (out of town December 10th-17th) <mscott@mozilla.org> for superreview: Bug 319037: Provide default value for mailnews.localizedRe https://bugzilla.mozilla.org/show_bug.cgi?id=319037 Attachment 205555: Allow localizers to specify default value https://bugzilla.mozilla.org/attachment.cgi?id=205555&action=edit ------- Additional Comments from David Bienvenu <bienvenu@nventure.com> it depends on the patch :-) In this case, I'm going to check with mscott - I don't think he'll have a proble...

The value set to controls at client side donot persist for each page load, else it is again set to the default value...??
 I am trying to set date to the textbox field using javascript (Client side), the function is as follows. function From_calendarPicker(){                                var    sDPath = "../Calender.aspx";                 var    oWin = window.showModalDialog(sDPath, null, "dialogWidth:300px;dialogHeight:250px;help:0;status:0;scroll:0;center:1");  &nbs...

superreview requested: [Bug 451232] Setting ui.textHighlightBackground pref to empty string or removing pref doesn't reset directly default highlight : [Attachment 334679] Patch v1
Graeme McCutcheon <graememcc_firefox@graeme-online.co.uk> has asked Robert O'Callahan (:roc) (Mozilla Corporation) <roc@ocallahan.org> for superreview: Bug 451232: Setting ui.textHighlightBackground pref to empty string or removing pref doesn't reset directly default highlight https://bugzilla.mozilla.org/show_bug.cgi?id=451232 Attachment 334679: Patch v1 https://bugzilla.mozilla.org/attachment.cgi?id=334679&action=edit ------- Additional Comments from Graeme McCutcheon <graememcc_firefox@graeme-online.co.uk> So, this is definitely not unique to the u...

Web resources about - superreview requested: [Bug 328128] nsIPrefBranch:: prefHasUserValue should document the fact that a pref no longer has a user set value if it is set to its default value - mozilla.dev.super-review

Resources last updated: 12/23/2015 11:15:00 AM