Mozilla #servo 31 Jan 2020. Servo/servo #25656 turned out a lot more. Servo had two major roles within Mozilla - as an incubator for novel browser technology we wanted to ship in Firefox, and more recently, as a lighter-weight vehicle for Mixed Reality products. The latter has been the focus for the last three years, and those products now appear to be winding down. Mozilla so far has refrained from making concrete plans for Servo - or Rust - to formally succeed Firefox or its use of C/C. For the time being, at least, Firefox remains Mozilla's browser of.
Stylo (a.k.a. QuantumCSS) will integrate Servo's CSS style system into Gecko, such that the style system code can be shared by Gecko and Servo. Stylo V1 will support Firefox on Windows, macOS, and Linux. Android support may ship in a later release. Stylo is a core part of Project Quantum.
To help test Stylo, download Firefox Nightly. Kaiba corp ygopro for mac. To confirm that Stylo is enabled, check about:support for 'Stylo: true (enabled by default)'. If it's not, then set the about:config pref layout.css.servo.enabled to true.
For a deep dive into Stylo internals, check out Lin Clark's Mozilla Hacks blog post 'Inside a super fast CSS engine: Quantum CSS (aka Stylo)'.
- 1Getting Involved
- 3Milestones
- 4Bugs
If you're interested in contributing to Stylo or Servo, drop by the #servo IRC channel on Mozilla's IRC server.
Sap serial number status esto. One front where you can help out is adding new properties to Servo's CSS parsing crate. This requires familiarity with Rust, but is otherwise a good way to dip your toes into Servo's style system. To get started, see Servo's CSS property hacking guide for more information.
If you do not know Rust, you can help implement keyword properties. See the Stylo hacking guide for more details. The Stylo hacking guide also has instructions for building Stylo if you wish to play with it yourself. If you know C++, Firefox bug 1277133 has information about easy Gecko bugs that will assist Stylo's integration with Gecko.
Some Stylo bugs that are good for new Servo contributors are tagged on GitHub as “easy” bugs and “less-easy” bugs.
Committing stylo changes
The process for committing a Stylo change that involves both Gecko and Servo changes is as follows:
- Create a Github PR against https://github.com/servo/servo with the Servo changes in question.
- Get that reviewed and checked into Servo's github repository.
- Wait for servo-vcs-sync to land the Servo changes on Gecko's autoland.
- Land the Gecko changes on autoland.
A more exhaustive / overwhelming description of the workflow is also available.
Try Syntax
For most Stylo changes, it should be sufficient to build on all platforms and test on Linux 64-bit only, which helps save resources in automation. To do this, you can use:
What Are Mozilla Service Workers
To build and test all Stylo platforms running in automation, you can use:
To build all possible platforms to ensure code compiles everywhere, but only test with Stylo enabled, you can use:
Stylo should not cause us to regress any CSS feature unless we explicitly decide that regressing that specific feature is acceptable. The existing test suite will be the primary benchmark of this, with cross-validation from an automated analysis and manual QA of the Alexa Top 500 sites.
- Stylo should be enabled by default on all desktop platforms.
- Stylo should have zero P1 and P2 bugs.
- Stylo should pass all (non-disabled) Gecko reftests and mochitests.
- Stylo should support at least as many CSS properties as than Gecko. Firefox = 375. Stylo = 375. Chrome = 276. DONE!
- Stylo should pass visual diff testing for the Alexa top sites. (bug 1331552) DONE!
- Stylo's sequential mode performance on Talos tp5 and tp6 should be at least as fast as Gecko's.
- Stylo's parallel mode performance on Talos tp5 and tp6 should be faster than Gecko's. Ideally, Stylo should scale linearly with the number of cores.
- Stylo's performance on Speedometer should be at least as fast as Gecko's.
- Stylo's memory usage (as measured by AWSY and user telemetry) should not exceed to Gecko's by more than X% (TBD).
- Stylo's crash rate should be less than or equal to Gecko's.
- Stylo's user engagement metrics should be greater than or equal to Gecko's.
Linux32 | Linux64 | macOS | Win32 | Win64 | Android | |
---|---|---|---|---|---|---|
Build | bug 1366050 | bug 1374824 | bug 1375774 | bug 1384258 | bug 1384258 | bug 1366049 (enabled 2017-11-23) Where is the expiration date on keebler crackers. |
AWSY | not tested | 1% regression | 1% regression | 2% regression | 3% regression | 0% change |
Speedometer | not tested | linux64 Perfherder | macOS Perfherder | win7-32 Perfherder | win10-64 Perfherder | not tested |
StyleBench | not tested | linux64 Perfherder | macOS Perfherder | win7-32 Perfherder | win10-64 Perfherder | not tested |
tp5 | not tested | linux64 | macOS | win7-32 | win10-64 | tp4m (tiny improvement) |
tp6a | not tested | not tested | macOS | win7-32 | win10-64 | not tested |
tp6f | not tested | not tested | macOS | win7-32 | win10-64 | not tested |
tp6g | not tested | not tested | macOS | win7-32 | win10-64 | not tested |
tp6y | not tested | not tested | macOS | win7-32 | win10-64 | not tested |
Autophone | not tested | not tested | not tested | not tested | not tested | remote-blank (tiny improvement) remote-nytimes (tiny regression) remote-tsvg (tiny regression bug 1420369) remote-twitter (tiny regression bug 1420369) |
stylo-central
- Run linux64-stylo tests in mozilla-central (bug 1330414) (dependencies)
- Enable stop-gap vcs sync (bug 1317525)
- Vendor servo in mozilla-central (bug 1322769)
- Make all tests green (by skipping or expecting failure)
stylo-nightly
- Enable Stylo in Nightly (bug 1330412) (dependencies)
- Implement parsing of all CSS properties supported by Firefox
- Fix and enable all tests (without skipping or expecting failure)
- Fix all crashes and assertion failures
- Fuzz Stylo
- Pass [Auto-tools/Projects/BugHunter Bughunter]
- Manual QA sign-off
- Smoke test Alexa Top N sites
- Exploratory testing of browser UI and features
stylo-release
- Stylo meta bug 1243581 (dependencies)
- Run experiment comparing Stylo vs Gecko on engagement ratio, performance, crash rate, and memory usage.
- Let Stylo ride the trains
- Enable Stylo on all platforms (bug stylo-nightly)
- Pass QA on all platforms (tests, fuzzing, BugHunter)
- Enable Stylo on all platforms (bug stylo-nightly)
- Ship Stylo to Release
P1 Bugs
No results.
0 Total;0 Open (0%);0 Resolved (0%);0 Verified (0%);
[stylo:p1] bugs:
No results.
0 Total;0 Open (0%);0 Resolved (0%);0 Verified (0%);
P2 Bugs
ID | Priority | Summary | Keywords | Assigned to |
---|---|---|---|---|
1406622 | P2 | stylo: use invalidation framework to handle restyles needed in response to content insertion/removal | Emilio Cobos Álvarez (:emilio) | |
1420369 | P2 | stylo: 6.21 - 6.54% remote-tsvg / remote-twitter (android-6-0-armv8-api16, android-7-1-armv8-api16) regression on push db56323cd08f4883e4824199b441a3141be655e5 (Thu Nov 23 2017) | perf, regression | Makoto Kato [:m_kato] |
1413654 | P2 | stylo: We rely on the initial ResolveStyleFor call when computing the body style for scroll propagation to deal with updating the stylist | ||
1422615 | P2 | stylo: Stop cascading an element for restyling if the element has display:none | ||
1441389 | P2 | stylo: Regression on tsvg_static on Linux when stylo-chrome is enabled | ||
1426295 | P2 | stylo: SVG images use more memory than Gecko | ||
1418902 | P2 | stylo: Stop updating animation data on pseudo elements whose content value is changed to 'none' | reproducible | |
1430706 | P2 | stylo: geckoservo::glue::Servo_ParseStyleAttribute is slower than Gecko's old CSS parser on ARM | ||
1391341 | P2 | stylo: Font size inherits differently than in Gecko | dev-doc-complete, regression | |
1422416 | P2 | stylo: No need to mark uncacheable for style adjustment based on reset properties only | Xidorn Quan [:xidorn] UTC+11 |
10 Total;10 Open (100%);0 Resolved (0%);0 Verified (0%);
[stylo:p2] bugs:
No results.
0 Total;0 Open (0%);0 Resolved (0%);0 Verified (0%);
P3 Bugs
ID | Priority | Summary | Keywords | Assigned to |
---|---|---|---|---|
1358955 | P3 | stylo: Update SMIL animations that depend on the cascade | Brian Birtles (:birtles) | |
1395488 | P3 | stylo: shrink nsStyleDisplay, nsStylePosition and ServoComputedData | Emilio Cobos Álvarez (:emilio) | |
1388566 | P3 | stylo: Don't flush throttled animations if the root element has normal dirty descendants bit in PreTraverse() | Hiroyuki Ikezoe (:hiro) | |
1413497 | P3 | stylo: tests/editor/reftests/824080-2.html test bug fails on Android/styloVsGecko | Makoto Kato [:m_kato] | |
1436676 | P3 | Remove the telemetry probe for parallel stylo traversals. | ||
1414941 | P3 | stylo: Assertion failure: mAllowZoom aNewData.mAllowZoom (expected mAllowZoom to be the same on both nsStyleFonts) [@ nsStyleFont::CalcDifference] | assertion, testcase | |
1435940 | P3 | stylo: Consider using invalidation for media feature values change | ||
1435949 | P3 | stylo: Consider changing how media_feature_affected_matches works | ||
1411119 | P3 | stylo: Assertion failure: !preTransformOverflows (GetVisualOverflowRect() won't return the pre-effects rect!), at /builds/worker/workspace/build/src/layout/svg/nsSVGIntegrationUtils.cpp:104 | assertion, testcase | |
1440561 | P3 | stylo: Figure out a better way to allow sync load of font when possible | ||
1446971 | P3 | A couple reftests for bug 883987 fail on stylo, but look invalid. | ||
1408323 | P3 | stylo: consider generating nsStyleConsts.h values from Servo | ||
1415236 | P3 | enable-stylo build fails probably when system-nspr is used | ||
1428036 | P3 | stylo: Add the CSS string which is parsed to crash report annotations | ||
1376082 | P3 | stylo: The hover generation is never updated, and we seem to be fine with it. | ||
1400915 | P3 | stylo: Still have a lot of Display, Position, Border structs on the HTML5 page even with STYLO_THREADS=1 | ||
1412441 | P3 | stylo: Merge structs.rs into bindings.rs | ||
1437785 | P3 | stylo: Author style disabling doesn't stop using @page, @font-face, @counter-style, etc. rules from author sheets | ||
1423964 | P3 | stylo-chrome: Gecko_GetDocumentLWTheme and ThreadSafeGetDocumentLWTheme show up in profile | ||
1408351 | P3 | Stylo: Crash in style::invalidation::stylesheets::StylesheetInvalidationSet::scan_component | crash, nightly-community, regression | |
1500512 | P3 | Cleanup stylo handling in AWSY | ||
1413731 | P3 | stylo: Investigate whether opaquing types more aggressively can help reduce build time | ||
1402212 | P3 | stylo: display value of non-existent pseudo flex item is not blockified | ||
1417578 | P3 | stylo: Audit internal properties that are accidentally exposed. | ||
1616236 | P3 | 'tp5n time_to_session_store_window_restored_ms opt e10s stylo' is too variable to be useful in a single 5-run | ||
1424253 | P3 | stylo: non-default 'font.minimum-size.' pref value breaks table layout | ||
1440640 | P3 | stylo: The search suggestions hint is not vertically centred with Dark/Light themes on Linux after dynamically switching theme | ||
1397380 | P3 | stylo: Investigate sources of stylo memory overhead on the html single-page spec | ||
1420423 | P3 | [meta] stylo: Large regression on some talos tests when enabling stylo-chrome | meta | |
1560981 | P3 | use the bindgen executable for stylo rather than building bindgen at build time | ||
1345688 | P3 | stylo: align binary representations of computed keyword values and eliminate the match statements | ||
1413533 | P3 | stylo: Consider making the bloom filter handle :-moz-any and such. | ||
1412774 | P3 | stylo: 1133905-4-v-rtl.html is always failure on Android/styloVsGecko | ||
1419943 | P3 | stylo-chrome: permafailure browser_aboutURLs.js | leaked 2 window(s) until shutdown [url = about:downloads] | ||
1440180 | P3 | stylo: Some computed values are not reported in memory report | ||
1445566 | P3 | stylo-chrome: <tab> element and its descendants are styled twice for each new tab | ||
1400510 | P3 | stylo: Add refcount logging for servo_arc so we can detect reference cycles that cause leaks | ||
1405633 | P3 | stylo: Run other stylo tests in rusttests tasks | ||
1400771 | P3 | stylo: preshints (mapped attributes) not handled correctly for elements created from non-docshell-associated document | ||
1416903 | P3 | stylo: OOM crash in core::result::unwrap_failed<T> | style::selector_map::MaybeCaseInsensitiveHashMap<T>::entry<T> | crash | |
1440222 | P3 | stylo: Computed values for anonymous content of nsCanvasFrame are not reported in memory report | ||
1427679 | P3 | stylo: Drop some tables in stylist for SVG-as-image when no SMIL presents | ||
1474789 | P3 | Stylo accesses the preference service from multiple threads. | ||
1370604 | P3 | stylo: Consider preloading the style sharing cache with styled siblings during incremental restyle | ||
1401074 | P3 | stylo: handle modifications to UA sheets by invalidating UA CascadeData cache | ||
1414902 | P3 | stylo: Assertion failure: !GetStyleContextInMap(aMap, aContent) (Already have an entry for aContent), at /builds/worker/workspace/build/src/layout/base/nsFrameManager.cpp:205 | assertion, testcase | |
1375222 | P3 | Stylo: Lots of 48-byte stack locals being copied in style::properties::PropertyDeclaration::parse_into | ||
1426681 | P3 | stylo: Crash in style::values::specified::image::{{impl}}::parse | crash, regression | |
1375225 | P3 | Stylo: Redundant object copies in style::properties::PropertyDeclaration::parse_into | ||
1383868 | P3 | stylo: Incorrect blockification of kids of display:inline-flex element that is not a flex container | Xidorn Quan [:xidorn] UTC+11 |
50 Total;50 Open (100%);0 Resolved (0%);0 Verified (0%);
[stylo:p3] bugs:
No results.
0 Total;0 Open (0%);0 Resolved (0%);0 Verified (0%);
Bugzilla Queries
- Stylo milestone bug trees:
- Build Stylo in Nightly (but pref'd off)
- Unprioritized 'stylo' whiteboard bugs to be triaged (Chris's list)
- Crash queries:
- Skipped or asserting tests: (these do not necessarily indicate Stylo bugs!)
Rust Web Browser
Priority Definitions:
- P1: Blocks shipping, more urgent. We should be working on these now/ASAP.
- P2: Blocks shipping, less urgent. We can wait a few weeks to start working on these.
- P3: Might block shipping. Needs re-evaluation, potentially closer to the deadline.
- P4: Doesn't block shipping, but nice to have.
- P5: Don't need to track.