Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove data field from websockets #7880

Merged
merged 1 commit into from Oct 5, 2015
Merged
Changes from all commits
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

remove data field from websockets

The data field is currently not used (no reads/ writes). This commit
removes this field.

Ref.-Issue: #7859
  • Loading branch information
romankl committed Oct 5, 2015
commit 1f82e5121c92cf91935d7445f734dbe66f1621a9
@@ -76,7 +76,6 @@ pub struct WebSocket {
clean_close: Cell<bool>, //Flag to tell if the websocket closed cleanly (not due to full or fail)
code: Cell<u16>, //Closing code
reason: DOMRefCell<DOMString>, //Closing reason
data: DOMRefCell<DOMString>, //Data from send - TODO: Remove after buffer is added.
binary_type: Cell<BinaryType>,
}

@@ -116,7 +115,6 @@ impl WebSocket {
clean_close: Cell::new(true),
code: Cell::new(0),
reason: DOMRefCell::new("".to_owned()),
data: DOMRefCell::new("".to_owned()),
binary_type: Cell::new(BinaryType::Blob),
}

ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.