Skip to content

Commit

Permalink
Bug 1219392 - Capitalize mozilla::unused to avoid conflicts. r=froydnj
Browse files Browse the repository at this point in the history
  • Loading branch information
poiru committed Nov 2, 2015
1 parent d267b10 commit 9985829
Show file tree
Hide file tree
Showing 240 changed files with 907 additions and 907 deletions.
256 changes: 128 additions & 128 deletions accessible/ipc/ProxyAccessible.cpp

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion caps/DomainPolicy.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ BroadcastDomainSetChange(DomainSetType aSetType, DomainSetChangeType aChangeType
SerializeURI(aDomain, uri);

for (uint32_t i = 0; i < parents.Length(); i++) {
unused << parents[i]->SendDomainSetChanged(aSetType, aChangeType, uri);
Unused << parents[i]->SendDomainSetChanged(aSetType, aChangeType, uri);
}
return NS_OK;
}
Expand Down
2 changes: 1 addition & 1 deletion chrome/nsChromeRegistryChrome.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -749,7 +749,7 @@ SendManifestEntry(const ChromeRegistryItem &aItem)
return;

for (uint32_t i = 0; i < parents.Length(); i++) {
unused << parents[i]->SendRegisterChromeItem(aItem);
Unused << parents[i]->SendRegisterChromeItem(aItem);
}
}

Expand Down
2 changes: 1 addition & 1 deletion docshell/base/nsDocShell.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5107,7 +5107,7 @@ nsDocShell::DisplayLoadError(nsresult aError, nsIURI* aURI,
// asserts). Satisfy that assertion now since GetDoc will force
// creation of one if it hasn't already been created.
if (mScriptGlobal) {
unused << mScriptGlobal->GetDoc();
Unused << mScriptGlobal->GetDoc();
}

// Display a message box
Expand Down
10 changes: 5 additions & 5 deletions dom/asmjscache/AsmJSCache.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ using mozilla::ipc::BackgroundChild;
using mozilla::ipc::IsOnBackgroundThread;
using mozilla::ipc::PBackgroundChild;
using mozilla::ipc::PrincipalInfo;
using mozilla::unused;
using mozilla::Unused;
using mozilla::HashString;

namespace mozilla {
Expand Down Expand Up @@ -478,7 +478,7 @@ class ParentRunnable final
FinishOnOwningThread();

if (!mActorDestroyed) {
unused << Send__delete__(this, mResult);
Unused << Send__delete__(this, mResult);
}
}

Expand Down Expand Up @@ -993,7 +993,7 @@ ParentRunnable::Run()

// Metadata is now open.
if (!SendOnOpenMetadataForRead(mMetadata)) {
unused << Send__delete__(this, JS::AsmJSCache_InternalError);
Unused << Send__delete__(this, JS::AsmJSCache_InternalError);
}

return NS_OK;
Expand Down Expand Up @@ -1035,7 +1035,7 @@ ParentRunnable::Run()
FileDescriptor::PlatformHandleType handle =
FileDescriptor::PlatformHandleType(PR_FileDesc2NativeHandle(mFileDesc));
if (!SendOnOpenCacheFile(mFileSize, FileDescriptor(handle))) {
unused << Send__delete__(this, JS::AsmJSCache_InternalError);
Unused << Send__delete__(this, JS::AsmJSCache_InternalError);
}

return NS_OK;
Expand Down Expand Up @@ -1451,7 +1451,7 @@ ChildRunnable::Run()
Release();

if (!mActorDestroyed) {
unused << Send__delete__(this, JS::AsmJSCache_Success);
Unused << Send__delete__(this, JS::AsmJSCache_Success);
}

mState = eFinished;
Expand Down
2 changes: 1 addition & 1 deletion dom/base/ImageEncoder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ class EncodingRunnable : public nsRunnable
rv = NS_DispatchToMainThread(mEncodingCompleteEvent);
if (NS_FAILED(rv)) {
// Better to leak than to crash.
unused << mEncodingCompleteEvent.forget();
Unused << mEncodingCompleteEvent.forget();
return rv;
}

Expand Down
12 changes: 6 additions & 6 deletions dom/base/nsContentPermissionHelper.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
#include "nsIDOMEvent.h"
#include "nsWeakPtr.h"

using mozilla::unused; // <snicker>
using mozilla::Unused; // <snicker>
using namespace mozilla::dom;
using namespace mozilla;

Expand Down Expand Up @@ -473,7 +473,7 @@ nsContentPermissionRequestProxy::nsContentPermissionRequesterProxy

mGetCallback = aCallback;
mWaitGettingResult = true;
unused << mParent->SendGetVisibility();
Unused << mParent->SendGetVisibility();
return NS_OK;
}

