layout should use mozilla::Preferences

RESOLVED FIXED in mozilla7

Status

()

Core
Layout
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: masayuki, Assigned: masayuki)

Tracking

Trunk
mozilla7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Created attachment 538187 [details] [diff] [review]
Patch v1.0
Attachment #538187 - Flags: review?(roc)
Comment on attachment 538187 [details] [diff] [review]
Patch v1.0

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

::: layout/tools/layout-debug/src/nsLayoutDebuggingTools.cpp
@@ +589,2 @@
>  
> +    nsCOMPtr<nsIPrefService> prefService = Preferences::GetService();

Doesn't need to be an nsCOMPtr.
Attachment #538187 - Flags: review?(roc) → review+
Created attachment 538398 [details] [diff] [review]
Patch v1.1 (mq)
Attachment #538187 - Attachment is obsolete: true
http://hg.mozilla.org/mozilla-central/rev/e37011790a8a
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
You need to log in before you can comment on or make changes to this bug.