Closed Bug 1318618 Opened 3 years ago Closed 3 years ago
CString data type in gfx Prefs in OOP compositor
58 bytes, text/x-review-board-request
Currently, our GfxPrefValue in PGPU.ipdl doesn't support nsCString data type that would affect some preferences can't be accessed in the compositor process.
Assignee: nobody → dmu
Ideally, we should choose nsCString as the data type for gfxPrefs to add values to Preference. However, there are some other files will include gfxPrefs.h outside XUL, and "nsStringFwd.h" can't be included by this kind of files. Otherwise, it would have an error like "#error Internal string headers are not available from external-linkage code." Therefore, I select to use std::string for implementing this.
Comment on attachment 8813478 [details] Bug 1318618 - Support nsCString data type in gfxPrefs; https://reviewboard.mozilla.org/r/94892/#review95386
Attachment #8813478 - Flags: review?(dvander) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/a8dabadba0ab Support nsCString data type in gfxPrefs; r=dvander
You need to log in before you can comment on or make changes to this bug.