Skip to content
This repository has been archived by the owner. It is now read-only.

Add support for creating similar draw targets and querying the current format and transform #112

Merged
merged 2 commits into from Nov 26, 2014

Conversation

@pcwalton
Copy link
Contributor

pcwalton commented Nov 8, 2014

More stuff needed for box-shadow.

r? @metajack

@SimonSapin
Copy link
Member

SimonSapin commented Nov 19, 2014

Looks good to me, except for the different kinds of C++ cast that I’d like someone else to have a look at.

@pcwalton pcwalton force-pushed the pcwalton:similar-draw-targets branch from 675dab6 to b92d24a Nov 19, 2014
@tbu-
Copy link

tbu- commented Nov 26, 2014

@SimonSapin
static_cast is like Rust's as: 5i32 as i64.
const_cast can be used to cast away a const from a type.
reinterpret_cast is like Rust's transmute.

@SimonSapin
Copy link
Member

SimonSapin commented Nov 26, 2014

@tbu-, do their usage in this PR look good to you?

@tbu-
Copy link

tbu- commented Nov 26, 2014

They look right, but unfortunately I don't know the interna of these types so I can't tell for sure.

(I don't know whether the function that is passed the const_casted object does not modify the object, neither do I know whether the reinterpret_casted things have the same representation. The static_casts should be fine.)

@metajack

This comment has been minimized.

Copy link

metajack commented on b92d24a Nov 26, 2014

r+

metajack added a commit that referenced this pull request Nov 26, 2014
Add support for creating similar draw targets and querying the current format and transform
@metajack metajack merged commit d323c3c into servo:master Nov 26, 2014
bors-servo pushed a commit to servo/servo that referenced this pull request Dec 3, 2014
This adds the infrastructure necessary to support stacking contexts that
are not containing blocks for absolutely-positioned elements. Our
infrastructure did not support that before. This minor revamp actually
ended up simplifying the logic around display list building and
stacking-relative position computation for absolutely-positioned flows,
which was nice.

This will need this PR: servo/rust-azure#112 I have not updated the Cargo.lock file yet because I want the merge commit.

r? @glennw
f? @SimonSapin
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Sep 30, 2019
…(from pcwalton:opacity); r=SimonSapin

This adds the infrastructure necessary to support stacking contexts that
are not containing blocks for absolutely-positioned elements. Our
infrastructure did not support that before. This minor revamp actually
ended up simplifying the logic around display list building and
stacking-relative position computation for absolutely-positioned flows,
which was nice.

This will need this PR: servo/rust-azure#112 I have not updated the Cargo.lock file yet because I want the merge commit.

r? glennw
f? SimonSapin

Source-Repo: https://github.com/servo/servo
Source-Revision: 68c90e27970808bddcb8c8a4e782bd4405e67a5c

UltraBlame original commit: 9fd9309fcce40850db268814c7cd8886ae472dfc
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 1, 2019
…(from pcwalton:opacity); r=SimonSapin

This adds the infrastructure necessary to support stacking contexts that
are not containing blocks for absolutely-positioned elements. Our
infrastructure did not support that before. This minor revamp actually
ended up simplifying the logic around display list building and
stacking-relative position computation for absolutely-positioned flows,
which was nice.

This will need this PR: servo/rust-azure#112 I have not updated the Cargo.lock file yet because I want the merge commit.

r? glennw
f? SimonSapin

Source-Repo: https://github.com/servo/servo
Source-Revision: 68c90e27970808bddcb8c8a4e782bd4405e67a5c

UltraBlame original commit: 9fd9309fcce40850db268814c7cd8886ae472dfc
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 1, 2019
…(from pcwalton:opacity); r=SimonSapin

This adds the infrastructure necessary to support stacking contexts that
are not containing blocks for absolutely-positioned elements. Our
infrastructure did not support that before. This minor revamp actually
ended up simplifying the logic around display list building and
stacking-relative position computation for absolutely-positioned flows,
which was nice.

This will need this PR: servo/rust-azure#112 I have not updated the Cargo.lock file yet because I want the merge commit.

r? glennw
f? SimonSapin

Source-Repo: https://github.com/servo/servo
Source-Revision: 68c90e27970808bddcb8c8a4e782bd4405e67a5c

UltraBlame original commit: 9fd9309fcce40850db268814c7cd8886ae472dfc
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.