Expand Down Expand Up @@ -608,7 +608,7 @@ nsContentPermissionRequestProxy::Cancel()

nsTArray<PermissionChoice> emptyChoices;

unused << ContentPermissionRequestParent::Send__delete__(mParent, false, emptyChoices);
Unused << ContentPermissionRequestParent::Send__delete__(mParent, false, emptyChoices);
mParent = nullptr;
return NS_OK;
}
Expand Down Expand Up @@ -672,7 +672,7 @@ nsContentPermissionRequestProxy::Allow(JS::HandleValue aChoices)
return NS_ERROR_FAILURE;
}

unused << ContentPermissionRequestParent::Send__delete__(mParent, true, choices);
Unused << ContentPermissionRequestParent::Send__delete__(mParent, true, choices);
mParent = nullptr;
return NS_OK;
}
Expand Down Expand Up @@ -776,7 +776,7 @@ RemotePermissionRequest::RecvGetVisibility()

bool isActive = false;
docshell->GetIsActive(&isActive);
unused << SendNotifyVisibility(isActive);
Unused << SendNotifyVisibility(isActive);
return true;
}

Expand All @@ -787,6 +787,6 @@ RemotePermissionRequest::NotifyVisibility(bool isVisible)
return NS_OK;
}

unused << SendNotifyVisibility(isVisible);
Unused << SendNotifyVisibility(isVisible);
return NS_OK;
}
2 changes: 1 addition & 1 deletion dom/base/nsFocusManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1123,7 +1123,7 @@ void
ActivateOrDeactivateChild(TabParent* aParent, void* aArg)
{
bool active = static_cast<bool>(aArg);
unused << aParent->SendParentActivated(active);
Unused << aParent->SendParentActivated(active);
}

void
Expand Down
12 changes: 6 additions & 6 deletions dom/base/nsFrameLoader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -990,8 +990,8 @@ nsFrameLoader::SwapWithOtherRemoteLoader(nsFrameLoader* aOther,

mInSwap = aOther->mInSwap = false;

unused << mRemoteBrowser->SendSwappedWithOtherRemoteLoader();
unused << aOther->mRemoteBrowser->SendSwappedWithOtherRemoteLoader();
Unused << mRemoteBrowser->SendSwappedWithOtherRemoteLoader();
Unused << aOther->mRemoteBrowser->SendSwappedWithOtherRemoteLoader();
return NS_OK;
}

Expand Down Expand Up @@ -2296,7 +2296,7 @@ nsFrameLoader::TryRemoteBrowser()
nsGkAtoms::mozpasspointerevents,
nsGkAtoms::_true,
eCaseMatters)) {
unused << mRemoteBrowser->SendSetUpdateHitRegion(true);
Unused << mRemoteBrowser->SendSetUpdateHitRegion(true);
}

