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

Fix binary recording and replay. #1252

Merged
merged 1 commit into from May 15, 2017
Merged

Fix binary recording and replay. #1252

merged 1 commit into from May 15, 2017

Conversation

@glennw
Copy link
Member

glennw commented May 15, 2017

  • Add SetWindowParameters to the list of messages that get serialized.
  • Ensure we wait for the three initial messages in correct order to detect the first frame.

This change is Reviewable

* Add SetWindowParameters to the list of messages that get serialized.
* Ensure we wait for the three initial messages in correct order to detect the first frame.
@glennw
Copy link
Member Author

glennw commented May 15, 2017

r? @kvark or anyone really

@jrmuizel
Copy link
Contributor

jrmuizel commented May 15, 2017

This gets broken often enough that it's probably worth adding some kind of testing...

@glennw
Copy link
Member Author

glennw commented May 15, 2017

@jrmuizel Agreed - opened #1253

@kvark
Copy link
Member

kvark commented May 15, 2017

Thank you!
@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented May 15, 2017

📌 Commit 9d34a91 has been approved by kvark

@bors-servo
Copy link
Contributor

bors-servo commented May 15, 2017

Testing commit 9d34a91 with merge 3db193c...

bors-servo added a commit that referenced this pull request May 15, 2017
Fix binary recording and replay.

* Add SetWindowParameters to the list of messages that get serialized.
* Ensure we wait for the three initial messages in correct order to detect the first frame.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/1252)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented May 15, 2017

☀️ Test successful - status-travis
Approved by: kvark
Pushing 3db193c to master...

@bors-servo bors-servo merged commit 9d34a91 into servo:master May 15, 2017
3 checks passed
3 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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