outparamdel remaining nsNodeInfoManager::GetNodeInfo

NEW
Unassigned

Status

()

Core
Rewriting and Analysis
9 years ago
9 years ago

People

(Reporter: (dormant account), Unassigned)

Tracking

(Blocks: 1 bug)

Trunk
x86
All
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
+++ This bug was initially created as a clone of Bug #450777 +++

patch coming soon
(Reporter)

Updated

9 years ago
Blocks: 266048
Summary: outparamdel nsNodeInfoManager::GetNodeInfo → outparamdel remaining nsNodeInfoManager::GetNodeInfo
(Reporter)

Updated

9 years ago
Depends on: 455540
(Reporter)

Comment 1

9 years ago
bz suggested that I make nsNodeInfoManager::GetNodeInfo to only fail with NS_ERROR_OUT_OF_MEMORY(instead of NS_ERROR_FAILURE), so I'll also include a followup patch to bug 450777 to change those error codes.
(Reporter)

Comment 2

9 years ago
Created attachment 338914 [details] [diff] [review]
remaining GetNodeInfo outparamdelling

bz, will get rid of the other overload.
Attachment #338914 - Flags: review?(jst)
(Reporter)

Updated

9 years ago
Blocks: 430328
No longer blocks: 266048
(Reporter)

Comment 3

9 years ago
Created attachment 339287 [details] [diff] [review]
s/NS_ERROR_FAILURE/NS_ERROR_OOM/ in previous patch

This switches error code in bug 450777 to be NS_ERROR_OUT_OF_MEMORY
Attachment #339287 - Flags: review?(jst)

Updated

9 years ago
Attachment #339287 - Flags: superreview+
Attachment #339287 - Flags: review?(jst)
Attachment #339287 - Flags: review+
(Reporter)

Comment 4

9 years ago
checked in S/FAILURE/OOM/ in 514ea093a63f

Updated

9 years ago
Attachment #338914 - Flags: review?(jst)
You need to log in before you can comment on or make changes to this bug.