Closed
Bug 578262
Opened 14 years ago
Closed 14 years ago
Clean up style and remove build warnings from dom/ipc
Categories
(Core :: IPC, defect)
Core
IPC
Tracking
()
RESOLVED
FIXED
People
(Reporter: cjones, Assigned: cjones)
References
Details
Attachments
(1 file, 1 obsolete file)
139.43 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
I'd like to make the following changes to dom/ipc - remove build warnings - rename initialLower() message names to InitialUpper() - s/PContentProcess/PContent/ - s/PIFrameEmbedding/PBrowser/
Comment 1•14 years ago
|
||
Please note that VC10 exposes some build warnings that don't appear in earlier versions.
Assignee | ||
Comment 2•14 years ago
|
||
In addition to what's listed in comment 0, I also - s/ContentProcessProcess/ContentProcess - s/nsFrameLoader::mChildProcess/nsFrameLoader::mRemoteBrowser/ - changed mouse event coord types from int to float, which was apparently done to work around possible lack of float pickler at the time. Would be nice if these things were reported ... This is a large-ish patch, but all mechanical.
Assignee: nobody → jones.chris.g
Attachment #456999 -
Flags: review?(benjamin)
Assignee | ||
Comment 3•14 years ago
|
||
Good bit of rottage since v0.
Attachment #456999 -
Attachment is obsolete: true
Attachment #457689 -
Flags: review?(benjamin)
Attachment #456999 -
Flags: review?(benjamin)
Updated•14 years ago
|
Attachment #457689 -
Flags: review?(benjamin) → review+
Assignee | ||
Comment 4•14 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/62d0fbc0e2ba
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•