Closed Bug 1192156 Opened 4 years ago Closed 4 years ago

IME related structs, classes and enums defined in nsIWidget.h should be separated to different header

Categories

(Core :: Widget, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla42
Tracking Status
firefox42 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

Details

(Keywords: inputmethod)

Attachments

(1 file)

Now, the head of nsIWidget.h has a lot of structs, classes and enum definition and implementation. That makes the nsIWidget definition lower. So, this is not good thing. We should move them into another header file. I think that it should be widget/IMEData.h and exported as mozilla/(widget/)IMEData.h.
The new header file steals about 700 lines from nsIWidget!

I wonder, we can move remaining two classes too. Then, we can make the new header name as mozilla/WidgetData.h or something. If you like this approach better, let me know. But I'm not sure if it's necessary since except IME specific, there are not so many lines (if we don't include the macros which define some constant values).
Attachment #8645270 - Flags: review?(roc)
Attachment #8645270 - Flags: review?(bugs)
Comment on attachment 8645270 [details] [diff] [review]
Separate IME specific structs, classes and enums from nsIWidget.h to IMEData.h

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

I don't think you need Olli for this.
Attachment #8645270 - Flags: review?(roc)
Attachment #8645270 - Flags: review?(bugs)
Attachment #8645270 - Flags: review+
Looking good anyhow
url:        https://hg.mozilla.org/integration/mozilla-inbound/rev/dbeadce8cb5fe539691e5ac88e7aec3e31df9102
changeset:  dbeadce8cb5fe539691e5ac88e7aec3e31df9102
user:       Masayuki Nakano <masayuki@d-toybox.com>
date:       Sun Aug 09 00:47:19 2015 +0900
description:
Bug 1192156 Separate IME specific structs, classes and enums from nsIWidget.h to IMEData.h r=roc
https://hg.mozilla.org/mozilla-central/rev/dbeadce8cb5f
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla42
You need to log in before you can comment on or make changes to this bug.