mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Bug 1217307 - Remove some unnecessary null checks in dom/base/. r=njn

This commit is contained in:
Birunthan Mohanathas 2015-11-19 09:13:49 +02:00
Родитель 65859175cd
Коммит b568910a25
19 изменённых файлов: 2 добавлений и 50 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

1
dom/base/FragmentOrElement.cpp
Просмотреть файл

@ -509,7 +509,6 @@ nsNodeSupportsWeakRefTearoff::GetWeakReference(nsIWeakReference** aInstancePtr)
nsINode::nsSlots* slots = mNode->Slots();
if (!slots->mWeakReference) {
slots->mWeakReference = new nsNodeWeakReference(mNode);
NS_ENSURE_TRUE(slots->mWeakReference, NS_ERROR_OUT_OF_MEMORY);
}
NS_ADDREF(*aInstancePtr = slots->mWeakReference);

11
dom/base/nsAttrValue.cpp
Просмотреть файл

@ -167,10 +167,7 @@ nsresult
nsAttrValue::Init()
{
NS_ASSERTION(!sEnumTableArray, "nsAttrValue already initialized");
sEnumTableArray = new nsTArray<const EnumTable*>;
NS_ENSURE_TRUE(sEnumTableArray, NS_ERROR_OUT_OF_MEMORY);
return NS_OK;
}
@ -1850,14 +1847,8 @@ nsAttrValue::EnsureEmptyAtomArray()
return true;
}
AtomArray* array = new AtomArray;
if (!array) {
Reset();
return false;
}
MiscContainer* cont = EnsureEmptyMiscContainer();
cont->mValue.mAtomArray = array;
cont->mValue.mAtomArray = new AtomArray;
cont->mType = eAtomArray;
return true;

1
dom/base/nsCCUncollectableMarker.cpp
Просмотреть файл

@ -56,7 +56,6 @@ nsCCUncollectableMarker::Init()
}
nsCOMPtr<nsIObserver> marker = new nsCCUncollectableMarker;
NS_ENSURE_TRUE(marker, NS_ERROR_OUT_OF_MEMORY);
nsCOMPtr<nsIObserverService> obs =
mozilla::services::GetObserverService();

1
dom/base/nsContentPermissionHelper.cpp
Просмотреть файл

