Use DebugOnly/NS_WARN_IF_FALSE in dom/src/storage/nsDOMStorage.cpp

RESOLVED FIXED in mozilla10

Status

()

Core
DOM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: espindola, Assigned: espindola)

Tracking

unspecified
mozilla10
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
Created attachment 570287 [details] [diff] [review]
Use DebugOnly/NS_WARN_IF_FALSE in dom/src/storage/nsDOMStorage.cpp

https://tbpl.mozilla.org/?tree=Try&rev=c55eaf1b8015
Assignee: nobody → respindola
Status: NEW → ASSIGNED
Attachment #570287 - Flags: review?(mak77)
Attachment #570287 - Flags: feedback?(honzab.moz)
Created attachment 570312 [details] [diff] [review]
Use DebugOnly/NS_WARN_IF_FALSE in dom/src/storage/nsDOMStorage.cpp

https://tbpl.mozilla.org/?tree=Try&rev=4e728fd19be8
Attachment #570287 - Attachment is obsolete: true
Attachment #570287 - Flags: review?(mak77)
Attachment #570287 - Flags: feedback?(honzab.moz)
Attachment #570312 - Flags: review?(mak77)
Attachment #570312 - Flags: feedback?(honzab.moz)
Comment on attachment 570312 [details] [diff] [review]
Use DebugOnly/NS_WARN_IF_FALSE in dom/src/storage/nsDOMStorage.cpp

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

It looks fine to me, does it compile? I don't know if some of the headers in this file already includes mozilla/Util.h
Attachment #570312 - Flags: review?(mak77) → review+
Comment on attachment 570312 [details] [diff] [review]
Use DebugOnly/NS_WARN_IF_FALSE in dom/src/storage/nsDOMStorage.cpp

https://hg.mozilla.org/integration/mozilla-inbound/rev/a92ec8fb2ab8
Attachment #570312 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/a92ec8fb2ab8
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10

Updated

6 years ago
OS: Linux → All
Hardware: x86_64 → All
Attachment #570312 - Flags: feedback?(honzab.moz) → feedback+
You need to log in before you can comment on or make changes to this bug.