diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-11-03 21:37:17 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-11-03 21:37:17 -0500 |
commit | 7b803860470029a6deadb4d50fc401ddce968f6c (patch) | |
tree | bbd40f4f5039942bb57965ef6172c17f680441a0 /components/script/dom/request.rs | |
parent | 97e205c8ef01545ad8abcf7ac2efa9e53b846847 (diff) | |
parent | 4d14f1eb67798ec13e7a8f45c5c59a3cc8433423 (diff) | |
download | servo-7b803860470029a6deadb4d50fc401ddce968f6c.tar.gz servo-7b803860470029a6deadb4d50fc401ddce968f6c.zip |
Auto merge of #14048 - antrik:nosync-ipc_sender, r=mbrubeck
layout/context: Wrap `image_cache_thread` in a `Mutex<>` as well
`SharedLayoutContext.image_cache_thread` didn't get the `Mutex<>`
treatment along with all the other channels in there, because
`ipc-channel::Sender` is presently inherently `Sync`. I believe this to
be an implementation accident though, that should be rectified in the
future -- not something users should actually rely on...
Note that sharing senders (be it `mpsc` or `ipc-channel`) in is probably
not a good idea anyway: just cloning them -- and letting them handle the
sharing internally -- should be both simpler and cheaper. But right now
that's how things are handled here; so let's go with the flow...
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/14048)
<!-- Reviewable:end -->
Diffstat (limited to 'components/script/dom/request.rs')
0 files changed, 0 insertions, 0 deletions