mirror of
https://github.com/mozilla/gecko-dev.git
synced 2024-12-01 08:42:13 +00:00
c15823d075
The biggest set of APIs from ns[T]StringObsolete which are still heavily used are the string searching APIs. It appears the intention was for these to be replaced by the `FindInReadable` APIs, however that doesn't appear to have happened. In addition, the APIs have some quirks around their handling of mixed character widths. These APIs generally supported both narrow strings and the native string type, probably because char16_t string literals weren't available until c++11. Finally they also used easy-to-confuse unlabeled boolean and integer optional arguments to control behaviour. These patches do the following major changes to the searching APIs: 1. The ASCII case-insensitive search method was split out as LowerCaseFindASCII, rather than using a boolean. This should be less error-prone and more explicit, and allows the method to continue to use narrow string literals for all string types (as only ASCII is supported). 2. The other [R]Find methods were restricted to only support arguments with matching character types. I considered adding a FindASCII method which would use narrow string literals for both wide and narrow strings but it would've been the same amount of work as changing all of the literals to unicode literals. This ends up being the bulk of the changes in the patch. 3. All find methods were re-implemented using std::basic_string_view's find algorithm or stl algorithms to reduce code complexity, and avoid the need to carry around the logic from nsStringObsolete.cpp. 4. The implementations were moved to nsTStringRepr.cpp. 5. An overload of Find was added to try to catch callers which previously called `Find(..., false)` or `Find(..., true)` to set case-sensitivity, due to booleans normally implicitly coercing to `index_type`. This should probably be removed at some point, but may be useful during the transition. Differential Revision: https://phabricator.services.mozilla.com/D148300 |
||
|---|---|---|
| .. | ||
| moz.build | ||
| nsIWebBrowserPersist.idl | ||
| nsIWebBrowserPersistDocument.idl | ||
| nsWebBrowserPersist.cpp | ||
| nsWebBrowserPersist.h | ||
| PWebBrowserPersistDocument.ipdl | ||
| PWebBrowserPersistResources.ipdl | ||
| PWebBrowserPersistSerialize.ipdl | ||
| WebBrowserPersistDocumentChild.cpp | ||
| WebBrowserPersistDocumentChild.h | ||
| WebBrowserPersistDocumentParent.cpp | ||
| WebBrowserPersistDocumentParent.h | ||
| WebBrowserPersistLocalDocument.cpp | ||
| WebBrowserPersistLocalDocument.h | ||
| WebBrowserPersistRemoteDocument.cpp | ||
| WebBrowserPersistRemoteDocument.h | ||
| WebBrowserPersistResourcesChild.cpp | ||
| WebBrowserPersistResourcesChild.h | ||
| WebBrowserPersistResourcesParent.cpp | ||
| WebBrowserPersistResourcesParent.h | ||
| WebBrowserPersistSerializeChild.cpp | ||
| WebBrowserPersistSerializeChild.h | ||
| WebBrowserPersistSerializeParent.cpp | ||
| WebBrowserPersistSerializeParent.h | ||