Compare commits
10
Commits
c772e8ee46
...
main
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
359c71413d
|
||
|
|
07b558eb0a | ||
|
|
8976fba9d0 | ||
|
|
5616744c04 | ||
|
|
e4bedb6dc6 | ||
|
|
534411f36f | ||
|
|
d5feda9da5 | ||
|
|
303a3aa926 | ||
|
|
37eb0a7825 | ||
|
|
3c3d7edcd6 |
@@ -8,3 +8,7 @@
|
||||
*.log
|
||||
*.tmp
|
||||
claude-goal/
|
||||
PRD.MD
|
||||
agents.md
|
||||
/ELY Browser Design
|
||||
/docs
|
||||
Generated
+55
-55
@@ -8459,7 +8459,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"accesskit",
|
||||
"arboard",
|
||||
@@ -8523,7 +8523,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-allocator"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"libc",
|
||||
"tikv-jemalloc-sys",
|
||||
@@ -8534,7 +8534,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-background-hang-monitor"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"backtrace",
|
||||
"crossbeam-channel",
|
||||
@@ -8549,7 +8549,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-background-hang-monitor-api"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"serde",
|
||||
"servo-base",
|
||||
@@ -8558,7 +8558,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-base"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"accesskit",
|
||||
"crossbeam-channel",
|
||||
@@ -8583,7 +8583,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-canvas"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"bytemuck",
|
||||
"crossbeam-channel",
|
||||
@@ -8608,7 +8608,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-canvas-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"crossbeam-channel",
|
||||
"euclid",
|
||||
@@ -8630,7 +8630,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-config"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"num_enum",
|
||||
"serde",
|
||||
@@ -8643,7 +8643,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-config-macro"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"proc-macro2",
|
||||
"quote",
|
||||
@@ -8654,7 +8654,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-constellation"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"accesskit",
|
||||
"backtrace",
|
||||
@@ -8698,7 +8698,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-constellation-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"base64",
|
||||
"content-security-policy",
|
||||
@@ -8733,7 +8733,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-default-resources"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"servo-embedder-traits",
|
||||
]
|
||||
@@ -8741,7 +8741,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-deny-public-fields"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"proc-macro2",
|
||||
"syn 2.0.117",
|
||||
@@ -8751,7 +8751,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-devtools"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"atomic_refcell",
|
||||
"base64",
|
||||
@@ -8781,7 +8781,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-devtools-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"http 1.4.0",
|
||||
"malloc_size_of_derive",
|
||||
@@ -8798,7 +8798,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-dom-struct"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"prettyplease",
|
||||
"proc-macro2",
|
||||
@@ -8809,7 +8809,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-embedder-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"accesskit",
|
||||
"bitflags 2.11.1",
|
||||
@@ -8843,7 +8843,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-fonts"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"app_units",
|
||||
"bitflags 2.11.1",
|
||||
@@ -8900,7 +8900,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-fonts-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"atomic_refcell",
|
||||
"dwrote",
|
||||
@@ -8924,7 +8924,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-geometry"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"app_units",
|
||||
"euclid",
|
||||
@@ -8937,7 +8937,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-hyper-serde"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"cookie 0.18.1",
|
||||
"headers",
|
||||
@@ -8951,7 +8951,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-jstraceable-derive"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"proc-macro2",
|
||||
"syn 2.0.117",
|
||||
@@ -8961,7 +8961,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-layout"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"accesskit",
|
||||
"app_units",
|
||||
@@ -9018,7 +9018,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-layout-api"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"app_units",
|
||||
"atomic_refcell",
|
||||
@@ -9051,7 +9051,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-malloc-size-of"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"app_units",
|
||||
"atomic_refcell",
|
||||
@@ -9094,7 +9094,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"servo-base",
|
||||
"servo-media-audio",
|
||||
@@ -9107,7 +9107,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-audio"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"byte-slice-cast",
|
||||
"euclid",
|
||||
@@ -9130,7 +9130,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-derive"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"proc-macro2",
|
||||
"quote",
|
||||
@@ -9140,7 +9140,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-dummy"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"servo-base",
|
||||
"servo-media",
|
||||
@@ -9154,7 +9154,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-ohos"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"crossbeam-channel",
|
||||
"libc",
|
||||
@@ -9177,7 +9177,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-player"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"malloc_size_of_derive",
|
||||
"serde",
|
||||
@@ -9190,7 +9190,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-streams"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"malloc_size_of_derive",
|
||||
"servo-malloc-size-of",
|
||||
@@ -9200,7 +9200,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-thread"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"euclid",
|
||||
"ipc-channel",
|
||||
@@ -9219,7 +9219,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"malloc_size_of_derive",
|
||||
"servo-malloc-size-of",
|
||||
@@ -9228,7 +9228,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-media-webrtc"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"log",
|
||||
"servo-media-streams",
|
||||
@@ -9238,7 +9238,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-metrics"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"malloc_size_of_derive",
|
||||
"servo-base",
|
||||
@@ -9253,7 +9253,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-net"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"async-compression",
|
||||
"async-recursion",
|
||||
@@ -9323,7 +9323,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-net-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"content-security-policy",
|
||||
"cookie 0.18.1",
|
||||
@@ -9363,7 +9363,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-paint"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"bitflags 2.11.1",
|
||||
"crossbeam-channel",
|
||||
@@ -9400,7 +9400,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-paint-api"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"bitflags 2.11.1",
|
||||
"crossbeam-channel",
|
||||
@@ -9435,7 +9435,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-pixels"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"euclid",
|
||||
"image",
|
||||
@@ -9450,7 +9450,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-profile"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"libc",
|
||||
"log",
|
||||
@@ -9468,7 +9468,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-profile-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"crossbeam-channel",
|
||||
"ipc-channel",
|
||||
@@ -9484,7 +9484,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-script"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"aes 0.9.1",
|
||||
"aes-gcm",
|
||||
@@ -9624,7 +9624,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-script-bindings"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"bitflags 2.11.1",
|
||||
"crossbeam-channel",
|
||||
@@ -9663,7 +9663,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-script-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"accesskit",
|
||||
"crossbeam-channel",
|
||||
@@ -9697,7 +9697,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-storage"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"libc",
|
||||
"log",
|
||||
@@ -9722,7 +9722,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-storage-traits"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"malloc_size_of_derive",
|
||||
"serde",
|
||||
@@ -9736,7 +9736,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-timers"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"crossbeam-channel",
|
||||
"malloc_size_of_derive",
|
||||
@@ -9746,7 +9746,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-tracing"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"proc-macro2",
|
||||
"quote",
|
||||
@@ -9756,7 +9756,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-url"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"encoding_rs",
|
||||
"malloc_size_of_derive",
|
||||
@@ -9770,7 +9770,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-wakelock"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"serde",
|
||||
"servo-embedder-traits",
|
||||
@@ -9779,7 +9779,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-webgl"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"bitflags 2.11.1",
|
||||
"byteorder",
|
||||
@@ -9803,7 +9803,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-webxr-api"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"euclid",
|
||||
"ipc-channel",
|
||||
@@ -9819,7 +9819,7 @@ dependencies = [
|
||||
[[package]]
|
||||
name = "servo-xpath"
|
||||
version = "0.4.0"
|
||||
source = "git+https://github.com/servo/servo.git?rev=a87625150758fe91a29e7f6553ff2f798b87027e#a87625150758fe91a29e7f6553ff2f798b87027e"
|
||||
source = "git+https://github.com/ZacharyZhang-NY/servo.git?rev=8f6850ceb0bd92aea4127d8078ebb2c0485e7f39#8f6850ceb0bd92aea4127d8078ebb2c0485e7f39"
|
||||
dependencies = [
|
||||
"log",
|
||||
"malloc_size_of_derive",
|
||||
|
||||
+1
-1
@@ -36,7 +36,7 @@ hpke = { version = "0.14.0", default-features = false, features = ["alloc", "cha
|
||||
image = "0.25.10"
|
||||
keyring-core = "1.0.0"
|
||||
rustix = "1.1.4"
|
||||
servo = { version = "0.4.0", git = "https://github.com/servo/servo.git", rev = "a87625150758fe91a29e7f6553ff2f798b87027e" }
|
||||
servo = { version = "0.4.0", git = "https://github.com/ZacharyZhang-NY/servo.git", rev = "8f6850ceb0bd92aea4127d8078ebb2c0485e7f39" }
|
||||
sha2 = "0.10.9"
|
||||
semver = "1.0.28"
|
||||
serde = { version = "1.0.228", features = ["derive"] }
|
||||
|
||||
@@ -13,7 +13,7 @@ in the same commit as any change that makes it stale.
|
||||
local persistence. No IO except through callers.
|
||||
- `crates/ely_servo_host` — Servo embedding, rendering contexts, the
|
||||
`ely_servo_sidecar` binary (one Servo process per profile; stdio JSON
|
||||
protocol v3 + macOS IOSurface Mach transport). See
|
||||
protocol v4 + macOS IOSurface Mach transport). See
|
||||
`docs/servo-embedding-architecture.md`.
|
||||
- `crates/ely_sync_client` — worker API client, device identity/trust,
|
||||
E2E snapshot crypto, native-keychain bearer storage, sync-owner store.
|
||||
@@ -88,9 +88,24 @@ Real and verified:
|
||||
(LOAD 1 → LOAD 2). The webview loading state reconciles through
|
||||
redirects/pushState (`awaiting_url_change`), and `run_dev.sh` resolves the
|
||||
macOS Metal toolchain so `cargo run`'s shader-build failure is fixed.
|
||||
- Scroll input: consecutive same-document wheel requests coalesce at the
|
||||
profile worker boundary while preserving total device-pixel distance;
|
||||
frames from the active render epoch remain presentable while newer input is
|
||||
queued. Clicks, text, and allow-once permission transfers retain strict
|
||||
ordering. Verified by generation, blocked-worker burst, and live-site tests.
|
||||
- macOS font fallback: CoreText selects a style- and language-compatible system
|
||||
font before Servo's static fallback list, preserving sans-serif CJK and
|
||||
Devanagari typography while retaining serif Chinese. Verified against
|
||||
PingFang SC, Hiragino Sans, Apple SD Gothic Neo, Kohinoor Devanagari, and
|
||||
Songti SC with real Servo canvas pixels.
|
||||
- Website color scheme: resolved System/Light/Dark state crosses sidecar
|
||||
protocol v4 and reaches `WebView::notify_theme_change`; GPUI appearance
|
||||
events trigger a shell repaint, and first-navigation WebView replacement
|
||||
preserves the selected scheme. Verified with real Servo media-query pixels.
|
||||
- Cross-platform: download Open/Reveal use per-OS launchers (macOS `open`,
|
||||
Windows `cmd start`/`explorer /select`, Linux `xdg-open`); the command
|
||||
overlay closes on Escape.
|
||||
overlay closes on Escape. The Windows app binary uses the GUI PE subsystem,
|
||||
so release launches do not open a companion console window.
|
||||
|
||||
Deferred deliberately (do NOT fake; ship with their subsystem):
|
||||
- Updates settings page — returns with a real updater.
|
||||
@@ -114,24 +129,22 @@ file:line evidence lives in the 2026-07-10 bug-sweep report):
|
||||
the syncable entities persist, but back/forward stacks, splits, tab
|
||||
groups, archived tabs, and downloads do not yet — extend the local-state
|
||||
document (its `settings`/`body` split is built to grow).
|
||||
3. Engine/webview: redirect or pushState leaves tab state "loading"
|
||||
forever and pins stale pixels (`ely_servo_host/src/runtime_webview.rs`
|
||||
requested-vs-current URL reconciliation); persistent-profile sidecars
|
||||
are never reclaimed while the app runs; sidecar stderr is nulled;
|
||||
final URLs >32KiB cause a reload loop.
|
||||
3. Engine/webview (partial): persistent-profile sidecars are never
|
||||
reclaimed while the app runs; sidecar stderr is nulled; final URLs
|
||||
>32KiB cause a reload loop. (Fixed: redirect/pushState loading-state
|
||||
reconciliation.)
|
||||
4. Downloads engine: pause/resume/cancel/retry are UI-only, progress
|
||||
never updates, checksum runs on the UI thread, open/reveal hardcode
|
||||
`/usr/bin/open`; `ely://auth/callback` exchange and save-page commands
|
||||
are unimplemented.
|
||||
never updates, checksum runs on the UI thread; `ely://auth/callback`
|
||||
exchange and save-page commands are unimplemented. (Fixed:
|
||||
cross-platform open/reveal.)
|
||||
5. Smaller confirmed papercuts: several synced mutations never schedule
|
||||
an upload (splits, group toggles, deletions), trash_space leaks split
|
||||
layouts, reload of the current URL is a no-op (crashed tabs can't
|
||||
reload in place), mid-Vec tab inserts skip sort normalization, Esc
|
||||
doesn't close the command overlay, "Switch workspace" palette entry
|
||||
mislabels its action, Profiles page cannot create/delete profiles,
|
||||
vault rotation silently skips devices without wrapping keys,
|
||||
SyncStatus counters are hardcoded, second in-process Servo host panics
|
||||
(upstream OnceLock).
|
||||
layouts, mid-Vec tab inserts skip sort normalization, "Switch
|
||||
workspace" palette entry mislabels its action, Profiles page cannot
|
||||
create/delete profiles, vault rotation silently skips devices without
|
||||
wrapping keys, SyncStatus counters are hardcoded, second in-process
|
||||
Servo host panics (upstream OnceLock). (Fixed: Esc closes the command
|
||||
overlay; the current-URL reload is real now — Cmd/Ctrl+R.)
|
||||
6. Integration tests litter the real data root with `profile_*` dirs
|
||||
(`~/Library/Application Support/com.elydora.ELY-Browser/profiles/`);
|
||||
tests should take an overridable data root.
|
||||
@@ -147,5 +160,9 @@ the trigger to ungate clipboard/geolocation/notification/webrtc prefs.
|
||||
- Red-green: reproduce with a failing test before fixing.
|
||||
- Files stay under 500 lines (CI-enforced); split like
|
||||
`runtime.rs`/`runtime_paint.rs` or `state/local_visibility.rs`.
|
||||
- GPUI color encoding is explicit: pass 24-bit `RRGGBB` tokens to `rgb()` and
|
||||
32-bit `RRGGBBAA` tokens to `rgba()`.
|
||||
- Release builds remap workspace, Cargo, Rustup, sysroot, and target paths in
|
||||
both Rust and native C/C++ objects; `verify_release_artifacts.sh` enforces it.
|
||||
- Tracing targets: `ely::sync`, `ely::local_state` — add spans/fields on
|
||||
every new failure path so field issues are diagnosable.
|
||||
|
||||
@@ -1,3 +1,5 @@
|
||||
#![cfg_attr(windows, windows_subsystem = "windows")]
|
||||
|
||||
mod brand;
|
||||
mod services;
|
||||
mod shell;
|
||||
|
||||
@@ -151,6 +151,7 @@ impl ServoLiveClient {
|
||||
height: request.height,
|
||||
page_zoom_percent: request.page_zoom_percent,
|
||||
device_pixel_ratio: request.device_pixel_ratio,
|
||||
color_scheme: request.color_scheme,
|
||||
scroll_delta_x: request.scroll_delta_x,
|
||||
scroll_delta_y: request.scroll_delta_y,
|
||||
scroll_point_x: request.scroll_point_x,
|
||||
|
||||
@@ -256,7 +256,7 @@ mod tests {
|
||||
#[test]
|
||||
fn reply_rejects_oversized_frame_before_readback_allocation() {
|
||||
let header = format!(
|
||||
"{{\"protocol_version\":3,\"error\":null,\"frame\":{{\"loaded_url\":null,\"title\":null,\"state\":\"complete\",\"width\":{0},\"height\":{0},\"device_pixel_ratio\":1.0,\"css_viewport_width\":{0},\"css_viewport_height\":{0},\"rgba_byte_count\":1073741824,\"pixels_changed\":true}}}}\n",
|
||||
"{{\"protocol_version\":4,\"error\":null,\"frame\":{{\"loaded_url\":null,\"title\":null,\"state\":\"complete\",\"width\":{0},\"height\":{0},\"device_pixel_ratio\":1.0,\"css_viewport_width\":{0},\"css_viewport_height\":{0},\"rgba_byte_count\":1073741824,\"pixels_changed\":true}}}}\n",
|
||||
MAX_FRAME_DIMENSION
|
||||
);
|
||||
let mut input = Cursor::new(header.into_bytes());
|
||||
@@ -269,19 +269,19 @@ mod tests {
|
||||
|
||||
#[test]
|
||||
fn reply_accepts_the_exact_header_limit() -> Result<(), ServoLiveError> {
|
||||
let mut header = br#"{"protocol_version":3,"error":null,"frame":null}"#.to_vec();
|
||||
let mut header = br#"{"protocol_version":4,"error":null,"frame":null}"#.to_vec();
|
||||
header.resize(MAX_RESPONSE_HEADER_BYTES - 1, b' ');
|
||||
header.push(b'\n');
|
||||
|
||||
let reply = read_reply(&mut Cursor::new(header))?;
|
||||
|
||||
assert_eq!(reply.protocol_version, Some(3));
|
||||
assert_eq!(reply.protocol_version, Some(4));
|
||||
Ok(())
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn reply_rejects_one_byte_over_the_header_limit() {
|
||||
let mut header = br#"{"protocol_version":3,"error":null,"frame":null}"#.to_vec();
|
||||
let mut header = br#"{"protocol_version":4,"error":null,"frame":null}"#.to_vec();
|
||||
header.resize(MAX_RESPONSE_HEADER_BYTES, b' ');
|
||||
header.push(b'\n');
|
||||
|
||||
@@ -327,7 +327,7 @@ mod tests {
|
||||
fn reply_parses_permission_consumption_without_a_frame() -> Result<(), ServoLiveError> {
|
||||
let profile_id = ely_domain::ProfileId::new();
|
||||
let header = format!(
|
||||
"{{\"protocol_version\":3,\"error\":null,\"frame\":null,\"permission_consumptions\":[{{\"profile_id\":\"{}\",\"origin\":\"https://example.com\",\"feature\":\"camera\",\"grant_revision\":7}}]}}\n",
|
||||
"{{\"protocol_version\":4,\"error\":null,\"frame\":null,\"permission_consumptions\":[{{\"profile_id\":\"{}\",\"origin\":\"https://example.com\",\"feature\":\"camera\",\"grant_revision\":7}}]}}\n",
|
||||
profile_id.as_str(),
|
||||
);
|
||||
let mut input = Cursor::new(header.into_bytes());
|
||||
@@ -350,7 +350,7 @@ mod tests {
|
||||
#[test]
|
||||
fn hardware_reply_uses_surface_without_rgba_allocation() -> Result<(), ServoLiveError> {
|
||||
let header = concat!(
|
||||
"{\"protocol_version\":3,\"error\":null,",
|
||||
"{\"protocol_version\":4,\"error\":null,",
|
||||
"\"surface_handle\":{\"mach_port_name\":91,\"surface_id\":7,\"width\":64,\"height\":48},",
|
||||
"\"current_surface_id\":7,",
|
||||
"\"frame\":{\"loaded_url\":null,\"title\":null,\"state\":\"complete\",",
|
||||
@@ -402,7 +402,7 @@ mod tests {
|
||||
#[cfg(target_os = "macos")]
|
||||
fn hardware_header(current_surface_id: u64, handle_width: u32, handle_height: u32) -> String {
|
||||
format!(
|
||||
"{{\"protocol_version\":3,\"error\":null,\"surface_handle\":{{\"mach_port_name\":91,\"surface_id\":7,\"width\":{handle_width},\"height\":{handle_height}}},\"current_surface_id\":{current_surface_id},\"frame\":{{\"loaded_url\":null,\"title\":null,\"state\":\"complete\",\"width\":64,\"height\":48,\"device_pixel_ratio\":1.0,\"css_viewport_width\":64,\"css_viewport_height\":48,\"rgba_byte_count\":0,\"pixels_changed\":true}}}}\n"
|
||||
"{{\"protocol_version\":4,\"error\":null,\"surface_handle\":{{\"mach_port_name\":91,\"surface_id\":7,\"width\":{handle_width},\"height\":{handle_height}}},\"current_surface_id\":{current_surface_id},\"frame\":{{\"loaded_url\":null,\"title\":null,\"state\":\"complete\",\"width\":64,\"height\":48,\"device_pixel_ratio\":1.0,\"css_viewport_width\":64,\"css_viewport_height\":48,\"rgba_byte_count\":0,\"pixels_changed\":true}}}}\n"
|
||||
)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -2,6 +2,7 @@
|
||||
use std::sync::Arc;
|
||||
use std::{collections::TryReserveError, io, path::PathBuf};
|
||||
|
||||
use ely_domain::ColorScheme;
|
||||
use serde::Serialize;
|
||||
use thiserror::Error;
|
||||
|
||||
@@ -24,6 +25,7 @@ pub(crate) struct ServoLiveEnsureRequest {
|
||||
pub(crate) page_zoom_percent: u16,
|
||||
/// Display scale factor used to derive Servo's CSS viewport.
|
||||
pub(crate) device_pixel_ratio: f32,
|
||||
pub(crate) color_scheme: ColorScheme,
|
||||
pub(crate) scroll_delta_x: i32,
|
||||
pub(crate) scroll_delta_y: i32,
|
||||
pub(crate) scroll_point_x: Option<u32>,
|
||||
|
||||
@@ -1,8 +1,9 @@
|
||||
use ely_domain::ColorScheme;
|
||||
use serde::{Deserialize, Serialize};
|
||||
|
||||
use super::ServoLiveSitePermission;
|
||||
|
||||
pub(super) const LIVE_PROTOCOL_VERSION: u32 = 3;
|
||||
pub(super) const LIVE_PROTOCOL_VERSION: u32 = 4;
|
||||
pub(super) const MAX_FRAME_DIMENSION: u32 = 16_384;
|
||||
pub(super) const MAX_FRAME_BYTE_COUNT: usize = 256 * 1024 * 1024;
|
||||
pub(super) const MAX_RESPONSE_HEADER_BYTES: usize = 256 * 1024;
|
||||
@@ -22,6 +23,7 @@ pub(super) enum LiveRequest {
|
||||
height: u32,
|
||||
page_zoom_percent: u16,
|
||||
device_pixel_ratio: f32,
|
||||
color_scheme: ColorScheme,
|
||||
scroll_delta_x: i32,
|
||||
scroll_delta_y: i32,
|
||||
scroll_point_x: Option<u32>,
|
||||
|
||||
@@ -11,9 +11,10 @@ use gpui_component::{input::Input, scroll::ScrollableElement};
|
||||
use crate::shell::auth::AuthFlowPhase;
|
||||
|
||||
use super::sync_controls::{
|
||||
button_bg, render_card_heading, render_dual_button_row, render_field_label,
|
||||
render_inline_error, render_input, render_policy_toggle, render_primary_button,
|
||||
render_reset_button, render_secondary_button, render_sign_out_button,
|
||||
button_bg, primary_control_background, render_card_heading, render_dual_button_row,
|
||||
render_field_label, render_inline_error, render_input, render_policy_toggle,
|
||||
render_primary_button, render_reset_button, render_secondary_button, render_sign_out_button,
|
||||
render_sync_now_button,
|
||||
};
|
||||
use super::{ElyShell, render_canvas_surface};
|
||||
impl ElyShell {
|
||||
@@ -36,9 +37,10 @@ impl ElyShell {
|
||||
render_canvas_surface(
|
||||
div()
|
||||
.size_full()
|
||||
.p(px(40.0))
|
||||
.flex()
|
||||
.justify_center()
|
||||
.overflow_y_scrollbar()
|
||||
.pt(px(28.0))
|
||||
.px(px(40.0))
|
||||
.pb(px(32.0))
|
||||
.child(render_sync_body(self, snapshot, cx)),
|
||||
)
|
||||
}
|
||||
@@ -48,7 +50,7 @@ fn render_sync_body(
|
||||
snapshot: &BrowserSnapshot,
|
||||
cx: &mut Context<ElyShell>,
|
||||
) -> AnyElement {
|
||||
let body = div().max_w(px(860.0)).flex().flex_col().gap(px(18.0)).child(
|
||||
let body = div().w_full().max_w(px(780.0)).mx_auto().flex().flex_col().gap(px(20.0)).child(
|
||||
div()
|
||||
.text_size(px(26.0))
|
||||
.font_weight(FontWeight(500.0))
|
||||
@@ -58,15 +60,9 @@ fn render_sync_body(
|
||||
if !profile_allows_sync_controls(&snapshot.active_profile_kind) {
|
||||
return body.child(render_private_profile_card()).into_any_element();
|
||||
}
|
||||
body.child(
|
||||
div()
|
||||
.grid()
|
||||
.grid_cols(2)
|
||||
.gap(px(18.0))
|
||||
.child(render_account_card(shell, snapshot, cx))
|
||||
.child(render_data_card(shell, snapshot, cx)),
|
||||
)
|
||||
.into_any_element()
|
||||
body.child(render_account_card(shell, snapshot, cx))
|
||||
.child(render_data_section(shell, snapshot, cx))
|
||||
.into_any_element()
|
||||
}
|
||||
fn profile_allows_sync_controls(profile_kind: &ProfileKind) -> bool {
|
||||
profile_kind == &ProfileKind::Standard
|
||||
@@ -101,13 +97,13 @@ fn render_account_card(
|
||||
|
||||
match snapshot.sync_status.connection() {
|
||||
SyncConnectionState::SignedOut => card
|
||||
.child(render_card_heading("Account"))
|
||||
.children(account_form(shell, &snapshot.active_profile_id, cx))
|
||||
.child(render_card_heading("Sign in to sync"))
|
||||
.child(render_account_form(shell, &snapshot.active_profile_id, cx))
|
||||
.into_any_element(),
|
||||
SyncConnectionState::CredentialUnavailable { message } => card
|
||||
.child(render_card_heading("Account"))
|
||||
.child(render_card_heading("Sign in to sync"))
|
||||
.child(render_inline_error(message))
|
||||
.children(account_form(shell, &snapshot.active_profile_id, cx))
|
||||
.child(render_account_form(shell, &snapshot.active_profile_id, cx))
|
||||
.into_any_element(),
|
||||
SyncConnectionState::SigningOut => card
|
||||
.child(
|
||||
@@ -296,7 +292,7 @@ fn render_device_approve_button(
|
||||
.px(px(12.0))
|
||||
.py(px(8.0))
|
||||
.rounded(px(8.0))
|
||||
.bg(rgba(colors::accent()))
|
||||
.bg(primary_control_background())
|
||||
.text_size(px(12.0))
|
||||
.font_weight(FontWeight(500.0))
|
||||
.text_color(rgb(0xfff5e6))
|
||||
@@ -354,24 +350,23 @@ fn render_device_note(message: &'static str) -> AnyElement {
|
||||
div().text_size(px(11.5)).text_color(rgb(colors::ink_4())).child(message).into_any_element()
|
||||
}
|
||||
|
||||
fn account_form(
|
||||
fn render_account_form(
|
||||
shell: &ElyShell,
|
||||
profile_id: &ProfileId,
|
||||
cx: &mut Context<ElyShell>,
|
||||
) -> Vec<AnyElement> {
|
||||
let mut elements: Vec<AnyElement> = Vec::new();
|
||||
) -> AnyElement {
|
||||
let phase = if shell.auth_flow_phase.belongs_to(profile_id) {
|
||||
shell.auth_flow_phase.clone()
|
||||
} else {
|
||||
AuthFlowPhase::Idle
|
||||
};
|
||||
let mut form = div().max_w(px(440.0)).flex().flex_col().gap(px(10.0));
|
||||
|
||||
elements.push(render_field_label("Email"));
|
||||
elements.push(render_input(&shell.auth_email_input));
|
||||
form = form.child(render_field_label("Email")).child(render_input(&shell.auth_email_input));
|
||||
|
||||
match &phase {
|
||||
AuthFlowPhase::Idle | AuthFlowPhase::Error { .. } => {
|
||||
elements.push(render_primary_button(
|
||||
form = form.child(render_primary_button(
|
||||
shell,
|
||||
"send-otp",
|
||||
"Send code",
|
||||
@@ -383,53 +378,63 @@ fn account_form(
|
||||
));
|
||||
}
|
||||
AuthFlowPhase::SendingCode { .. } => {
|
||||
elements.push(render_primary_button(shell, "send-otp", "Sending", true, cx, |_, _| {}));
|
||||
form = form.child(render_primary_button(
|
||||
shell,
|
||||
"send-otp",
|
||||
"Sending",
|
||||
true,
|
||||
cx,
|
||||
|_, _| {},
|
||||
));
|
||||
}
|
||||
AuthFlowPhase::AwaitingOtp { .. } | AuthFlowPhase::Verifying { .. } => {
|
||||
elements.push(render_field_label("Code"));
|
||||
elements.push(render_input(&shell.auth_otp_input));
|
||||
elements.push(render_dual_button_row(
|
||||
shell,
|
||||
phase.is_busy(),
|
||||
cx,
|
||||
|shell, cx| shell.submit_email_otp_verify(cx),
|
||||
|shell, cx| shell.submit_email_otp_request(cx),
|
||||
));
|
||||
form = form
|
||||
.child(render_field_label("Code"))
|
||||
.child(render_input(&shell.auth_otp_input))
|
||||
.child(render_dual_button_row(
|
||||
shell,
|
||||
phase.is_busy(),
|
||||
cx,
|
||||
|shell, cx| shell.submit_email_otp_verify(cx),
|
||||
|shell, cx| shell.submit_email_otp_request(cx),
|
||||
));
|
||||
}
|
||||
}
|
||||
|
||||
if let Some(message) = phase.error_message() {
|
||||
elements.push(render_inline_error(message));
|
||||
form = form.child(render_inline_error(message));
|
||||
}
|
||||
|
||||
elements
|
||||
form.into_any_element()
|
||||
}
|
||||
|
||||
fn render_data_card(
|
||||
fn render_data_section(
|
||||
shell: &ElyShell,
|
||||
snapshot: &BrowserSnapshot,
|
||||
cx: &mut Context<ElyShell>,
|
||||
) -> AnyElement {
|
||||
let can_sync_now = matches!(
|
||||
snapshot.sync_status.connection(),
|
||||
SyncConnectionState::SignedIn
|
||||
| SyncConnectionState::AwaitingDeviceApproval
|
||||
| SyncConnectionState::SyncReady { .. }
|
||||
| SyncConnectionState::SyncError { .. }
|
||||
);
|
||||
div()
|
||||
.p(px(18.0))
|
||||
.rounded(px(12.0))
|
||||
.bg(rgba(card_bg()))
|
||||
.flex()
|
||||
.flex_col()
|
||||
.gap(px(12.0))
|
||||
.max_h(px(640.0))
|
||||
.overflow_y_scrollbar()
|
||||
.child(
|
||||
div()
|
||||
.flex()
|
||||
.items_center()
|
||||
.justify_between()
|
||||
.gap(px(10.0))
|
||||
.child(render_card_heading("Data"))
|
||||
.child(render_reset_button(shell, cx)),
|
||||
.child(render_card_heading("Choose what to sync"))
|
||||
.when(can_sync_now, |header| header.child(render_sync_now_button(shell, cx))),
|
||||
)
|
||||
.child(
|
||||
div().flex().flex_col().gap(px(2.0)).children(
|
||||
div().border_t_1().border_color(rgba(colors::divider())).flex().flex_col().children(
|
||||
snapshot
|
||||
.sync_status
|
||||
.objects()
|
||||
@@ -438,6 +443,7 @@ fn render_data_card(
|
||||
.map(|(index, status)| render_sync_object_row(shell, index, status, cx)),
|
||||
),
|
||||
)
|
||||
.child(div().pt(px(2.0)).flex().child(render_reset_button(cx)))
|
||||
.into_any_element()
|
||||
}
|
||||
|
||||
@@ -452,7 +458,7 @@ fn render_sync_object_row(
|
||||
.items_center()
|
||||
.justify_between()
|
||||
.gap(px(12.0))
|
||||
.py(px(9.0))
|
||||
.py(px(11.0))
|
||||
.border_b_1()
|
||||
.border_color(rgba(colors::divider()))
|
||||
.child(
|
||||
|
||||
@@ -1,11 +1,11 @@
|
||||
use ely_design_system::colors;
|
||||
use ely_domain::{SyncObjectPolicy, SyncObjectStatus};
|
||||
use gpui::{
|
||||
AnyElement, Context, FontWeight, InteractiveElement, IntoElement, ParentElement, SharedString,
|
||||
StatefulInteractiveElement, Styled, div, prelude::FluentBuilder, px, rgb, rgba,
|
||||
AnyElement, Context, FontWeight, InteractiveElement, IntoElement, ParentElement, Rgba,
|
||||
SharedString, StatefulInteractiveElement, Styled, div, prelude::FluentBuilder, px, rgb, rgba,
|
||||
};
|
||||
use gpui_component::{
|
||||
Disableable, Sizable,
|
||||
Disableable, IconName, Sizable,
|
||||
button::{Button, ButtonVariants},
|
||||
input::{Input, InputState},
|
||||
};
|
||||
@@ -13,6 +13,10 @@ use gpui_component::{
|
||||
use crate::shell::ElyShell;
|
||||
use crate::shell::chrome::animations::{chrome_motion_feedback, toggle_thumb_motion};
|
||||
|
||||
pub(super) fn primary_control_background() -> Rgba {
|
||||
rgb(colors::accent())
|
||||
}
|
||||
|
||||
pub(super) fn render_primary_button<F>(
|
||||
shell: &ElyShell,
|
||||
id: &'static str,
|
||||
@@ -30,7 +34,7 @@ where
|
||||
.px(px(14.0))
|
||||
.py(px(8.0))
|
||||
.rounded(px(8.0))
|
||||
.bg(rgba(colors::accent()))
|
||||
.bg(primary_control_background())
|
||||
.text_size(px(12.5))
|
||||
.font_weight(FontWeight(500.0))
|
||||
.text_color(rgb(0xfff5e6))
|
||||
@@ -128,21 +132,21 @@ pub(super) fn render_inline_error(message: &str) -> AnyElement {
|
||||
.into_any_element()
|
||||
}
|
||||
|
||||
pub(super) fn render_reset_button(shell: &ElyShell, cx: &mut Context<ElyShell>) -> AnyElement {
|
||||
div()
|
||||
.flex()
|
||||
.gap(px(8.0))
|
||||
.child(render_primary_button(shell, "sync-upload", "Sync now", false, cx, |shell, _| {
|
||||
shell.trigger_cloud_sync_upload();
|
||||
pub(super) fn render_sync_now_button(shell: &ElyShell, cx: &mut Context<ElyShell>) -> AnyElement {
|
||||
render_primary_button(shell, "sync-upload", "Sync now", false, cx, |shell, _| {
|
||||
shell.trigger_cloud_sync_upload();
|
||||
})
|
||||
}
|
||||
|
||||
pub(super) fn render_reset_button(cx: &mut Context<ElyShell>) -> AnyElement {
|
||||
Button::new("sync-reset")
|
||||
.ghost()
|
||||
.xsmall()
|
||||
.icon(IconName::Undo2)
|
||||
.label("Reset sync preferences")
|
||||
.on_click(cx.listener(|shell, _, _, cx| {
|
||||
shell.reset_sync_settings(cx);
|
||||
}))
|
||||
.child(render_secondary_button(
|
||||
shell,
|
||||
"sync-reset",
|
||||
"Reset to defaults",
|
||||
false,
|
||||
cx,
|
||||
|shell, cx| shell.reset_sync_settings(cx),
|
||||
))
|
||||
.into_any_element()
|
||||
}
|
||||
|
||||
@@ -155,7 +159,7 @@ pub(super) fn render_policy_toggle(
|
||||
let enabled = status.policy() == SyncObjectPolicy::Enabled;
|
||||
let next_policy = if enabled { SyncObjectPolicy::Paused } else { SyncObjectPolicy::Enabled };
|
||||
let kind = status.kind();
|
||||
let track_color = if enabled { colors::accent() } else { 0x281e1426 };
|
||||
let track_color = if enabled { primary_control_background() } else { rgba(0x281e1426) };
|
||||
let id = SharedString::from(format!("sync-policy-{index}"));
|
||||
let press_id = shell.chrome_motion_animation_id(id.as_str());
|
||||
let thumb_press_id = press_id.clone();
|
||||
@@ -166,7 +170,7 @@ pub(super) fn render_policy_toggle(
|
||||
.w(px(34.0))
|
||||
.h(px(20.0))
|
||||
.rounded_full()
|
||||
.bg(rgba(track_color))
|
||||
.bg(track_color)
|
||||
.p(px(2.0))
|
||||
.cursor_pointer()
|
||||
.hover(|style| style.opacity(0.9))
|
||||
@@ -227,3 +231,13 @@ pub(super) fn button_bg() -> u32 {
|
||||
fn button_bg_hover() -> u32 {
|
||||
colors::pick(0xffffffeb, 0x1f1d1beb)
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
mod tests {
|
||||
use super::primary_control_background;
|
||||
|
||||
#[test]
|
||||
fn primary_control_background_is_opaque_brand_accent() {
|
||||
assert_eq!(u32::from(primary_control_background()), 0xc96442ff);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -136,6 +136,7 @@ pub struct ElyShell {
|
||||
pub(crate) local_state_save_scheduled: bool,
|
||||
_command_subscription: Subscription,
|
||||
_translucency_subscription: Subscription,
|
||||
_appearance_subscription: Subscription,
|
||||
_quit_save_subscription: Option<Subscription>,
|
||||
}
|
||||
|
||||
@@ -251,6 +252,8 @@ impl ElyShell {
|
||||
}
|
||||
ShellState::StartupError(_) => None,
|
||||
};
|
||||
let appearance_subscription =
|
||||
cx.observe_window_appearance(window, |_shell, _window, cx| cx.notify());
|
||||
let mut shell = Self {
|
||||
state,
|
||||
focus_handle: cx.focus_handle(),
|
||||
@@ -307,6 +310,7 @@ impl ElyShell {
|
||||
auth_flow_phase: auth::AuthFlowPhase::Idle,
|
||||
_command_subscription: command_subscription,
|
||||
_translucency_subscription: translucency_subscription,
|
||||
_appearance_subscription: appearance_subscription,
|
||||
_quit_save_subscription: None,
|
||||
};
|
||||
shell._quit_save_subscription = Some(local_persistence::register_quit_save(cx));
|
||||
|
||||
@@ -27,35 +27,39 @@ impl Render for ElyShell {
|
||||
match &self.state {
|
||||
ShellState::Ready(core) => match core.snapshot() {
|
||||
Ok(snapshot) => {
|
||||
apply_color_mode(
|
||||
resolve_color_mode(snapshot.appearance.theme_mode(), appearance),
|
||||
cx,
|
||||
);
|
||||
let color_scheme =
|
||||
resolve_color_scheme(snapshot.appearance.theme_mode(), appearance);
|
||||
self.web_surfaces.set_color_scheme(color_scheme);
|
||||
apply_color_scheme(color_scheme, cx);
|
||||
match active_tab_from_snapshot(&snapshot) {
|
||||
Some(active_tab) => self.render_browser(&snapshot, active_tab, window, cx),
|
||||
None => render_error("active tab missing from snapshot".to_string()),
|
||||
}
|
||||
}
|
||||
Err(error) => {
|
||||
apply_color_mode(
|
||||
resolve_color_mode(ely_domain::ThemeMode::default(), appearance),
|
||||
cx,
|
||||
);
|
||||
let color_scheme =
|
||||
resolve_color_scheme(ely_domain::ThemeMode::default(), appearance);
|
||||
self.web_surfaces.set_color_scheme(color_scheme);
|
||||
apply_color_scheme(color_scheme, cx);
|
||||
render_error(error.to_string())
|
||||
}
|
||||
},
|
||||
ShellState::StartupError(message) => {
|
||||
apply_color_mode(
|
||||
resolve_color_mode(ely_domain::ThemeMode::default(), appearance),
|
||||
cx,
|
||||
);
|
||||
let color_scheme =
|
||||
resolve_color_scheme(ely_domain::ThemeMode::default(), appearance);
|
||||
self.web_surfaces.set_color_scheme(color_scheme);
|
||||
apply_color_scheme(color_scheme, cx);
|
||||
render_error(message.clone())
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
fn apply_color_mode(mode: colors::Mode, cx: &mut Context<ElyShell>) {
|
||||
fn apply_color_scheme(color_scheme: ely_domain::ColorScheme, cx: &mut Context<ElyShell>) {
|
||||
let mode = match color_scheme {
|
||||
ely_domain::ColorScheme::Light => colors::Mode::Light,
|
||||
ely_domain::ColorScheme::Dark => colors::Mode::Dark,
|
||||
};
|
||||
colors::set_mode(mode);
|
||||
|
||||
let component_mode = match mode {
|
||||
@@ -68,22 +72,19 @@ fn apply_color_mode(mode: colors::Mode, cx: &mut Context<ElyShell>) {
|
||||
gpui_component::Theme::global_mut(cx).font_family = SANS_FAMILY.into();
|
||||
}
|
||||
|
||||
fn resolve_color_mode(
|
||||
fn resolve_color_scheme(
|
||||
theme_mode: ely_domain::ThemeMode,
|
||||
window_appearance: gpui::WindowAppearance,
|
||||
) -> colors::Mode {
|
||||
match theme_mode {
|
||||
ely_domain::ThemeMode::Light => colors::Mode::Light,
|
||||
ely_domain::ThemeMode::Dark => colors::Mode::Dark,
|
||||
ely_domain::ThemeMode::System => match window_appearance {
|
||||
gpui::WindowAppearance::Dark | gpui::WindowAppearance::VibrantDark => {
|
||||
colors::Mode::Dark
|
||||
}
|
||||
gpui::WindowAppearance::Light | gpui::WindowAppearance::VibrantLight => {
|
||||
colors::Mode::Light
|
||||
}
|
||||
},
|
||||
}
|
||||
) -> ely_domain::ColorScheme {
|
||||
let system = match window_appearance {
|
||||
gpui::WindowAppearance::Dark | gpui::WindowAppearance::VibrantDark => {
|
||||
ely_domain::ColorScheme::Dark
|
||||
}
|
||||
gpui::WindowAppearance::Light | gpui::WindowAppearance::VibrantLight => {
|
||||
ely_domain::ColorScheme::Light
|
||||
}
|
||||
};
|
||||
theme_mode.resolve(system)
|
||||
}
|
||||
|
||||
fn active_tab_from_snapshot(snapshot: &BrowserSnapshot) -> Option<&BrowserTab> {
|
||||
@@ -428,3 +429,31 @@ pub(super) fn tab_profile_label(tab: &BrowserTab, profiles: &[ely_domain::Profil
|
||||
.map(|profile| format!("Profile: {}", profile.name()))
|
||||
.unwrap_or_else(|| format!("Profile: {}", tab.profile_id().as_str()))
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
mod tests {
|
||||
use ely_domain::{ColorScheme, ThemeMode};
|
||||
use gpui::WindowAppearance;
|
||||
|
||||
use super::resolve_color_scheme;
|
||||
|
||||
#[test]
|
||||
fn resolved_color_scheme_tracks_browser_and_system_modes() {
|
||||
assert_eq!(
|
||||
resolve_color_scheme(ThemeMode::System, WindowAppearance::Dark),
|
||||
ColorScheme::Dark,
|
||||
);
|
||||
assert_eq!(
|
||||
resolve_color_scheme(ThemeMode::System, WindowAppearance::VibrantLight),
|
||||
ColorScheme::Light,
|
||||
);
|
||||
assert_eq!(
|
||||
resolve_color_scheme(ThemeMode::Light, WindowAppearance::Dark),
|
||||
ColorScheme::Light,
|
||||
);
|
||||
assert_eq!(
|
||||
resolve_color_scheme(ThemeMode::Dark, WindowAppearance::Light),
|
||||
ColorScheme::Dark,
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
use std::collections::BTreeMap;
|
||||
use std::time::{Duration, Instant};
|
||||
|
||||
use ely_domain::{BrowserTab, ProfileId, TabId, UrlText};
|
||||
use ely_domain::{BrowserTab, ColorScheme, ProfileId, TabId, UrlText};
|
||||
|
||||
use crate::services::{ProfileDataMode, servo_live::ServoLivePermissionGrant};
|
||||
|
||||
@@ -23,16 +23,27 @@ pub(super) struct WebSurfaceStore {
|
||||
/// Singleton because only one tab at a time holds keyboard focus
|
||||
/// across the whole window. Lives on the store, not per-tab.
|
||||
pub(super) keyboard_focus: Option<WebSurfaceKeyboardFocusState>,
|
||||
color_scheme: ColorScheme,
|
||||
}
|
||||
|
||||
impl WebSurfaceStore {
|
||||
pub(super) fn new() -> Self {
|
||||
Self { runtime: WebSurfaceRuntime::new(), surfaces: BTreeMap::new(), keyboard_focus: None }
|
||||
Self {
|
||||
runtime: WebSurfaceRuntime::new(),
|
||||
surfaces: BTreeMap::new(),
|
||||
keyboard_focus: None,
|
||||
color_scheme: ColorScheme::Light,
|
||||
}
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
pub(super) fn new_with_runtime(runtime: WebSurfaceRuntime) -> Self {
|
||||
Self { runtime, surfaces: BTreeMap::new(), keyboard_focus: None }
|
||||
Self {
|
||||
runtime,
|
||||
surfaces: BTreeMap::new(),
|
||||
keyboard_focus: None,
|
||||
color_scheme: ColorScheme::Light,
|
||||
}
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
@@ -72,6 +83,7 @@ impl WebSurfaceStore {
|
||||
tab.profile_id().clone(),
|
||||
profile_data_mode,
|
||||
tab.zoom_percent(),
|
||||
self.color_scheme,
|
||||
permissions,
|
||||
);
|
||||
let scope_changed = self.surface_mut(tab.id()).reset_for_scope_change(&ensure_key);
|
||||
@@ -365,6 +377,11 @@ impl WebSurfaceStore {
|
||||
}
|
||||
}
|
||||
|
||||
pub(super) fn set_color_scheme(&mut self, color_scheme: ColorScheme) {
|
||||
self.color_scheme = color_scheme;
|
||||
self.runtime.set_color_scheme(color_scheme);
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
pub(super) fn surface_for_test(&self, tab_id: &TabId) -> Option<&PerTabSurface> {
|
||||
self.surfaces.get(tab_id)
|
||||
|
||||
@@ -4,7 +4,7 @@ use std::{
|
||||
time::{Duration, Instant},
|
||||
};
|
||||
|
||||
use ely_domain::{BrowserTab, TabId};
|
||||
use ely_domain::{BrowserTab, ColorScheme, TabId};
|
||||
|
||||
use crate::services::{
|
||||
ProfileDataMode,
|
||||
@@ -43,6 +43,7 @@ pub(super) struct WebSurfaceRuntime {
|
||||
transient_cleanup_error: Option<String>,
|
||||
client_factory: LiveRuntimeClientFactory,
|
||||
last_generation: u64,
|
||||
color_scheme: ColorScheme,
|
||||
}
|
||||
|
||||
const SIDECAR_RESTART_BASE_DELAY: Duration = Duration::from_millis(250);
|
||||
@@ -60,6 +61,7 @@ impl WebSurfaceRuntime {
|
||||
transient_cleanup_error,
|
||||
client_factory: new_servo_live_client,
|
||||
last_generation: 0,
|
||||
color_scheme: ColorScheme::Light,
|
||||
}
|
||||
}
|
||||
|
||||
@@ -74,6 +76,7 @@ impl WebSurfaceRuntime {
|
||||
transient_cleanup_error: None,
|
||||
client_factory,
|
||||
last_generation: 0,
|
||||
color_scheme: ColorScheme::Light,
|
||||
}
|
||||
}
|
||||
|
||||
@@ -101,9 +104,11 @@ impl WebSurfaceRuntime {
|
||||
let submitted_at = Instant::now();
|
||||
let started_loading = {
|
||||
let session = session_for_scope(&mut self.sessions, tab.id(), scope.clone());
|
||||
let started_loading = session.started_loading(&requested_url, size, zoom_percent);
|
||||
let started_loading =
|
||||
session.started_loading(&requested_url, size, zoom_percent, self.color_scheme);
|
||||
if started_loading {
|
||||
session.pending_user_navigation = false;
|
||||
session.frame_generation_floor = Some(generation);
|
||||
}
|
||||
if user_navigation_input {
|
||||
session.pending_user_navigation = true;
|
||||
@@ -111,6 +116,7 @@ impl WebSurfaceRuntime {
|
||||
session.requested_url = requested_url.clone();
|
||||
session.size = size;
|
||||
session.zoom_percent = zoom_percent;
|
||||
session.color_scheme = Some(self.color_scheme);
|
||||
session.scroll_offset = next_scroll_offset;
|
||||
session.generation = Some(generation);
|
||||
session.cadence.note_ensure(input_kind, started_loading, submitted_at);
|
||||
@@ -125,6 +131,7 @@ impl WebSurfaceRuntime {
|
||||
height: size.height,
|
||||
page_zoom_percent: zoom_percent,
|
||||
device_pixel_ratio: size.device_pixel_ratio_f32(),
|
||||
color_scheme: self.color_scheme,
|
||||
scroll_delta_x,
|
||||
scroll_delta_y,
|
||||
scroll_point_x,
|
||||
@@ -226,6 +233,10 @@ impl WebSurfaceRuntime {
|
||||
})
|
||||
}
|
||||
|
||||
pub(super) fn set_color_scheme(&mut self, color_scheme: ColorScheme) {
|
||||
self.color_scheme = color_scheme;
|
||||
}
|
||||
|
||||
pub(super) fn prepare_tab_scope(
|
||||
&mut self,
|
||||
tab_id: &TabId,
|
||||
@@ -288,7 +299,7 @@ impl WebSurfaceRuntime {
|
||||
continue;
|
||||
};
|
||||
if !self.sessions.get(&tab_id_obj).is_some_and(|session| {
|
||||
&session.scope == scope && session.generation == Some(generation)
|
||||
&session.scope == scope && session.accepts_frame_generation(generation)
|
||||
}) {
|
||||
continue;
|
||||
}
|
||||
@@ -472,3 +483,6 @@ mod retry_tests;
|
||||
#[cfg(test)]
|
||||
#[path = "web_surface_runtime_tests.rs"]
|
||||
mod tests;
|
||||
#[cfg(test)]
|
||||
#[path = "web_surface_theme_tests.rs"]
|
||||
mod theme_tests;
|
||||
|
||||
@@ -145,6 +145,35 @@ fn late_frame_is_discarded_after_tab_session_closes() -> Result<(), String> {
|
||||
Ok(())
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn earlier_frame_from_same_render_epoch_remains_presentable() -> Result<(), String> {
|
||||
let mut runtime = WebSurfaceRuntime::new_with_client_factory(empty_client_factory);
|
||||
let tab_id = TabId::new();
|
||||
let profile_id = ProfileId::new();
|
||||
let tab = web_tab(tab_id.clone(), profile_id.clone(), "https://example.com/page")?;
|
||||
|
||||
runtime.ensure_tab(&tab, surface_size(), ProfileDataMode::Transient, &[], pending_input())?;
|
||||
let earlier = current_generation(&runtime, &tab_id)?;
|
||||
runtime.ensure_tab(&tab, surface_size(), ProfileDataMode::Transient, &[], pending_input())?;
|
||||
let latest = current_generation(&runtime, &tab_id)?;
|
||||
|
||||
let mut frames = Vec::new();
|
||||
runtime.collect_responses(
|
||||
&scope(&profile_id),
|
||||
vec![WorkerResponse::Frame {
|
||||
generation: earlier,
|
||||
tab_id: tab_id.as_str().to_string(),
|
||||
frame: live_frame(),
|
||||
}],
|
||||
Instant::now(),
|
||||
&mut frames,
|
||||
);
|
||||
|
||||
assert!(earlier < latest);
|
||||
assert!(matches!(frames.as_slice(), [WebSurfaceRuntimeFrame::Ready { .. }]));
|
||||
Ok(())
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn scope_change_returns_submitted_permission_grants_on_the_next_tick() -> Result<(), String> {
|
||||
let mut runtime = WebSurfaceRuntime::new_with_client_factory(empty_client_factory);
|
||||
|
||||
@@ -57,7 +57,9 @@ fn successful_frame_resets_scope_retry_state() {
|
||||
let scope = WebSurfaceRuntimeScope::new(ProfileId::new(), ProfileDataMode::Transient);
|
||||
let tab_id = TabId::new();
|
||||
let generation = RequestGeneration::new(1);
|
||||
session_for_scope(&mut runtime.sessions, &tab_id, scope.clone()).generation = Some(generation);
|
||||
let session = session_for_scope(&mut runtime.sessions, &tab_id, scope.clone());
|
||||
session.generation = Some(generation);
|
||||
session.frame_generation_floor = Some(generation);
|
||||
runtime.note_scope_failure(&scope, Instant::now());
|
||||
let mut frames = Vec::new();
|
||||
|
||||
|
||||
@@ -8,7 +8,7 @@ use crate::services::{
|
||||
transient_profile_data_dir,
|
||||
},
|
||||
};
|
||||
use ely_domain::{ProfileId, TabId};
|
||||
use ely_domain::{ColorScheme, ProfileId, TabId};
|
||||
|
||||
use super::{
|
||||
web_surface_cadence::WebSurfacePollCadence,
|
||||
@@ -42,6 +42,8 @@ pub(super) struct WebSurfaceSession {
|
||||
pub(super) scroll_offset: WebSurfaceScrollOffset,
|
||||
pub(super) pending_user_navigation: bool,
|
||||
pub(super) generation: Option<RequestGeneration>,
|
||||
pub(super) frame_generation_floor: Option<RequestGeneration>,
|
||||
pub(super) color_scheme: Option<ColorScheme>,
|
||||
pub(super) cadence: WebSurfacePollCadence,
|
||||
}
|
||||
|
||||
@@ -55,6 +57,8 @@ impl WebSurfaceSession {
|
||||
scroll_offset: WebSurfaceScrollOffset::default(),
|
||||
pending_user_navigation: false,
|
||||
generation: None,
|
||||
frame_generation_floor: None,
|
||||
color_scheme: None,
|
||||
cadence: WebSurfacePollCadence::default(),
|
||||
}
|
||||
}
|
||||
@@ -64,10 +68,17 @@ impl WebSurfaceSession {
|
||||
requested_url: &str,
|
||||
size: WebSurfaceSize,
|
||||
zoom_percent: u16,
|
||||
color_scheme: ColorScheme,
|
||||
) -> bool {
|
||||
self.requested_url != requested_url
|
||||
|| self.size != size
|
||||
|| self.zoom_percent != zoom_percent
|
||||
|| self.color_scheme != Some(color_scheme)
|
||||
}
|
||||
|
||||
pub(super) fn accepts_frame_generation(&self, generation: RequestGeneration) -> bool {
|
||||
self.frame_generation_floor.is_some_and(|floor| generation >= floor)
|
||||
&& self.generation.is_some_and(|latest| generation <= latest)
|
||||
}
|
||||
|
||||
pub(super) fn url_change_for(
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
use std::time::{Duration, Instant};
|
||||
|
||||
use ely_domain::{ProfileId, TabId};
|
||||
use ely_domain::{ColorScheme, ProfileId, TabId};
|
||||
use gpui::{Bounds, Pixels};
|
||||
|
||||
use crate::services::ProfileDataMode;
|
||||
@@ -309,6 +309,7 @@ pub(super) struct WebSurfaceEnsureKey {
|
||||
profile_id: ProfileId,
|
||||
profile_data_mode: ProfileDataMode,
|
||||
zoom_percent: u16,
|
||||
color_scheme: ColorScheme,
|
||||
permissions: Vec<WebSurfaceSitePermission>,
|
||||
}
|
||||
|
||||
@@ -319,6 +320,7 @@ impl WebSurfaceEnsureKey {
|
||||
profile_id: ProfileId,
|
||||
profile_data_mode: ProfileDataMode,
|
||||
zoom_percent: u16,
|
||||
color_scheme: ColorScheme,
|
||||
permissions: &[WebSurfaceSitePermission],
|
||||
) -> Self {
|
||||
Self {
|
||||
@@ -327,6 +329,7 @@ impl WebSurfaceEnsureKey {
|
||||
profile_id,
|
||||
profile_data_mode,
|
||||
zoom_percent,
|
||||
color_scheme,
|
||||
permissions: permissions.to_vec(),
|
||||
}
|
||||
}
|
||||
@@ -446,6 +449,7 @@ mod tests {
|
||||
profile_id.clone(),
|
||||
ProfileDataMode::Persistent,
|
||||
100,
|
||||
ColorScheme::Light,
|
||||
&[],
|
||||
)
|
||||
}
|
||||
|
||||
@@ -0,0 +1,73 @@
|
||||
use std::sync::Mutex;
|
||||
|
||||
use ely_domain::{BrowserTab, ColorScheme, ProfileId, SpaceId, TabId, UrlText};
|
||||
use gpui::{Bounds, point, px, size};
|
||||
|
||||
use crate::services::{
|
||||
ProfileDataMode,
|
||||
servo_live::{ServoLiveEnsureRequest, ServoLiveFrame},
|
||||
};
|
||||
|
||||
use super::{
|
||||
super::{
|
||||
web_surface::WebSurfaceStore,
|
||||
web_surface_state::WebSurfaceInputOutcome,
|
||||
web_surface_worker::{LiveRuntimeClient, LiveRuntimeClientError},
|
||||
},
|
||||
WebSurfaceRuntime,
|
||||
};
|
||||
|
||||
static COLOR_SCHEMES: Mutex<Vec<ColorScheme>> = Mutex::new(Vec::new());
|
||||
|
||||
struct ThemeRecordingClient;
|
||||
|
||||
impl LiveRuntimeClient for ThemeRecordingClient {
|
||||
fn ensure(
|
||||
&mut self,
|
||||
request: ServoLiveEnsureRequest,
|
||||
) -> Result<Option<ServoLiveFrame>, LiveRuntimeClientError> {
|
||||
COLOR_SCHEMES
|
||||
.lock()
|
||||
.map_err(|_| "color scheme recorder lock was poisoned".to_string())?
|
||||
.push(request.color_scheme);
|
||||
Ok(None)
|
||||
}
|
||||
|
||||
fn poll(&mut self, _tab_id: String) -> Result<Option<ServoLiveFrame>, LiveRuntimeClientError> {
|
||||
Ok(None)
|
||||
}
|
||||
|
||||
fn close(&mut self, _tab_id: String) -> Result<(), LiveRuntimeClientError> {
|
||||
Ok(())
|
||||
}
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn browser_color_scheme_reaches_each_web_surface_ensure() -> Result<(), String> {
|
||||
COLOR_SCHEMES.lock().map_err(|_| "color scheme recorder lock was poisoned")?.clear();
|
||||
let runtime =
|
||||
WebSurfaceRuntime::new_with_client_factory(|_| Ok(Box::new(ThemeRecordingClient)));
|
||||
let mut store = WebSurfaceStore::new_with_runtime(runtime);
|
||||
let tab = BrowserTab::new(
|
||||
TabId::new(),
|
||||
SpaceId::new(),
|
||||
ProfileId::new(),
|
||||
"Theme",
|
||||
UrlText::parse("https://example.com/theme").map_err(|error| error.to_string())?,
|
||||
);
|
||||
let bounds = Bounds::new(point(px(0.0), px(0.0)), size(px(640.0), px(480.0)));
|
||||
|
||||
assert_eq!(store.record_viewport_size(tab.id(), bounds, 1.0), WebSurfaceInputOutcome::Applied,);
|
||||
let _ = store.ensure_surface(&tab, ProfileDataMode::Transient, &[]);
|
||||
store.flush_runtime_for_test();
|
||||
|
||||
store.set_color_scheme(ColorScheme::Dark);
|
||||
let _ = store.ensure_surface(&tab, ProfileDataMode::Transient, &[]);
|
||||
store.flush_runtime_for_test();
|
||||
|
||||
assert_eq!(
|
||||
*COLOR_SCHEMES.lock().map_err(|_| "color scheme recorder lock was poisoned")?,
|
||||
vec![ColorScheme::Light, ColorScheme::Dark],
|
||||
);
|
||||
Ok(())
|
||||
}
|
||||
@@ -13,8 +13,8 @@ use crate::services::servo_live::{
|
||||
#[path = "web_surface_worker_dispatch.rs"]
|
||||
mod dispatch;
|
||||
use dispatch::{
|
||||
dispatch_result, forward_permission_consumptions, preserve_latest_hover,
|
||||
request_has_ordered_input,
|
||||
can_merge_consecutive_scroll, dispatch_result, forward_permission_consumptions,
|
||||
merge_consecutive_scroll, preserve_latest_hover, request_has_ordered_input,
|
||||
};
|
||||
|
||||
/// Blocking transport for one profile-scoped Servo sidecar.
|
||||
@@ -225,11 +225,12 @@ impl LiveRuntimeWorker {
|
||||
}
|
||||
let mut request = WorkerRequest::Ensure { generation, request: Box::new(request) };
|
||||
if let Some(pending) = q.pending.get_mut(&tab_id) {
|
||||
let replace_tail = pending.back().is_some_and(|tail| {
|
||||
matches!(tail, WorkerRequest::Poll { .. })
|
||||
|| (!request_has_ordered_input(&request) && !request_has_ordered_input(tail))
|
||||
});
|
||||
if replace_tail && let Some(tail) = pending.back_mut() {
|
||||
if let Some(tail) = pending.back_mut()
|
||||
&& (can_merge_consecutive_scroll(&request, tail)
|
||||
|| matches!(tail, WorkerRequest::Poll { .. })
|
||||
|| (!request_has_ordered_input(&request) && !request_has_ordered_input(tail)))
|
||||
{
|
||||
merge_consecutive_scroll(&mut request, tail);
|
||||
preserve_latest_hover(&mut request, tail);
|
||||
*tail = request;
|
||||
} else {
|
||||
@@ -241,7 +242,6 @@ impl LiveRuntimeWorker {
|
||||
}
|
||||
cvar.notify_one();
|
||||
}
|
||||
|
||||
pub(super) fn submit_poll(&self, generation: RequestGeneration, tab_id: String) -> bool {
|
||||
let (lock, cvar) = &*self.queue;
|
||||
let mut q = match lock.lock() {
|
||||
|
||||
@@ -6,6 +6,56 @@ use super::{
|
||||
LiveRuntimeClient, LiveRuntimeClientError, RequestGeneration, WorkerRequest, WorkerResponse,
|
||||
};
|
||||
|
||||
pub(super) fn can_merge_consecutive_scroll(
|
||||
latest: &WorkerRequest,
|
||||
previous: &WorkerRequest,
|
||||
) -> bool {
|
||||
let (
|
||||
WorkerRequest::Ensure { request: latest, .. },
|
||||
WorkerRequest::Ensure { request: previous, .. },
|
||||
) = (latest, previous)
|
||||
else {
|
||||
return false;
|
||||
};
|
||||
is_scroll_only(latest)
|
||||
&& is_scroll_only(previous)
|
||||
&& latest.profile_id == previous.profile_id
|
||||
&& latest.url == previous.url
|
||||
&& latest.width == previous.width
|
||||
&& latest.height == previous.height
|
||||
&& latest.page_zoom_percent == previous.page_zoom_percent
|
||||
&& latest.device_pixel_ratio == previous.device_pixel_ratio
|
||||
&& latest.color_scheme == previous.color_scheme
|
||||
}
|
||||
|
||||
pub(super) fn merge_consecutive_scroll(latest: &mut WorkerRequest, previous: &WorkerRequest) {
|
||||
if !can_merge_consecutive_scroll(latest, previous) {
|
||||
return;
|
||||
}
|
||||
let (
|
||||
WorkerRequest::Ensure { request: latest, .. },
|
||||
WorkerRequest::Ensure { request: previous, .. },
|
||||
) = (latest, previous)
|
||||
else {
|
||||
return;
|
||||
};
|
||||
|
||||
latest.scroll_delta_x = previous.scroll_delta_x.saturating_add(latest.scroll_delta_x);
|
||||
latest.scroll_delta_y = previous.scroll_delta_y.saturating_add(latest.scroll_delta_y);
|
||||
if latest.hover_x.is_none() && latest.hover_y.is_none() {
|
||||
latest.hover_x = previous.hover_x;
|
||||
latest.hover_y = previous.hover_y;
|
||||
}
|
||||
}
|
||||
|
||||
fn is_scroll_only(request: &crate::services::servo_live::ServoLiveEnsureRequest) -> bool {
|
||||
(request.scroll_delta_x != 0 || request.scroll_delta_y != 0)
|
||||
&& request.click_x.is_none()
|
||||
&& request.click_y.is_none()
|
||||
&& request.typed_text.is_none()
|
||||
&& !request.site_permissions.iter().any(|permission| permission.state == "allow-once")
|
||||
}
|
||||
|
||||
pub(super) fn forward_permission_consumptions(
|
||||
client: &mut dyn LiveRuntimeClient,
|
||||
response_tx: &mpsc::Sender<WorkerResponse>,
|
||||
|
||||
@@ -0,0 +1,95 @@
|
||||
use std::{
|
||||
sync::{Arc, Mutex, mpsc},
|
||||
time::Duration,
|
||||
};
|
||||
|
||||
use crate::services::servo_live::{ServoLiveEnsureRequest, ServoLiveFrame};
|
||||
|
||||
use super::super::{
|
||||
LiveRuntimeClient, LiveRuntimeClientError, LiveRuntimeWorker, RequestGeneration,
|
||||
};
|
||||
|
||||
struct BlockingScrollClient {
|
||||
deltas: Arc<Mutex<Vec<(i32, i32)>>>,
|
||||
first_started_tx: Option<mpsc::Sender<()>>,
|
||||
release_first_rx: mpsc::Receiver<()>,
|
||||
}
|
||||
|
||||
impl LiveRuntimeClient for BlockingScrollClient {
|
||||
fn ensure(
|
||||
&mut self,
|
||||
request: ServoLiveEnsureRequest,
|
||||
) -> Result<Option<ServoLiveFrame>, LiveRuntimeClientError> {
|
||||
self.deltas
|
||||
.lock()
|
||||
.map_err(|_| "scroll recorder lock was poisoned".to_string())?
|
||||
.push((request.scroll_delta_x, request.scroll_delta_y));
|
||||
if let Some(first_started_tx) = self.first_started_tx.take() {
|
||||
first_started_tx.send(()).map_err(|error| error.to_string())?;
|
||||
self.release_first_rx.recv().map_err(|error| error.to_string())?;
|
||||
}
|
||||
Ok(None)
|
||||
}
|
||||
|
||||
fn poll(&mut self, _tab_id: String) -> Result<Option<ServoLiveFrame>, LiveRuntimeClientError> {
|
||||
Ok(None)
|
||||
}
|
||||
|
||||
fn close(&mut self, _tab_id: String) -> Result<(), LiveRuntimeClientError> {
|
||||
Ok(())
|
||||
}
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn consecutive_scroll_updates_coalesce_without_losing_distance() -> Result<(), String> {
|
||||
let deltas = Arc::new(Mutex::new(Vec::new()));
|
||||
let client_deltas = deltas.clone();
|
||||
let (first_started_tx, first_started_rx) = mpsc::channel();
|
||||
let (release_first_tx, release_first_rx) = mpsc::channel();
|
||||
let worker = LiveRuntimeWorker::new(move || {
|
||||
Ok(Box::new(BlockingScrollClient {
|
||||
deltas: client_deltas,
|
||||
first_started_tx: Some(first_started_tx),
|
||||
release_first_rx,
|
||||
}))
|
||||
})?;
|
||||
|
||||
worker.submit_ensure(RequestGeneration::new(1), ensure_request(0));
|
||||
first_started_rx.recv_timeout(Duration::from_secs(1)).map_err(|error| error.to_string())?;
|
||||
for generation in 2..=101 {
|
||||
worker.submit_ensure(RequestGeneration::new(generation), ensure_request(1));
|
||||
}
|
||||
release_first_tx.send(()).map_err(|error| error.to_string())?;
|
||||
worker.wait_until_idle();
|
||||
|
||||
assert_eq!(
|
||||
*deltas.lock().map_err(|_| "scroll recorder lock was poisoned".to_string())?,
|
||||
vec![(0, 0), (100, 100)]
|
||||
);
|
||||
Ok(())
|
||||
}
|
||||
|
||||
fn ensure_request(scroll_delta: i32) -> ServoLiveEnsureRequest {
|
||||
ServoLiveEnsureRequest {
|
||||
tab_id: "tab-a".to_string(),
|
||||
profile_id: "profile".to_string(),
|
||||
url: "https://example.com/".to_string(),
|
||||
width: 640,
|
||||
height: 480,
|
||||
page_zoom_percent: 100,
|
||||
device_pixel_ratio: 1.0,
|
||||
color_scheme: ely_domain::ColorScheme::Light,
|
||||
scroll_delta_x: scroll_delta,
|
||||
scroll_delta_y: scroll_delta,
|
||||
scroll_point_x: (scroll_delta != 0).then_some(1),
|
||||
scroll_point_y: (scroll_delta != 0).then_some(1),
|
||||
click_x: None,
|
||||
click_y: None,
|
||||
hover_x: None,
|
||||
hover_y: None,
|
||||
typed_text: None,
|
||||
site_permission_generation: 1,
|
||||
site_permissions: Vec::new(),
|
||||
allow_once_grants: Vec::new(),
|
||||
}
|
||||
}
|
||||
@@ -450,6 +450,7 @@ fn ensure_request(tab_id: &str, input: RecordedInput) -> ServoLiveEnsureRequest
|
||||
height: 480,
|
||||
page_zoom_percent: 100,
|
||||
device_pixel_ratio: 1.0,
|
||||
color_scheme: ely_domain::ColorScheme::Light,
|
||||
scroll_delta_x: i32::from(scroll),
|
||||
scroll_delta_y: i32::from(scroll),
|
||||
scroll_point_x: scroll.then_some(1),
|
||||
@@ -464,3 +465,6 @@ fn ensure_request(tab_id: &str, input: RecordedInput) -> ServoLiveEnsureRequest
|
||||
allow_once_grants: Vec::new(),
|
||||
}
|
||||
}
|
||||
|
||||
#[path = "web_surface_worker_scroll_tests.rs"]
|
||||
mod scroll_tests;
|
||||
|
||||
@@ -19,6 +19,24 @@ pub enum ThemeMode {
|
||||
Dark,
|
||||
}
|
||||
|
||||
#[derive(Clone, Copy, Debug, Default, Deserialize, Eq, Ord, PartialEq, PartialOrd, Serialize)]
|
||||
#[serde(rename_all = "kebab-case")]
|
||||
pub enum ColorScheme {
|
||||
#[default]
|
||||
Light,
|
||||
Dark,
|
||||
}
|
||||
|
||||
impl ThemeMode {
|
||||
pub fn resolve(self, system: ColorScheme) -> ColorScheme {
|
||||
match self {
|
||||
Self::System => system,
|
||||
Self::Light => ColorScheme::Light,
|
||||
Self::Dark => ColorScheme::Dark,
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
pub const DEFAULT_TRANSLUCENCY_PCT: u8 = 40;
|
||||
pub const MAX_TRANSLUCENCY_PCT: u8 = 100;
|
||||
|
||||
@@ -77,7 +95,7 @@ impl AppearanceSettings {
|
||||
|
||||
#[cfg(test)]
|
||||
mod tests {
|
||||
use super::{AppearanceSettings, ThemeMode, WallpaperTheme};
|
||||
use super::{AppearanceSettings, ColorScheme, ThemeMode, WallpaperTheme};
|
||||
|
||||
#[test]
|
||||
fn default_settings_use_dawn_and_system_mode() {
|
||||
@@ -103,6 +121,13 @@ mod tests {
|
||||
assert_eq!(settings.translucency_pct(), 75);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn theme_mode_resolves_explicit_and_system_color_schemes() {
|
||||
assert_eq!(ThemeMode::System.resolve(ColorScheme::Dark), ColorScheme::Dark);
|
||||
assert_eq!(ThemeMode::Light.resolve(ColorScheme::Dark), ColorScheme::Light);
|
||||
assert_eq!(ThemeMode::Dark.resolve(ColorScheme::Light), ColorScheme::Dark);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn translucency_setter_clamps_above_max() {
|
||||
let mut settings = AppearanceSettings::default();
|
||||
|
||||
@@ -24,7 +24,8 @@ mod tab_group;
|
||||
mod url_text;
|
||||
|
||||
pub use appearance::{
|
||||
AppearanceSettings, DEFAULT_TRANSLUCENCY_PCT, MAX_TRANSLUCENCY_PCT, ThemeMode, WallpaperTheme,
|
||||
AppearanceSettings, ColorScheme, DEFAULT_TRANSLUCENCY_PCT, MAX_TRANSLUCENCY_PCT, ThemeMode,
|
||||
WallpaperTheme,
|
||||
};
|
||||
pub use archive::{ArchiveSource, ArchivedTab};
|
||||
pub use bookmark::BookmarkEntry;
|
||||
|
||||
@@ -21,8 +21,8 @@ use super::{
|
||||
},
|
||||
live_request::{MAX_REQUEST_LINE_BYTES, RequestLineRead, read_request_line},
|
||||
live_session::{
|
||||
LiveInput, LiveSession, apply_input, apply_layout, apply_permissions, bind_profile,
|
||||
ensure_session,
|
||||
LiveInput, LiveSession, apply_color_scheme, apply_input, apply_layout, apply_permissions,
|
||||
bind_profile, ensure_session,
|
||||
},
|
||||
};
|
||||
|
||||
@@ -159,6 +159,7 @@ fn handle_request(
|
||||
height,
|
||||
page_zoom_percent,
|
||||
device_pixel_ratio,
|
||||
color_scheme,
|
||||
scroll_delta_x,
|
||||
scroll_delta_y,
|
||||
scroll_point_x,
|
||||
@@ -184,6 +185,7 @@ fn handle_request(
|
||||
let session =
|
||||
ensure_session(host, sessions, tab_id.clone(), &tab, &profile, width, height)?;
|
||||
apply_layout(host, session, width, height, page_zoom_percent, device_pixel_ratio)?;
|
||||
apply_color_scheme(host, session, color_scheme)?;
|
||||
apply_permissions(
|
||||
host,
|
||||
session,
|
||||
|
||||
@@ -1,5 +1,6 @@
|
||||
use std::{io, path::PathBuf};
|
||||
|
||||
use ely_domain::ColorScheme;
|
||||
use ely_servo_host::{
|
||||
ConsumedPermission, IOSurfaceHandle, RenderedFrame, ServoHostError, WebViewSnapshot,
|
||||
WebViewState,
|
||||
@@ -10,7 +11,7 @@ use thiserror::Error;
|
||||
#[cfg(all(feature = "hardware-render", target_os = "macos"))]
|
||||
use super::iosurface_mach::IOSurfaceMachError;
|
||||
|
||||
pub(super) const LIVE_PROTOCOL_VERSION: u32 = 3;
|
||||
pub(super) const LIVE_PROTOCOL_VERSION: u32 = 4;
|
||||
pub(super) const MAX_FRAME_DIMENSION: u32 = 16_384;
|
||||
pub(super) const MAX_FRAME_BYTE_COUNT: usize = 256 * 1024 * 1024;
|
||||
pub(super) const MAX_RESPONSE_HEADER_BYTES: usize = 256 * 1024;
|
||||
@@ -35,6 +36,7 @@ pub(super) enum LiveRequest {
|
||||
page_zoom_percent: u16,
|
||||
#[serde(default = "default_device_pixel_ratio")]
|
||||
device_pixel_ratio: f32,
|
||||
color_scheme: ColorScheme,
|
||||
#[serde(default)]
|
||||
scroll_delta_x: i32,
|
||||
#[serde(default)]
|
||||
@@ -401,7 +403,7 @@ mod tests {
|
||||
#[test]
|
||||
fn ensure_defaults_optional_input_fields() -> Result<(), serde_json::Error> {
|
||||
let request = serde_json::from_str::<LiveRequest>(
|
||||
r#"{"type":"ensure","tab_id":"tab","profile_id":"profile","url":"https://example.com","width":800,"height":600,"site_permission_generation":0}"#,
|
||||
r#"{"type":"ensure","tab_id":"tab","profile_id":"profile","url":"https://example.com","width":800,"height":600,"color_scheme":"dark","site_permission_generation":0}"#,
|
||||
)?;
|
||||
|
||||
assert!(matches!(
|
||||
@@ -409,6 +411,7 @@ mod tests {
|
||||
LiveRequest::Ensure {
|
||||
page_zoom_percent: 100,
|
||||
device_pixel_ratio: 1.0,
|
||||
color_scheme: ColorScheme::Dark,
|
||||
scroll_delta_x: 0,
|
||||
scroll_delta_y: 0,
|
||||
ready_surface_ids,
|
||||
@@ -419,19 +422,28 @@ mod tests {
|
||||
Ok(())
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn ensure_requires_color_scheme() {
|
||||
let request = serde_json::from_str::<LiveRequest>(
|
||||
r#"{"type":"ensure","tab_id":"tab","profile_id":"profile","url":"https://example.com","width":800,"height":600,"site_permission_generation":0}"#,
|
||||
);
|
||||
|
||||
assert!(request.is_err());
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn handshake_deserializes_protocol_version() -> Result<(), serde_json::Error> {
|
||||
let request =
|
||||
serde_json::from_str::<LiveRequest>(r#"{"type":"handshake","protocol_version":3}"#)?;
|
||||
serde_json::from_str::<LiveRequest>(r#"{"type":"handshake","protocol_version":4}"#)?;
|
||||
|
||||
assert!(matches!(request, LiveRequest::Handshake { protocol_version: 3 }));
|
||||
assert!(matches!(request, LiveRequest::Handshake { protocol_version: 4 }));
|
||||
Ok(())
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn site_permission_requires_revision() {
|
||||
let request = serde_json::from_str::<LiveRequest>(
|
||||
r#"{"type":"ensure","tab_id":"tab","profile_id":"profile","url":"https://example.com","width":800,"height":600,"site_permission_generation":0,"site_permissions":[{"origin":"https://example.com","feature":"camera","state":"allow-once"}]}"#,
|
||||
r#"{"type":"ensure","tab_id":"tab","profile_id":"profile","url":"https://example.com","width":800,"height":600,"color_scheme":"light","site_permission_generation":0,"site_permissions":[{"origin":"https://example.com","feature":"camera","state":"allow-once"}]}"#,
|
||||
);
|
||||
|
||||
assert!(request.is_err());
|
||||
|
||||
@@ -1,11 +1,11 @@
|
||||
use std::collections::{HashMap, hash_map::Entry};
|
||||
|
||||
use ely_domain::{ProfileId, TabId, validate_zoom_percent};
|
||||
use ely_domain::{ColorScheme, ProfileId, TabId, validate_zoom_percent};
|
||||
use ely_servo_host::{
|
||||
HidpiScaleRequest, KeyboardTextRequest, MouseClickRequest, MouseHoverRequest, PageZoomRequest,
|
||||
PermissionDecision, PermissionSnapshotEntry, PermissionSnapshotRequest,
|
||||
PermissionSnapshotState, RenderedFrame, ResizeRequest, ScrollRequest, ServoHost,
|
||||
ServoSurfaceSize, SoftwareServoHost,
|
||||
ColorSchemeRequest, HidpiScaleRequest, KeyboardTextRequest, MouseClickRequest,
|
||||
MouseHoverRequest, PageZoomRequest, PermissionDecision, PermissionSnapshotEntry,
|
||||
PermissionSnapshotRequest, PermissionSnapshotState, RenderedFrame, ResizeRequest,
|
||||
ScrollRequest, ServoHost, ServoSurfaceSize, SoftwareServoHost,
|
||||
};
|
||||
|
||||
use super::live_protocol::{LiveSidecarError, LiveSitePermission};
|
||||
@@ -170,6 +170,18 @@ pub(super) fn apply_permissions(
|
||||
.map_err(LiveSidecarError::from)
|
||||
}
|
||||
|
||||
pub(super) fn apply_color_scheme(
|
||||
host: &mut SoftwareServoHost,
|
||||
session: &LiveSession,
|
||||
color_scheme: ColorScheme,
|
||||
) -> Result<(), LiveSidecarError> {
|
||||
host.set_color_scheme(ColorSchemeRequest {
|
||||
webview_id: session.webview_id.clone(),
|
||||
color_scheme,
|
||||
})
|
||||
.map_err(LiveSidecarError::from)
|
||||
}
|
||||
|
||||
pub(super) struct LiveInput {
|
||||
pub(super) scroll_delta_x: i32,
|
||||
pub(super) scroll_delta_y: i32,
|
||||
|
||||
@@ -67,7 +67,7 @@ fn request_line_accepts_a_bounded_eof_terminated_frame() -> Result<(), Box<dyn s
|
||||
}
|
||||
|
||||
fn padded_handshake_line(bytes: usize) -> Vec<u8> {
|
||||
let mut line = br#"{"type":"handshake","protocol_version":3}"#.to_vec();
|
||||
let mut line = br#"{"type":"handshake","protocol_version":4}"#.to_vec();
|
||||
assert!(bytes > line.len());
|
||||
line.resize(bytes - 1, b' ');
|
||||
line.push(b'\n');
|
||||
|
||||
@@ -1,5 +1,6 @@
|
||||
use ely_domain::{
|
||||
ProfileId, SiteOrigin, SitePermissionDecision, SitePermissionFeature, TabId, UrlText, WebViewId,
|
||||
ColorScheme, ProfileId, SiteOrigin, SitePermissionDecision, SitePermissionFeature, TabId,
|
||||
UrlText, WebViewId,
|
||||
};
|
||||
|
||||
use crate::ServoHostError;
|
||||
@@ -280,6 +281,12 @@ pub struct HidpiScaleRequest {
|
||||
pub scale_factor: f32,
|
||||
}
|
||||
|
||||
#[derive(Clone, Debug, Eq, PartialEq)]
|
||||
pub struct ColorSchemeRequest {
|
||||
pub webview_id: WebViewId,
|
||||
pub color_scheme: ColorScheme,
|
||||
}
|
||||
|
||||
#[derive(Clone, Debug, Eq, PartialEq)]
|
||||
pub struct MouseClickRequest {
|
||||
pub webview_id: WebViewId,
|
||||
@@ -385,6 +392,8 @@ pub trait ServoHost {
|
||||
|
||||
fn set_hidpi_scale(&mut self, request: HidpiScaleRequest) -> Result<(), ServoHostError>;
|
||||
|
||||
fn set_color_scheme(&mut self, request: ColorSchemeRequest) -> Result<(), ServoHostError>;
|
||||
|
||||
fn click(&mut self, request: MouseClickRequest) -> Result<(), ServoHostError>;
|
||||
|
||||
fn hover(&mut self, request: MouseHoverRequest) -> Result<(), ServoHostError>;
|
||||
|
||||
@@ -22,11 +22,12 @@ pub use error::ServoHostError;
|
||||
#[cfg(feature = "hardware-render")]
|
||||
pub use hardware_rendering_context::HardwareOffscreenContext;
|
||||
pub use host::{
|
||||
ConsumedPermission, HidpiScaleRequest, KeyboardTextRequest, MAX_PAGE_TITLE_BYTES,
|
||||
MouseClickRequest, MouseDragRequest, MouseHoverRequest, NavigationRequest, PageZoomRequest,
|
||||
PermissionDecision, PermissionSnapshotEntry, PermissionSnapshotRequest,
|
||||
PermissionSnapshotState, RenderedFrame, RenderedFrameSummary, ResizeRequest, ScrollRequest,
|
||||
ServoHost, TouchTapRequest, WebViewSnapshot, WebViewSnapshotPending, WebViewState,
|
||||
ColorSchemeRequest, ConsumedPermission, HidpiScaleRequest, KeyboardTextRequest,
|
||||
MAX_PAGE_TITLE_BYTES, MouseClickRequest, MouseDragRequest, MouseHoverRequest,
|
||||
NavigationRequest, PageZoomRequest, PermissionDecision, PermissionSnapshotEntry,
|
||||
PermissionSnapshotRequest, PermissionSnapshotState, RenderedFrame, RenderedFrameSummary,
|
||||
ResizeRequest, ScrollRequest, ServoHost, TouchTapRequest, WebViewSnapshot,
|
||||
WebViewSnapshotPending, WebViewState,
|
||||
};
|
||||
pub use iosurface_handle::{IOSurfaceHandle, IOSurfaceIdentity};
|
||||
#[cfg(feature = "servo-engine")]
|
||||
|
||||
@@ -9,11 +9,11 @@ use std::{
|
||||
};
|
||||
|
||||
use dpi::PhysicalSize;
|
||||
use ely_domain::{ProfileId, TabId, WebViewId};
|
||||
use ely_domain::{ColorScheme, ProfileId, TabId, WebViewId};
|
||||
use raw_window_handle::{HasDisplayHandle, HasWindowHandle};
|
||||
use servo::{
|
||||
DevicePoint, DeviceVector2D, Opts, Scroll, Servo, ServoBuilder, WebViewBuilder, WebViewPoint,
|
||||
WebViewVector,
|
||||
DevicePoint, DeviceVector2D, Opts, Scroll, Servo, ServoBuilder, Theme, WebViewBuilder,
|
||||
WebViewPoint, WebViewVector,
|
||||
};
|
||||
|
||||
#[path = "runtime_context.rs"]
|
||||
@@ -32,8 +32,8 @@ use runtime_preferences::ely_servo_preferences;
|
||||
use url::Url;
|
||||
|
||||
use crate::{
|
||||
ConsumedPermission, HidpiScaleRequest, KeyboardTextRequest, MouseClickRequest,
|
||||
MouseDragRequest, MouseHoverRequest, NavigationRequest, PageZoomRequest,
|
||||
ColorSchemeRequest, ConsumedPermission, HidpiScaleRequest, KeyboardTextRequest,
|
||||
MouseClickRequest, MouseDragRequest, MouseHoverRequest, NavigationRequest, PageZoomRequest,
|
||||
PermissionSnapshotRequest, RenderedFrame, ResizeRequest, ScrollRequest, ServoHost,
|
||||
ServoHostError, TouchTapRequest, WebViewSnapshot, WebViewState,
|
||||
runtime_input::{
|
||||
@@ -204,13 +204,15 @@ impl ServoHost for SoftwareServoHost {
|
||||
webview.delegate.set_state(WebViewState::Loading);
|
||||
if should_create_initial_document {
|
||||
let hidpi_scale_factor = webview.webview.hidpi_scale_factor();
|
||||
webview.webview = WebViewBuilder::new(&servo, webview.rendering_context.clone())
|
||||
let replacement = WebViewBuilder::new(&servo, webview.rendering_context.clone())
|
||||
.delegate(webview.delegate.clone())
|
||||
.url(url)
|
||||
// The live path pushes DPR before first navigation. Preserve that scale when
|
||||
// replacing the about:blank WebView so CSS viewport = physical surface / DPR.
|
||||
.hidpi_scale_factor(hidpi_scale_factor)
|
||||
.build();
|
||||
replacement.notify_theme_change(servo_theme(webview.color_scheme));
|
||||
webview.webview = replacement;
|
||||
// The input-accepting invariant lives in `webview_for_input`.
|
||||
webview.webview.show();
|
||||
webview.webview.focus();
|
||||
@@ -303,6 +305,18 @@ impl ServoHost for SoftwareServoHost {
|
||||
Ok(())
|
||||
}
|
||||
|
||||
fn set_color_scheme(&mut self, request: ColorSchemeRequest) -> Result<(), ServoHostError> {
|
||||
let webview = self
|
||||
.webviews
|
||||
.get_mut(&request.webview_id)
|
||||
.ok_or_else(|| ServoHostError::WebViewNotFound { id: request.webview_id.clone() })?;
|
||||
if webview.color_scheme != request.color_scheme {
|
||||
webview.color_scheme = request.color_scheme;
|
||||
webview.webview.notify_theme_change(servo_theme(request.color_scheme));
|
||||
}
|
||||
Ok(())
|
||||
}
|
||||
|
||||
fn hover(&mut self, request: MouseHoverRequest) -> Result<(), ServoHostError> {
|
||||
let webview = self.webview_for_input(&request.webview_id)?;
|
||||
send_mouse_hover(&webview.webview, request.x, request.y);
|
||||
@@ -440,6 +454,7 @@ impl SoftwareServoHost {
|
||||
webview,
|
||||
delegate,
|
||||
requested_url: None,
|
||||
color_scheme: ColorScheme::Light,
|
||||
},
|
||||
);
|
||||
|
||||
@@ -472,3 +487,10 @@ impl SoftwareServoHost {
|
||||
Ok(webview)
|
||||
}
|
||||
}
|
||||
|
||||
fn servo_theme(color_scheme: ColorScheme) -> Theme {
|
||||
match color_scheme {
|
||||
ColorScheme::Light => Theme::Light,
|
||||
ColorScheme::Dark => Theme::Dark,
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1,6 +1,6 @@
|
||||
use std::{cell::Cell, cell::RefCell, rc::Rc};
|
||||
|
||||
use ely_domain::{ProfileId, TabId, WebViewId};
|
||||
use ely_domain::{ColorScheme, ProfileId, TabId, WebViewId};
|
||||
use servo::{LoadStatus, RenderingContext, WebView, WebViewDelegate};
|
||||
use url::Url;
|
||||
|
||||
@@ -19,6 +19,7 @@ pub(super) struct HostWebView {
|
||||
pub(super) webview: WebView,
|
||||
pub(super) delegate: Rc<HostWebViewDelegate>,
|
||||
pub(super) requested_url: Option<String>,
|
||||
pub(super) color_scheme: ColorScheme,
|
||||
}
|
||||
|
||||
impl HostWebView {
|
||||
|
||||
@@ -0,0 +1,114 @@
|
||||
#![cfg(all(feature = "servo-engine", target_os = "macos"))]
|
||||
|
||||
use std::{error::Error, thread, time::Duration};
|
||||
|
||||
use ely_domain::{ProfileId, TabId, UrlText};
|
||||
use ely_servo_host::{
|
||||
NavigationRequest, ServoHost, ServoSurfaceSize, SoftwareServoHost, WebViewState,
|
||||
};
|
||||
|
||||
const EXPECTED_PASS_COLOR: [u8; 3] = [31, 143, 76];
|
||||
|
||||
#[test]
|
||||
fn macos_fallback_preserves_css_family_style_and_language() -> Result<(), Box<dyn Error>> {
|
||||
let mut host = SoftwareServoHost::new(ServoSurfaceSize::new(320, 240))?;
|
||||
let tab_id = TabId::new();
|
||||
let profile_id = ProfileId::new();
|
||||
let webview_id = host.create_webview(tab_id.clone(), profile_id)?;
|
||||
|
||||
host.navigate(NavigationRequest {
|
||||
webview_id: webview_id.clone(),
|
||||
tab_id,
|
||||
url: UrlText::parse(font_probe_url())?,
|
||||
})?;
|
||||
|
||||
for _ in 0..5_000 {
|
||||
host.tick();
|
||||
let snapshot = host.snapshot(&webview_id)?;
|
||||
if snapshot.has_pending_frame() {
|
||||
host.paint(&webview_id)?;
|
||||
}
|
||||
|
||||
let snapshot = host.snapshot(&webview_id)?;
|
||||
if snapshot.state() == &WebViewState::Complete
|
||||
&& host
|
||||
.last_rendered_frame()
|
||||
.is_ok_and(|frame| center_pixel_rgb(&frame) == EXPECTED_PASS_COLOR)
|
||||
{
|
||||
return Ok(());
|
||||
}
|
||||
thread::sleep(Duration::from_millis(2));
|
||||
}
|
||||
|
||||
let actual = host.last_rendered_frame().map(|frame| center_pixel_rgb(&frame));
|
||||
let title = host.snapshot(&webview_id)?.title().map(str::to_owned);
|
||||
Err(format!(
|
||||
"macOS font fallback probe failed with center pixel {actual:?} and title {title:?}"
|
||||
)
|
||||
.into())
|
||||
}
|
||||
|
||||
fn font_probe_url() -> String {
|
||||
let html = r#"<!doctype html>
|
||||
<html><head><meta charset="utf-8"><title>starting</title><style>
|
||||
html,body{margin:0;width:100%;height:100%;background:rgb(181,37,46)}
|
||||
canvas{position:absolute;left:-10000px;top:-10000px}
|
||||
</style></head><body><script>
|
||||
try {
|
||||
const cases = [
|
||||
['zh-Hans', '搜索中文', '48px Arial, sans-serif', "48px 'PingFang SC'"],
|
||||
['ja', '日本語検索', '48px Arial, sans-serif', "48px 'Hiragino Sans'"],
|
||||
['ko', '한국어검색', '48px Arial, sans-serif', "48px 'Apple SD Gothic Neo'"],
|
||||
['hi', 'हिन्दी', '48px Arial, sans-serif', "48px 'Kohinoor Devanagari'"],
|
||||
['zh-Hans', '搜索中文', '48px Times New Roman, serif', "48px 'Songti SC'"]
|
||||
];
|
||||
function pixels(lang, text, font) {
|
||||
const canvas = document.createElement('canvas');
|
||||
canvas.lang = lang;
|
||||
canvas.width = 512;
|
||||
canvas.height = 96;
|
||||
document.body.appendChild(canvas);
|
||||
const context = canvas.getContext('2d');
|
||||
context.fillStyle = 'black';
|
||||
context.font = font;
|
||||
context.fillText(text, 8, 64);
|
||||
return context.getImageData(0, 0, canvas.width, canvas.height).data;
|
||||
}
|
||||
function equal(left, right) {
|
||||
if (left.length !== right.length) return false;
|
||||
for (let index = 0; index < left.length; index++) {
|
||||
if (left[index] !== right[index]) return false;
|
||||
}
|
||||
return true;
|
||||
}
|
||||
const results = cases.map(([lang, text, requested, expected]) =>
|
||||
equal(pixels(lang, text, requested), pixels(lang, text, expected))
|
||||
);
|
||||
const passed = results.every(Boolean);
|
||||
document.title = results.map((result, index) => `${index}:${result}`).join(',');
|
||||
document.body.style.background = passed ? 'rgb(31,143,76)' : 'rgb(181,37,46)';
|
||||
} catch (error) {
|
||||
document.title = `${error.name}:${error.message}`;
|
||||
document.body.style.background = 'rgb(181,37,46)';
|
||||
}
|
||||
</script></body></html>"#;
|
||||
format!("data:text/html,{}", percent_encode(html))
|
||||
}
|
||||
|
||||
fn percent_encode(value: &str) -> String {
|
||||
value
|
||||
.bytes()
|
||||
.map(|byte| match byte {
|
||||
b'A'..=b'Z' | b'a'..=b'z' | b'0'..=b'9' | b'-' | b'_' | b'.' | b'~' => {
|
||||
(byte as char).to_string()
|
||||
}
|
||||
_ => format!("%{byte:02X}"),
|
||||
})
|
||||
.collect()
|
||||
}
|
||||
|
||||
fn center_pixel_rgb(frame: &ely_servo_host::RenderedFrame) -> [u8; 3] {
|
||||
let index = (((frame.height() / 2) * frame.width() + frame.width() / 2) * 4) as usize;
|
||||
let rgba = &frame.rgba_bytes()[index..index + 4];
|
||||
[rgba[0], rgba[1], rgba[2]]
|
||||
}
|
||||
@@ -208,7 +208,7 @@ fn live_sidecar_drains_an_oversized_request_line_and_recovers_the_handshake()
|
||||
let mut stdin = child.stdin.take().ok_or_else(|| io::Error::other("missing stdin"))?;
|
||||
let stdout = child.stdout.take().ok_or_else(|| io::Error::other("missing stdout"))?;
|
||||
let mut stdout = BufReader::new(stdout);
|
||||
let mut oversized = br#"{"type":"handshake","protocol_version":3}"#.to_vec();
|
||||
let mut oversized = br#"{"type":"handshake","protocol_version":4}"#.to_vec();
|
||||
oversized.resize(MAX_REQUEST_LINE_BYTES + 4_096, b' ');
|
||||
oversized.push(b'\n');
|
||||
|
||||
|
||||
@@ -24,7 +24,7 @@ use super::pages::{
|
||||
pub(super) const WIDTH: u32 = 360;
|
||||
pub(super) const HEIGHT: u32 = 240;
|
||||
pub(super) const RESPONSE_TIMEOUT: Duration = Duration::from_secs(20);
|
||||
pub(super) const LIVE_PROTOCOL_VERSION: u32 = 3;
|
||||
pub(super) const LIVE_PROTOCOL_VERSION: u32 = 4;
|
||||
pub(super) const MAX_FRAME_DIMENSION: u32 = 16_384;
|
||||
const MAX_FRAME_BYTE_COUNT: usize = 256 * 1024 * 1024;
|
||||
|
||||
@@ -38,6 +38,7 @@ pub(super) fn ensure_request(tab_id: &TabId, profile_id: &ProfileId, url: &str)
|
||||
"height": HEIGHT,
|
||||
"page_zoom_percent": 100,
|
||||
"device_pixel_ratio": 1.0,
|
||||
"color_scheme": "light",
|
||||
"site_permission_generation": 0,
|
||||
"site_permissions": [],
|
||||
})
|
||||
|
||||
@@ -6,12 +6,13 @@ use std::{
|
||||
process::{Command, Stdio},
|
||||
};
|
||||
|
||||
use ely_domain::{ProfileId, SiteOrigin, SitePermissionFeature, TabId, UrlText};
|
||||
use ely_domain::{ColorScheme, ProfileId, SiteOrigin, SitePermissionFeature, TabId, UrlText};
|
||||
use ely_servo_host::{
|
||||
HidpiScaleRequest, KeyboardTextRequest, MouseClickRequest, MouseDragRequest, NavigationRequest,
|
||||
PageZoomRequest, PermissionDecision, PermissionSnapshotEntry, PermissionSnapshotRequest,
|
||||
PermissionSnapshotState, ResizeRequest, ScrollRequest, ServoHost, ServoHostError,
|
||||
ServoSurfaceSize, SoftwareServoHost, TouchTapRequest, WebViewState,
|
||||
ColorSchemeRequest, HidpiScaleRequest, KeyboardTextRequest, MouseClickRequest,
|
||||
MouseDragRequest, NavigationRequest, PageZoomRequest, PermissionDecision,
|
||||
PermissionSnapshotEntry, PermissionSnapshotRequest, PermissionSnapshotState, ResizeRequest,
|
||||
ScrollRequest, ServoHost, ServoHostError, ServoSurfaceSize, SoftwareServoHost, TouchTapRequest,
|
||||
WebViewState,
|
||||
};
|
||||
|
||||
const MINIMUM_CONTENT_PIXELS: u64 = 1_000;
|
||||
@@ -29,6 +30,7 @@ const CLICK_PROBE_URL: &str = "data:text/html,%3C!doctype%20html%3E%3Ctitle%3ECl
|
||||
const DRAG_PROBE_URL: &str = "data:text/html,%3C%21doctype%20html%3E%3Ctitle%3EDrag%20Probe%3C%2Ftitle%3E%3Cstyle%3Ebody%7Bmargin%3A0%3Bbackground%3A%23f6d365%3B%7Dbutton%7Bposition%3Aabsolute%3Bleft%3A80px%3Btop%3A80px%3Bwidth%3A220px%3Bheight%3A90px%3Bfont%3A28px%20sans-serif%3Bbackground%3A%23ffffff%3Bcolor%3A%23111111%3B%7D%3C%2Fstyle%3E%3Cbutton%20id%3Dbox%3EDrag%3C%2Fbutton%3E%3Cscript%3Elet%20dragging%3Dfalse%3Bconst%20box%3Ddocument.getElementById%28%27box%27%29%3BaddEventListener%28%27mousedown%27%2Cevent%3D%3E%7Bif%28event.target%3D%3D%3Dbox%29%7Bdragging%3Dtrue%3B%7D%7D%29%3BaddEventListener%28%27mousemove%27%2Cevent%3D%3E%7Bif%28dragging%26%26event.clientX%3E280%29%7Bdocument.body.style.background%3D%27%230039ff%27%3Bdocument.title%3D%27Dragged%27%3Bbox.textContent%3D%27Dragged%27%3B%7D%7D%29%3BaddEventListener%28%27mouseup%27%2C%28%29%3D%3E%7Bdragging%3Dfalse%3B%7D%29%3B%3C%2Fscript%3E";
|
||||
const TOUCH_PROBE_URL: &str = "data:text/html,%3C%21doctype%20html%3E%3Ctitle%3ETouch%20Probe%3C%2Ftitle%3E%3Cstyle%3Ebody%7Bmargin%3A0%3Bbackground%3A%23c7f5d9%3B%7Dbutton%7Bposition%3Aabsolute%3Bleft%3A80px%3Btop%3A80px%3Bwidth%3A220px%3Bheight%3A90px%3Bfont%3A28px%20sans-serif%3Bbackground%3A%23ffffff%3Bcolor%3A%23111111%3Btouch-action%3Amanipulation%3B%7D%3C%2Fstyle%3E%3Cbutton%20ontouchstart%3D%22document.body.dataset.touch%3D%27start%27%3B%22%20onpointerdown%3D%22if%28%21document.body.dataset.pointerType%29%7Bdocument.body.dataset.pointerType%3Devent.pointerType%3B%7D%22%20onclick%3D%22if%28document.body.dataset.pointerType%21%3D%3D%27touch%27%29%7Bdocument.title%3Ddocument.body.dataset.pointerType%3Breturn%3B%7Ddocument.body.style.background%3D%27%230039ff%27%3Bdocument.title%3D%27Touched%27%3Bthis.textContent%3D%27Touched%27%3B%22%3ETap%3C%2Fbutton%3E";
|
||||
const TEXT_PROBE_URL: &str = "data:text/html,%3C!doctype%20html%3E%3Ctitle%3EText%20Probe%3C%2Ftitle%3E%3Cstyle%3Ebody%7Bmargin%3A0%3Bbackground%3A%23d9e8ff%3Bfont%3A28px%20sans-serif%3B%7Dinput%7Bposition%3Aabsolute%3Bleft%3A80px%3Btop%3A80px%3Bwidth%3A260px%3Bheight%3A70px%3Bfont%3A28px%20sans-serif%3B%7Doutput%7Bposition%3Aabsolute%3Bleft%3A80px%3Btop%3A180px%3Bfont%3A32px%20sans-serif%3B%7D%3C%2Fstyle%3E%3Cinput%20id%3Dq%20autofocus%20oninput%3D%22document.body.style.background%3D%27%230039ff%27%3Bdocument.getElementById%28%27out%27%29.textContent%3Dthis.value%3B%22%3E%3Coutput%20id%3Dout%3Eempty%3C%2Foutput%3E";
|
||||
const THEME_PROBE_URL: &str = "data:text/html,%3C!doctype%20html%3E%3Ctitle%3ETheme%20Probe%3C%2Ftitle%3E%3Cstyle%3Ebody%7Bmargin%3A0%3Bbackground%3A%23f1e2d3%3B%7D%40media%28prefers-color-scheme%3Adark%29%7Bbody%7Bbackground%3A%23112233%3B%7D%7D%3C%2Fstyle%3E";
|
||||
const TEXT_PROBE_VALUE: &str = "ely42";
|
||||
|
||||
struct PrdSiteCompatibilityCase {
|
||||
@@ -235,6 +237,29 @@ fn exercise_real_servo_webview_lifecycle() -> Result<(), Box<dyn Error>> {
|
||||
"mismatch: {mismatch:?}"
|
||||
);
|
||||
|
||||
host.set_color_scheme(ColorSchemeRequest {
|
||||
webview_id: webview_id.clone(),
|
||||
color_scheme: ColorScheme::Dark,
|
||||
})?;
|
||||
host.navigate(NavigationRequest {
|
||||
webview_id: webview_id.clone(),
|
||||
tab_id: tab_id.clone(),
|
||||
url: UrlText::parse(THEME_PROBE_URL)?,
|
||||
})?;
|
||||
wait_for_rendered_webview_with_center_pixel(&mut host, &webview_id, None, [17, 34, 51])?;
|
||||
|
||||
let previous_frame_hash = host.last_rendered_frame()?.sample_hash();
|
||||
host.set_color_scheme(ColorSchemeRequest {
|
||||
webview_id: webview_id.clone(),
|
||||
color_scheme: ColorScheme::Light,
|
||||
})?;
|
||||
wait_for_rendered_webview_with_center_pixel(
|
||||
&mut host,
|
||||
&webview_id,
|
||||
Some(previous_frame_hash),
|
||||
[241, 226, 211],
|
||||
)?;
|
||||
|
||||
let url = UrlText::parse(CLICK_PROBE_URL)?;
|
||||
|
||||
host.navigate(NavigationRequest { webview_id: webview_id.clone(), tab_id, url })?;
|
||||
|
||||
@@ -50,8 +50,9 @@ transfers the IOSurface send right; the JSON port number is diagnostic metadata.
|
||||
|
||||
The protocol supports:
|
||||
|
||||
- `handshake`: verify protocol version `3` before accepting browser commands.
|
||||
- `ensure`: create or update a WebView, navigation, viewport, zoom, permissions, and input.
|
||||
- `handshake`: verify protocol version `4` before accepting browser commands.
|
||||
- `ensure`: create or update a WebView, navigation, viewport, zoom, resolved color scheme,
|
||||
permissions, and input.
|
||||
- `poll`: advance Servo and return pending frame or metadata state.
|
||||
- `close`: destroy one tab's WebView.
|
||||
- `shutdown`: acknowledge graceful process shutdown so Servo flushes profile storage.
|
||||
@@ -108,8 +109,10 @@ The integration gates prove these invariants with real Servo networking:
|
||||
|
||||
## Upstream Servo Route
|
||||
|
||||
ELY pins Servo upstream commit `a87625150758fe91a29e7f6553ff2f798b87027e`, workspace version
|
||||
`0.4.0`, in `Cargo.toml` and `Cargo.lock`.
|
||||
ELY pins fork commit `8f6850ceb0bd92aea4127d8078ebb2c0485e7f39`, based on Servo upstream
|
||||
commit `a87625150758fe91a29e7f6553ff2f798b87027e`, workspace version `0.4.0`, in
|
||||
`Cargo.toml` and `Cargo.lock`. The fork moves macOS CoreText fallback ahead of Servo's static
|
||||
fallback-family list so the first CSS font preserves its serif or sans-serif cascade and language.
|
||||
|
||||
Servo's rendering lifecycle remains authoritative inside each sidecar:
|
||||
|
||||
@@ -128,6 +131,8 @@ Relevant upstream evidence at the pinned revision:
|
||||
- `ports/servoshell/window.rs` repaints with `webview.paint()` and presents the context.
|
||||
- `ports/servoshell/running_app_state.rs` maps `notify_new_frame_ready` to repaint scheduling.
|
||||
- `components/paint/paint.rs` owns one WebRender painter per rendering context.
|
||||
- `components/fonts/font.rs` asks CoreText for a style- and language-compatible macOS fallback
|
||||
before consulting the cross-platform fallback-family list.
|
||||
|
||||
## Ownership
|
||||
|
||||
|
||||
@@ -11,6 +11,13 @@ ely_append_encoded_rustflag() {
|
||||
export CARGO_ENCODED_RUSTFLAGS
|
||||
}
|
||||
|
||||
ely_append_c_family_flag() {
|
||||
local flag="$1"
|
||||
CFLAGS="${CFLAGS:+${CFLAGS} }${flag}"
|
||||
CXXFLAGS="${CXXFLAGS:+${CXXFLAGS} }${flag}"
|
||||
export CFLAGS CXXFLAGS
|
||||
}
|
||||
|
||||
ely_path_variants() {
|
||||
local path="$1"
|
||||
local canonical_path=""
|
||||
@@ -34,6 +41,14 @@ ely_add_release_path_remap() {
|
||||
while IFS= read -r variant; do
|
||||
[[ -n "${variant}" ]] || continue
|
||||
ely_append_encoded_rustflag "--remap-path-prefix=${variant}=${replacement}"
|
||||
case "$(uname -s)" in
|
||||
MINGW*|MSYS*|CYGWIN*)
|
||||
ely_append_c_family_flag "/pathmap:${variant}=${replacement}"
|
||||
;;
|
||||
*)
|
||||
ely_append_c_family_flag "-ffile-prefix-map=${variant}=${replacement}"
|
||||
;;
|
||||
esac
|
||||
done < <(ely_path_variants "${path}")
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user