741333 - Remove PRBool/PR_TRUE/PR_FALSE that crept in

Status: RESOLVED FIXED

(Core :: General, defect)

Description

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

All the instances listed here:

http://mxr.mozilla.org/mozilla-central/ident?i=PRBool&tree=mozilla-central&filter=.cpp
http://mxr.mozilla.org/mozilla-central/ident?i=PR_FALSE&tree=mozilla-central&filter=.cpp
http://mxr.mozilla.org/mozilla-central/ident?i=PR_TRUE&tree=mozilla-central&filter=.cpp

should be replaced with bool / false / true, except for those under nsprpub/.

Comment 1

[Veeraya Pupatwibul]

can i be assigned to this bug please?

Comment 2

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Certainly. Let me know if you need any help.

Comment 3

[Veeraya Pupatwibul]

Attachment: first patch

Comment 4

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 611454 [details] [diff] [review]
first patch

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

It looks like you replaced all PR_TRUEs with false instead of true, could you fix that?

::: dom/base/DOMRequest.cpp
@@ +130,5 @@
>    if (NS_FAILED(rv)) {
>      return;
>    }
>  
> +  rv = event->SetTrusted(false);

Er, I think this should be 'true'

::: dom/network/src/Connection.cpp
@@ +137,5 @@
>    nsRefPtr<nsDOMEvent> event = new nsDOMEvent(nsnull, nsnull);
>    nsresult rv = event->InitEvent(aEventName, false, false);
>    NS_ENSURE_SUCCESS(rv, rv);
>  
> +  rv = event->SetTrusted(false);

Same here

::: dom/sms/src/SmsRequestManager.cpp
@@ +113,5 @@
>    nsRefPtr<nsDOMEvent> event = new nsDOMEvent(nsnull, nsnull);
>    nsresult rv = event->InitEvent(aEventName, false, false);
>    NS_ENSURE_SUCCESS(rv, rv);
>  
> +  rv = event->SetTrusted(false);

here

Comment 5

[Veeraya Pupatwibul]

Attachment: PR_TRUE is now 'true'

Hope it's correct this time:)

Comment 6

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 611457 [details] [diff] [review]
PR_TRUE is now 'true'

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

Hrm, I'm not sure the changes in security/manager/ssl/src/ and testing/mochitest/ssltunnel/ssltunnel.cpp are good… Could you leave those as they are?

Looks good otherwise.

::: toolkit/xre/nsAppRunner.cpp
@@ +2633,5 @@
>   *
>   * By defining the symbol here, we can avoid the wasted lookup and hopefully
>   * improve startup performance.
>   */
> +NS_VISIBILITY_DEFAULT bool nspr_use_zone_allocator = false;

I think you need to leave this one alone.

Comment 7

[Veeraya Pupatwibul]

Attachment: removed changes for security/manager/ssl/src/, testing/mochitest/ssltunnel/ssltunnel.cpp and toolkit/xre/nsAppRunner.cpp

Comment 8

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 611476 [details] [diff] [review]
removed changes for security/manager/ssl/src/, testing/mochitest/ssltunnel/ssltunnel.cpp and toolkit/xre/nsAppRunner.cpp

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

Looks good, thanks!

Comment 9

[Mozilla RelEng Bot]

Autoland Patchset:
	Patches: 611476
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/pushloghtml?changeset=1b39bb4588fc
Try run started, revision 1b39bb4588fc. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=1b39bb4588fc

Comment 10

[Mozilla RelEng Bot]

Try run for 1b39bb4588fc is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=1b39bb4588fc
Results (out of 15 total builds):
    success: 11
    failure: 4
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-1b39bb4588fc

Comment 11

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Looks like I left out the headers in my links... Could you also change:

MayHaveDOMMutationObservers() in content/base/public/nsIDocument.h
ShouldSkipSpellCheck() in editor/libeditor/base/nsEditor.h
IsBasicLatinLetterOrNumeral() in widget/gtk2/nsGtkKeyUtils.h
mStreamInitialized, mStreamEnded, and mStreamInitialized again in netwerk/cache/nsCacheEntryDescriptor.h

Sorry about that.

Comment 12

[Veeraya Pupatwibul]

Attachment: lastest patch with changes in header files

Comment 13

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

Comment on attachment 611879 [details] [diff] [review]
lastest patch with changes in header files

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

Looks good, thanks!

Comment 14

[Mozilla RelEng Bot]

Autoland Patchset:
	Patches: 611879
	Branch: mozilla-central => try
	Destination: http://hg.mozilla.org/try/pushloghtml?changeset=d1fe05316552
Try run started, revision d1fe05316552. To cancel or monitor the job, see: https://tbpl.mozilla.org/?tree=Try&rev=d1fe05316552

Comment 15

[Mozilla RelEng Bot]

Try run for d1fe05316552 is complete.
Detailed breakdown of the results available here:
    https://tbpl.mozilla.org/?tree=Try&rev=d1fe05316552
Results (out of 15 total builds):
    success: 15
Builds (or logs if builds failed) available at:
http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-d1fe05316552

Comment 16

[Jacek Caban]

https://hg.mozilla.org/integration/mozilla-inbound/rev/523d5c5c53eb

Comment 17

[(no longer active)]

https://hg.mozilla.org/mozilla-central/rev/523d5c5c53eb