@ -151,7 +151,6 @@ bool
ContentPermissionRequestParent::Recvprompt()
{
mProxy = new nsContentPermissionRequestProxy();
NS_ASSERTION(mProxy, "Alloc of request proxy failed");
if (NS_FAILED(mProxy->Init(mRequests, this))) {
mProxy->Cancel();
}

2
dom/base/nsContentPolicy.cpp
Просмотреть файл

@ -36,8 +36,6 @@ nsresult
NS_NewContentPolicy(nsIContentPolicy **aResult)
{
*aResult = new nsContentPolicy;
if (!*aResult)
return NS_ERROR_OUT_OF_MEMORY;
NS_ADDREF(*aResult);
return NS_OK;
}

1
dom/base/nsDOMClassInfo.cpp
Просмотреть файл

@ -1411,7 +1411,6 @@ nsDOMConstructor::Create(const char16_t* aName,
IsConstructable(aData);
*aResult = new nsDOMConstructor(aName, constructable, currentInner);
NS_ENSURE_TRUE(*aResult, NS_ERROR_OUT_OF_MEMORY);
NS_ADDREF(*aResult);
return NS_OK;
}

1
dom/base/nsDOMWindowUtils.cpp
Просмотреть файл

@ -1898,7 +1898,6 @@ nsDOMWindowUtils::SendQueryContentEvent(uint32_t aType,
NS_ENSURE_SUCCESS(rv, rv);
nsQueryContentEventResult* result = new nsQueryContentEventResult();
NS_ENSURE_TRUE(result, NS_ERROR_OUT_OF_MEMORY);
result->SetEventResult(widget, queryEvent);
NS_ADDREF(*aResult = result);
return NS_OK;

7
dom/base/nsDocument.cpp
Просмотреть файл

@ -314,8 +314,6 @@ nsIdentifierMapEntry::AddContentChangeCallback(nsIDocument::IDTargetObserver aCa
{
if (!mChangeCallbacks) {
mChangeCallbacks = new nsTHashtable<ChangeCallbackEntry>;
if (!mChangeCallbacks)
return;
}
ChangeCallback cc = { aCallback, aData, aForImage };
@ -1247,9 +1245,6 @@ IMPL_SHIM(nsIApplicationCacheContainer)
return NS_NOINTERFACE; \
} \
nsCOMPtr<_i> shim = new _i##Shim(this, real); \
if (!shim) { \
return NS_ERROR_OUT_OF_MEMORY; \
} \
shim.forget(aSink); \
return NS_OK; \
} \
@ -10034,8 +10029,6 @@ nsIDocument::RegisterActivityObserver(nsISupports* aSupports)
{
if (!mActivityObservers) {
mActivityObservers = new nsTHashtable<nsPtrHashKey<nsISupports> >();
if (!mActivityObservers)
return;
}
mActivityObservers->PutEntry(aSupports);
}

4
dom/base/nsDocumentEncoder.cpp
Просмотреть файл

@ -1307,8 +1307,6 @@ nsresult
NS_NewTextEncoder(nsIDocumentEncoder** aResult)
{
*aResult = new nsDocumentEncoder;
if (!*aResult)
return NS_ERROR_OUT_OF_MEMORY;
NS_ADDREF(*aResult);
return NS_OK;
}
@ -2054,8 +2052,6 @@ nsresult
NS_NewHTMLCopyTextEncoder(nsIDocumentEncoder** aResult)
{
*aResult = new nsHTMLCopyEncoder;
if (!*aResult)
return NS_ERROR_OUT_OF_MEMORY;
NS_ADDREF(*aResult);
return NS_OK;
}

1
dom/base/nsFocusManager.cpp
Просмотреть файл

@ -190,7 +190,6 @@ nsresult
nsFocusManager::Init()
{
nsFocusManager* fm = new nsFocusManager();
NS_ENSURE_TRUE(fm, NS_ERROR_OUT_OF_MEMORY);
NS_ADDREF(fm);
sInstance = fm;

2
dom/base/nsGenConImageContent.cpp
Просмотреть файл

@ -76,8 +76,6 @@ NS_NewGenConImageContent(nsIContent** aResult, already_AddRefed<mozilla::dom::No
{
NS_PRECONDITION(aImageRequest, "Must have request!");
nsGenConImageContent *it = new nsGenConImageContent(aNodeInfo);
if (!it)
return NS_ERROR_OUT_OF_MEMORY;
NS_ADDREF(*aResult = it);
nsresult rv = it->Init(aImageRequest);
if (NS_FAILED(rv))

1
dom/base/nsGenericDOMDataNode.cpp
Просмотреть файл

@ -343,7 +343,6 @@ nsGenericDOMDataNode::SetTextInternal(uint32_t aOffset, uint32_t aCount,
// Allocate new buffer
int32_t newLength = textLength - aCount + aLength;
char16_t* to = new char16_t[newLength];
NS_ENSURE_TRUE(to, NS_ERROR_OUT_OF_MEMORY);
// Copy over appropriate data
if (aOffset) {

2
dom/base/nsGlobalWindowCommands.cpp
Просмотреть файл

@ -988,7 +988,6 @@ nsClipboardDragDropHookCommand::GetCommandStateParams(const char *aCommandName,
#define NS_REGISTER_ONE_COMMAND(_cmdClass, _cmdName) \
{ \
_cmdClass* theCmd = new _cmdClass(); \
if (!theCmd) return NS_ERROR_OUT_OF_MEMORY; \
rv = inCommandTable->RegisterCommand(_cmdName, \
static_cast<nsIControllerCommand *>(theCmd)); \
}
@ -996,7 +995,6 @@ nsClipboardDragDropHookCommand::GetCommandStateParams(const char *aCommandName,
#define NS_REGISTER_FIRST_COMMAND(_cmdClass, _cmdName) \
{ \
_cmdClass* theCmd = new _cmdClass(); \
if (!theCmd) return NS_ERROR_OUT_OF_MEMORY; \
rv = inCommandTable->RegisterCommand(_cmdName, \
static_cast<nsIControllerCommand *>(theCmd));

4
dom/base/nsImageLoadingContent.cpp
Просмотреть файл

@ -385,10 +385,6 @@ nsImageLoadingContent::AddObserver(imgINotificationObserver* aObserver)
}
observer->mNext = new ImageObserver(aObserver);
if (! observer->mNext) {
return NS_ERROR_OUT_OF_MEMORY;
}
ReplayImageStatus(mCurrentRequest, aObserver);
ReplayImageStatus(mPendingRequest, aObserver);

2
dom/base/nsNameSpaceManager.cpp
Просмотреть файл

@ -189,7 +189,7 @@ nsresult nsNameSpaceManager::AddNameSpace(const nsAString& aURI,
"BAD! AddNameSpace not called in right order!");
nsString* uri = new nsString(aURI);
if (!uri || !mURIArray.AppendElement(uri)) {
if (!mURIArray.AppendElement(uri)) {
delete uri;
return NS_ERROR_OUT_OF_MEMORY;
}

3
dom/base/nsObjectLoadingContent.cpp
Просмотреть файл

@ -2936,9 +2936,6 @@ nsObjectLoadingContent::AsyncStartPluginInstance()
}
nsCOMPtr<nsIRunnable> event = new nsAsyncInstantiateEvent(this);
if (!event) {
return NS_ERROR_OUT_OF_MEMORY;
}
nsresult rv = NS_DispatchToCurrentThread(event);
if (NS_SUCCEEDED(rv)) {
// Track pending events

2
dom/base/nsScriptNameSpaceManager.cpp
Просмотреть файл

@ -480,8 +480,6 @@ nsScriptNameSpaceManager::RegisterDOMCIData(const char *aName,
"Someone tries to register classinfo data for a class that isn't new or external!");
s->mData = new nsExternalDOMClassInfoData;
NS_ENSURE_TRUE(s->mData, NS_ERROR_OUT_OF_MEMORY);
s->mType = nsGlobalNameStruct::eTypeExternalClassInfo;
s->mData->mName = aName;
s->mData->mNameUTF16 = className;

2
dom/base/nsTextFragment.cpp
Просмотреть файл

@ -42,8 +42,6 @@ nsTextFragment::Init()
for (i = 0; i <= TEXTFRAG_MAX_NEWLINES; ++i) {
sSpaceSharedString[i] = new char[1 + i + TEXTFRAG_WHITE_AFTER_NEWLINE];
sTabSharedString[i] = new char[1 + i + TEXTFRAG_WHITE_AFTER_NEWLINE];
NS_ENSURE_TRUE(sSpaceSharedString[i] && sTabSharedString[i],
NS_ERROR_OUT_OF_MEMORY);
sSpaceSharedString[i][0] = ' ';
sTabSharedString[i][0] = ' ';
uint32_t j;

4
dom/base/nsTextNode.cpp
Просмотреть файл

@ -214,10 +214,6 @@ NS_NewAttributeContent(nsNodeInfoManager *aNodeInfoManager,
nsAttributeTextNode* textNode = new nsAttributeTextNode(ni,
aNameSpaceID,
aAttrName);
if (!textNode) {
return NS_ERROR_OUT_OF_MEMORY;
}
NS_ADDREF(*aResult = textNode);
return NS_OK;