mirror of
https://github.com/servo/servo.git
synced 2025-06-06 16:45:39 +00:00
resources: Remove baked in resources (#36042)
This allows removing a LazyLock around the resources. We override the baked in resources unconditionally in servoshell upon initialization anyway ([desktop](9f93ccd942/ports/servoshell/desktop/cli.rs (L15)
), [android](9f93ccd942/ports/servoshell/egl/android/simpleservo.rs (L49)
), [ohos](9f93ccd942/ports/servoshell/egl/ohos/simpleservo.rs (L43)
) ), meaning that the baked in resources are unused in servoshell. For 3rd-party embedders, we probably also want to the let them know early that they should initialize the resources, instead of restricting the panics to production mode. Rippy is the only resource which was required. Since it is only 253 bytes large, we just bake that resource in as a fallback. We do want to make using the resources easy from tests, so we add some logic to keep the baked in resources for tests only and initialize the resource reader on first access. --- <!-- 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 - [ ] These changes fix #___ (GitHub issue number if applicable) - [ ] There are tests for these changes OR - [ ] These changes do not require tests because ___ Signed-off-by: Jonathan Schwender <schwenderjonathan@gmail.com>
This commit is contained in:
parent
30390f8c5e
commit
d7e560c9c0
4 changed files with 53 additions and 28 deletions
|
@ -31,6 +31,11 @@ use webrender_api::{ImageDescriptor, ImageDescriptorFlags, ImageFormat};
|
|||
|
||||
use crate::resource_thread::CoreResourceThreadPool;
|
||||
|
||||
// We bake in rippy.png as a fallback, in case the embedder does not provide
|
||||
// a rippy resource. this version is 253 bytes large, don't exchange it against
|
||||
// something in higher resolution.
|
||||
const FALLBACK_RIPPY: &[u8] = include_bytes!("../../resources/rippy.png");
|
||||
|
||||
//
|
||||
// TODO(gw): Remaining work on image cache:
|
||||
// * Make use of the prefetch support in various parts of the code.
|
||||
|
@ -51,7 +56,9 @@ fn decode_bytes_sync(key: LoadKey, bytes: &[u8], cors: CorsStatus) -> DecoderMsg
|
|||
}
|
||||
|
||||
fn get_placeholder_image(compositor_api: &CrossProcessCompositorApi, data: &[u8]) -> Arc<Image> {
|
||||
let mut image = load_from_memory(data, CorsStatus::Unsafe).unwrap();
|
||||
let mut image = load_from_memory(data, CorsStatus::Unsafe)
|
||||
.or_else(|| load_from_memory(FALLBACK_RIPPY, CorsStatus::Unsafe))
|
||||
.expect("load fallback image failed");
|
||||
set_webrender_image_key(compositor_api, &mut image);
|
||||
Arc::new(image)
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue