Bug 1711663 Comment 8 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Some arguments, member variables and method names were not renamed (on purpose), for example:
aFileInfo wasn't renamed to aDatabaseFileInfo or aTestFileInfo
mFileInfo wasn't renamed to mDatabaseFileInfo or mTestFileInfo
FileInfoPtr wasn't renamed to DatabaseFileInfoPtr or TestFileInfoPtr

Here's an explanation for that:
We know that there are no other `FileInfo` and `FileManager` based classes in ActorsParent.cpp, only `DatabaseFileInfo` and `DatabaseFileManager`, so we don't need to prefix method arguments, member variables and method names with `Database`. There are other similar types for which we use similar logic for naming like [here](https://searchfox.org/mozilla-central/rev/2f1a015b004b79f1145c81cdf86b15481a5630e2/dom/indexedDB/ActorsParent.cpp#9418) or [here](https://searchfox.org/mozilla-central/rev/2f1a015b004b79f1145c81cdf86b15481a5630e2/dom/indexedDB/ActorsParent.cpp#2195)
Method arguments, member variables and method names were not renamed (on purpose), for example:
aFileInfo wasn't renamed to aDatabaseFileInfo or aTestFileInfo
mFileInfo wasn't renamed to mDatabaseFileInfo or mTestFileInfo
FileInfoPtr wasn't renamed to DatabaseFileInfoPtr or TestFileInfoPtr

Here's an explanation for that:
We know that there are no other `FileInfo` and `FileManager` based classes in ActorsParent.cpp, only `DatabaseFileInfo` and `DatabaseFileManager`, so we don't need to prefix method arguments, member variables and method names with `Database`. There are other similar types for which we use similar logic for naming like [here](https://searchfox.org/mozilla-central/rev/2f1a015b004b79f1145c81cdf86b15481a5630e2/dom/indexedDB/ActorsParent.cpp#9418) or [here](https://searchfox.org/mozilla-central/rev/2f1a015b004b79f1145c81cdf86b15481a5630e2/dom/indexedDB/ActorsParent.cpp#2195)
Method arguments, member variables and method names were not renamed (on purpose), for example:
aFileInfo wasn't renamed to aDatabaseFileInfo or aSimpleFileInfo
mFileInfo wasn't renamed to mDatabaseFileInfo or mSimpleFileInfo
FileInfoPtr wasn't renamed to DatabaseFileInfoPtr or SimpleFileInfoPtr

Here's an explanation for that:
We know that there are no other `FileInfo` and `FileManager` based classes in ActorsParent.cpp, only `DatabaseFileInfo` and `DatabaseFileManager`, so we don't need to prefix method arguments, member variables and method names with `Database`. There are other similar types for which we use similar logic for naming like [here](https://searchfox.org/mozilla-central/rev/2f1a015b004b79f1145c81cdf86b15481a5630e2/dom/indexedDB/ActorsParent.cpp#9418) or [here](https://searchfox.org/mozilla-central/rev/2f1a015b004b79f1145c81cdf86b15481a5630e2/dom/indexedDB/ActorsParent.cpp#2195)

Back to Bug 1711663 Comment 8