The default bug view has changed. See this FAQ.

Eliminate NS_(ceil|floor)

RESOLVED FIXED in mozilla8

Status

()

Core
Graphics
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jrmuizel, Assigned: jrmuizel)

Tracking

Trunk
mozilla8
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
I don't see any reason for having these guys around.
(Assignee)

Comment 1

6 years ago
Created attachment 536122 [details] [diff] [review]
Remove NS_(floor|ceil)
Attachment #536122 - Flags: review?(roc)
Comment on attachment 536122 [details] [diff] [review]
Remove NS_(floor|ceil)

Review of attachment 536122 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #536122 - Flags: review?(roc) → review+
(Assignee)

Comment 3

6 years ago
Created attachment 536959 [details] [diff] [review]
Remove unnecessary floor in nsNativeTheme::QueueAnimatedContentForRefresh

MSVC wasn't building the patch because we were flooring an int. This patch gets rid of that.
Attachment #536959 - Flags: review?(bent.mozilla)
Attachment #536959 - Flags: review?(bent.mozilla) → review+
(Assignee)

Comment 4

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/188621c4b912
attachment 536959 [details] [diff] [review] in mozilla-central for Firefox 7:
http://hg.mozilla.org/mozilla-central/rev/188621c4b912

Not resolving the bug because attachment 536122 [details] [diff] [review] has not yet landed.  You can resolve this bug when that patch lands (or if it's no longer needed).
Assignee: nobody → jmuizelaar
Target Milestone: --- → mozilla7
http://hg.mozilla.org/mozilla-central/rev/6fd7a1b9ef64
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED

Updated

6 years ago
OS: Mac OS X → All
Hardware: x86 → All
Target Milestone: mozilla7 → mozilla8
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.