The default bug view has changed. See this FAQ.

[BeOS] nsRenderingContextBeOS cleanup

RESOLVED FIXED

Status

Core Graveyard
GFX: BeOS
RESOLVED FIXED
12 years ago
8 years ago

People

(Reporter: tqh, Assigned: tqh)

Tracking

({fixed1.8})

Trunk
Other
BeOS
fixed1.8

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

12 years ago
The current version has a poor init and the codestyle is quite weird in some places.
(Assignee)

Comment 1

12 years ago
Created attachment 197447 [details] [diff] [review]
Cleanup, and rewritten init

Hopefully this patch is already good, it was quite a big file so I may have
missed something.
Assignee: beos → thesuckiestemail
Status: NEW → ASSIGNED
(Assignee)

Comment 2

12 years ago
Comment on attachment 197447 [details] [diff] [review]
Cleanup, and rewritten init

r?
Attachment #197447 - Flags: review?(sergei_d)
(Assignee)

Updated

12 years ago
Blocks: 296856
(Assignee)

Comment 3

12 years ago
Created attachment 197464 [details] [diff] [review]
Updated according to comments

Better checks in init, removed comments and changed order on mClipRegion
nullcheck (although not important).
Attachment #197447 - Attachment is obsolete: true
(Assignee)

Comment 4

12 years ago
Created attachment 197469 [details] [diff] [review]
Second init improved
(Assignee)

Updated

12 years ago
Attachment #197464 - Attachment is obsolete: true
Attachment #197469 - Flags: review?(sergei_d)

Comment 5

12 years ago
Comment on attachment 197469 [details] [diff] [review]
Second init improved

r=sergei_d
Attachment #197469 - Flags: review?(sergei_d) → review+

Comment 6

12 years ago
Patch landed:
nsRenderingContextBeOS.cpp
new revision: 1.53; previous revision: 1.52
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 7

12 years ago
Comment on attachment 197469 [details] [diff] [review]
Second init improved

BeOS-only change. Tested, will not affect any other platform in any way.
Requesting approval for MOZILLA_1_8_BRANCH
Attachment #197469 - Flags: approval1.8b5?

Updated

12 years ago
Attachment #197469 - Flags: approval1.8b5? → approval1.8b5+

Updated

12 years ago
Keywords: fixed1.8
(Assignee)

Comment 8

10 years ago
Comment on attachment 197447 [details] [diff] [review]
Cleanup, and rewritten init

removing old review req.
Attachment #197447 - Flags: review?(sergei_d)
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.