ReallyLoadFrameScripts();
Expand Down Expand Up @@ -2341,14 +2341,14 @@ nsFrameLoader::DeactivateRemoteFrame() {
void
nsFrameLoader::ActivateUpdateHitRegion() {
if (mRemoteBrowser) {
unused << mRemoteBrowser->SendSetUpdateHitRegion(true);
Unused << mRemoteBrowser->SendSetUpdateHitRegion(true);
}
}

void
nsFrameLoader::DeactivateUpdateHitRegion() {
if (mRemoteBrowser) {
unused << mRemoteBrowser->SendSetUpdateHitRegion(false);
Unused << mRemoteBrowser->SendSetUpdateHitRegion(false);
}
}

Expand Down Expand Up @@ -2857,7 +2857,7 @@ nsFrameLoader::RequestNotifyAfterRemotePaint()
{
// If remote browsing (e10s), handle this with the TabParent.
if (mRemoteBrowser) {
unused << mRemoteBrowser->SendRequestNotifyAfterRemotePaint();
Unused << mRemoteBrowser->SendRequestNotifyAfterRemotePaint();
}

return NS_OK;
Expand Down
4 changes: 2 additions & 2 deletions dom/base/nsGlobalWindow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -11623,7 +11623,7 @@ nsGlobalWindow::SetTimeoutOrInterval(nsIScriptTimeoutHandler *aHandler,
}

// The timeout is now also held in the timer's closure.
unused << copy.forget();
Unused << copy.forget();
} else {
// If we are frozen, however, then we instead simply set
// timeout->mTimeRemaining to be the "time remaining" in the timeout (i.e.,
Expand Down Expand Up @@ -12029,7 +12029,7 @@ nsGlobalWindow::RunTimeout(nsTimeout *aTimeout)
// through a timeout that fired while a modal (to this window)
// dialog was open or through other non-obvious paths.
MOZ_ASSERT(dummy_timeout->HasRefCntOne(), "dummy_timeout may leak");
unused << timeoutExtraRef.forget().take();
Unused << timeoutExtraRef.forget().take();

mTimeoutInsertionPoint = last_insertion_point;

Expand Down
6 changes: 3 additions & 3 deletions dom/base/nsScriptLoader.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -819,8 +819,8 @@ NotifyOffThreadScriptLoadCompletedRunnable::~NotifyOffThreadScriptLoadCompletedR
} else {
MOZ_ASSERT(false, "We really shouldn't leak!");
// Better to leak than crash.
unused << mRequest.forget();
unused << mLoader.forget();
Unused << mRequest.forget();
Unused << mLoader.forget();
}
}
}
Expand Down Expand Up @@ -890,7 +890,7 @@ nsScriptLoader::AttemptAsyncScriptCompile(nsScriptLoadRequest* aRequest)
mDocument->BlockOnload();
aRequest->mProgress = nsScriptLoadRequest::Progress_Compiling;

unused << runnable.forget();
Unused << runnable.forget();
return NS_OK;
}

Expand Down
4 changes: 2 additions & 2 deletions dom/bluetooth/bluedroid/BluetoothDaemonA2dpInterface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ BluetoothDaemonA2dpModule::ConnectCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -86,7 +86,7 @@ BluetoothDaemonA2dpModule::DisconnectCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand Down
20 changes: 10 additions & 10 deletions dom/bluetooth/bluedroid/BluetoothDaemonAvrcpInterface.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ BluetoothDaemonAvrcpModule::GetPlayStatusRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -94,7 +94,7 @@ BluetoothDaemonAvrcpModule::ListPlayerAppAttrRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -118,7 +118,7 @@ BluetoothDaemonAvrcpModule::ListPlayerAppValueRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -143,7 +143,7 @@ BluetoothDaemonAvrcpModule::GetPlayerAppValueRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -167,7 +167,7 @@ BluetoothDaemonAvrcpModule::GetPlayerAppAttrTextRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -191,7 +191,7 @@ BluetoothDaemonAvrcpModule::GetPlayerAppValueTextRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -215,7 +215,7 @@ BluetoothDaemonAvrcpModule::GetElementAttrRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -237,7 +237,7 @@ BluetoothDaemonAvrcpModule::SetPlayerAppValueRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand Down Expand Up @@ -266,7 +266,7 @@ BluetoothDaemonAvrcpModule::RegisterNotificationRspCmd(
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand All @@ -288,7 +288,7 @@ BluetoothDaemonAvrcpModule::SetVolumeCmd(uint8_t aVolume,
if (NS_FAILED(rv)) {
return rv;
}
unused << pdu.forget();
Unused << pdu.forget();
return NS_OK;
}

Expand Down
Loading

0 comments on commit 9985829

Please sign in to comment.