-
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Blob::{new, new_inherited} to take Strings #11768
Conversation
Heads up! This PR modifies the following files:
|
@@ -156,7 +156,7 @@ impl Blob { | |||
}; | |||
|
|||
let slice = DataSlice::from_bytes(bytes); | |||
Ok(Blob::new(global, BlobImpl::new_from_slice(slice), &blobPropertyBag.get_typestring())) | |||
Ok(Blob::new(global, BlobImpl::new_from_slice(slice), blobPropertyBag.get_typestring().to_owned())) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should be able to remove this to_owned call
Nice job, just two small comments. |
Neato, updating. |
Thank you! @bors-servo r+ |
📌 Commit af325a9 has been approved by |
Update Blob::{new, new_inherited} to take Strings <!-- Please describe your changes on the following line: --> --- <!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: --> - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy` does not report any errors - [X] These changes fix #11762. <!-- Either: --> - [ ] There are tests for these changes OR - [X] These changes do not require tests because no logic changes, only interface changes. <!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. --> <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11768) <!-- Reviewable:end -->
💔 Test failed - linux-rel |
|
@bors-servo retry #11631 |
⚡ Previous build results for android, arm32, arm64, linux-dev, mac-dev-unit, mac-rel-wpt, windows are reusable. Rebuilding only linux-rel, mac-rel-css... |
☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows |
./mach build -d
does not report any errors./mach test-tidy
does not report any errorsThis change is