| Commit message (Collapse) | Author | Age | Files | Lines |
|\ |
|
| | |
|
| |
| |
| |
| | |
They were used for Root members, but those no longer use lifetimes.
|
|\ \
| | |
| | |
| | | |
This actually fixes 5159.
|
| | |
| | |
| | |
| | | |
This actually fixes 5159
|
|\ \ \
| |/ /
|/| |
| | |
| | |
| | | |
Fixes #5158
Part of this involved switching to OsStrings for process arguments. Those now interface awkwardly with some remaining use old_io, but that will only be needed until rustc is newer.
|
|/ / |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Table layout code has been refactored to push the spacing down to
rowgroups and rows; this will aid the implementation of
`border-collapse` as well.
r? @SimonSapin
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
`cellspacing` attribute per HTML5 § 14.3.9.
Table layout code has been refactored to push the spacing down to
rowgroups and rows; this will aid the implementation of
`border-collapse` as well.
This commit also fixes two nasty issues in table layout:
* In fixed layout, extra space would not be divided among columns that
had auto width but had nonzero minimum width.
* In automatic layout, extra space would be distributed to constrained
columns as well even if unconstrained columns with percentage equal to
zero were present.
|
|\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
This is GIF specific. It's also done when the image is PNG but PNG is
handled separately with the PNG crate, whereas GIFs are handled by the
stb-image crate and the distinction between alpha and non-alpha-supporting
images was missing.
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
This is GIF specific. It's also done when the image is PNG but PNG is
handled separately with the PNG crate, whereas GIFs are handled by the
stb-image crate and the distinction between alpha and non-alpha-supporting
images was missing.
|
|\ \ \
| | | |
| | | |
| | | | |
Fix for issue #2108. That issue mentions an `unwrap_object` which doesn't seem to exist so I renamed `unwrap` to `native_from_reflector` and `unwrap_jsmanaged` to `native_from_reflector_jsmanaged`. The latter is a bit unweildy - maybe a shorter name might be better?
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As noted by @bholley. "unwrap" is confusing because we are
both stripping off wrappers *and* getting a native from a
reflector. Changing the "unwrap" usage to "native_from_reflector"
for clarity.
This renames 'unwrap' to 'native_from_reflector' and
'unwrap_jsmanaged' to 'native_from_reflector_jsmanaged'.
|
|\ \ \ \
| |/ / /
|/| | | |
|
|/ / / |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | | |
This is a cleanup for issue #3959
Do I need to inline force_relayout?
|
|/ / / |
|
|\ \ \
| |/ /
|/| |
| | | |
This will be re-introduced in a follow up PR with a different usage, but I'm trying to create small, independent PRs that are easier to review than one large change.
|
| | | |
|
|/ / |
|
|\ \
| | |
| | |
| | | |
This doesn't have any effect on functionality, it just simplifies a few upcoming changes with how FrameIds work.
|
| |/
| |
| |
| | |
This doesn't have any effect on functionality, it just simplifies a few upcoming changes with how FrameIds work.
|
|\ \
| |/
|/|
| | |
This fixes some test failures that begin occurring when other events (such as resize) are fixed to only occur when needed.
|
|/
|
|
| |
This fixes some test failures that begin occurring when other events (such as resize) are fixed to only occur when needed.
|
|\
| |
| |
| |
| | |
This patch enables the use of `arc()` on the canvas.
I couldn't add reftest this time, as it involves some antialiasing issues, and so the reference doesn't match.
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | | |
Opening this PR to invite feedback.
Of the many `match` statement candidates for conversion to `if let`, several included `if` guards. Since `if let` doesn't support this syntax, I used nested if statements. If this is undesirable, say the word and I can revert those cases to `match`.
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Only simple alphabetic and numeric counter styles are supported. (This
is most of them though.)
Although this PR adds a sequential pass to layout, I verified that on
pages that contain a reasonable number of ordered lists (Reddit
`/r/rust`), the time spent in generated content resolution is dwarfed by
the time spent in the parallelizable parts of layout. So I don't expect
this to negatively affect our parallelism expect perhaps in pathological
cases.
Reconstructed from #5138 via raw diffing.
r? @SimonSapin
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
§ 12.3-12.5.
Only simple alphabetic and numeric counter styles are supported. (This
is most of them though.)
Although this PR adds a sequential pass to layout, I verified that on
pages that contain a reasonable number of ordered lists (Reddit
`/r/rust`), the time spent in generated content resolution is dwarfed by
the time spent in the parallelizable parts of layout. So I don't expect
this to negatively affect our parallelism expect perhaps in pathological
cases.
|
|\ \ \ \
| |/ / /
|/| | |
| | | | |
...via introduction of Worker::WorkerErrorHandler (Closes #5171).
|
| | | |
| | | |
| | | |
| | | | |
introduction of Worker::WorkerErrorHandler (Closes #5171).
|
|\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
...and vice-versa. This is not a complete fix for all mixed-direction layout cases, but it fixes enough problems to make some simple test cases pass, like tha attached reftest.
There are FIXME comments for many of the remaining issues. In particular, this does not yet handle RTL layout of fixed/absolute elements, nor does it completely fix the overconstrained_block reftest.
r? @SimonSapin or @pcwalton
|
| | | |
| | | |
| | | |
| | | |
| | | | |
This is necessary for correctly converting `relative_containing_block_size`
to physical coordinates.
|
|/ / /
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
...and vice-versa. This is not a complete fix for all mixed-direction layout
cases, but it fixes enough problems to make some simple test cases pass, like
tha attached reftest.
There are FIXME comments for many of the remaining issues. In particular,
this does not yet handle RTL layout of fixed/absolute elements.
|
|\ \ \
| |/ /
|/| | |
|
|/ / |
|
|\ \
| |/
|/| |
remove the trailing slash for consistence.
|
|/ |
|
|\
| |
| | |
Ignore /.cargo/
|
|/ |
|
|\
| |
| |
| |
| | |
Fix warnings and introduce usage of `std::env` over deprecated `std::os`
functions.
|
| |
| |
| |
| |
| | |
Fix warnings and introduce usage of `std::env` over deprecated `std::os`
functions.
|
|\ \
| |/
|/| |
|
| | |
|
|\ \
| | |
| | |
| | | |
This just requires updating glutin and cocoa to pick up some backported fixes.
|
| | |
| | |
| | |
| | | |
This just requires updating glutin and cocoa to pick up some backported fixes.
|