mirror of
https://github.com/servo/servo.git
synced 2025-08-03 04:30:10 +01:00
Auto merge of #17640 - emilio:starts-with-ignore-ascii-case, r=stshine
style: Fix starts_with_ignore_ascii_case. In particular, fix a panic when the input is not ASCII and we happen to index in something that is not a char boundary. This fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1379380 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/17640) <!-- Reviewable:end -->
This commit is contained in:
commit
104e0b4730
2 changed files with 14 additions and 3 deletions
|
@ -148,6 +148,6 @@ pub fn str_join<I, T>(strs: I, join: &str) -> String
|
|||
|
||||
/// Returns true if a given string has a given prefix with case-insensitive match.
|
||||
pub fn starts_with_ignore_ascii_case(string: &str, prefix: &str) -> bool {
|
||||
string.len() > prefix.len() &&
|
||||
string[0..prefix.len()].eq_ignore_ascii_case(prefix)
|
||||
string.len() >= prefix.len() &&
|
||||
string.as_bytes()[0..prefix.len()].eq_ignore_ascii_case(prefix.as_bytes())
|
||||
}
|
||||
|
|
|
@ -2,7 +2,7 @@
|
|||
* License, v. 2.0. If a copy of the MPL was not distributed with this
|
||||
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
|
||||
|
||||
use style::str::{split_html_space_chars, str_join};
|
||||
use style::str::{split_html_space_chars, str_join, starts_with_ignore_ascii_case};
|
||||
|
||||
#[test]
|
||||
pub fn split_html_space_chars_whitespace() {
|
||||
|
@ -33,3 +33,14 @@ pub fn test_str_join_many() {
|
|||
let expected = "-alpha--beta-gamma-";
|
||||
assert_eq!(actual, expected);
|
||||
}
|
||||
|
||||
#[test]
|
||||
pub fn test_starts_with_ignore_ascii_case_basic() {
|
||||
assert!(starts_with_ignore_ascii_case("-webkit-", "-webkit-"));
|
||||
assert!(starts_with_ignore_ascii_case("-webkit-foo", "-webkit-"));
|
||||
}
|
||||
|
||||
#[test]
|
||||
pub fn test_starts_with_ignore_ascii_case_char_boundary() {
|
||||
assert!(!starts_with_ignore_ascii_case("aaaaa💩", "-webkit-"));
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue