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

Autogenerate list of prototypes and code to register DOM bindings #577

Merged
merged 1 commit into from Jul 12, 2013

Conversation

@jdm
Copy link
Member

jdm commented Jul 11, 2013

No description provided.

@jdm
Copy link
Member Author

jdm commented Jul 11, 2013

This is on top of #573, so only the last commit is new.

@@ -429,17 +429,19 @@ def resolve(self, parentScope):
pass

This comment has been minimized.

@pcwalton

pcwalton Jul 11, 2013

Contributor

Are these upstream changes?

This comment has been minimized.

@jdm

jdm Jul 11, 2013

Author Member

Yes.

}

impl MouseEvent {
pub fn new(type_: DOMString, can_bubble: bool, cancelable: bool,

This comment has been minimized.

@pcwalton

pcwalton Jul 11, 2013

Contributor

nit, usually I line these up under the (, and I'm not a big fan of type_ (prefer ty), but I won't block the review on this.

bors-servo pushed a commit that referenced this pull request Jul 12, 2013
bors-servo pushed a commit that referenced this pull request Jul 12, 2013
@jdm

This comment has been minimized.

Copy link
Owner Author

jdm commented on 4f24ef1 Jul 12, 2013

r=metajack

This comment has been minimized.

Copy link
Owner Author

jdm replied Jul 12, 2013

@bors: retry

@bors-servo

This comment has been minimized.

Copy link
Contributor

bors-servo commented on 4f24ef1 Jul 12, 2013

saw approval from jdm, metajack
at jdm@4f24ef1

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jul 12, 2013

merging jdm/servo/autogen = 4f24ef1 into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jul 12, 2013

jdm/servo/autogen = 4f24ef1 merged ok, testing candidate = 1d2f684

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jul 12, 2013

saw approval from jdm, metajack
at jdm@4f24ef1

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jul 12, 2013

merging jdm/servo/autogen = 4f24ef1 into auto

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jul 12, 2013

jdm/servo/autogen = 4f24ef1 merged ok, testing candidate = 821192d

This comment has been minimized.

Copy link
Contributor

bors-servo replied Jul 12, 2013

fast-forwarding master to auto = 821192d

@bors-servo bors-servo closed this Jul 12, 2013
@bors-servo bors-servo merged commit 4f24ef1 into servo:master Jul 12, 2013
1 check passed
1 check passed
default all tests passed
@jdm jdm deleted the jdm:autogen branch Aug 4, 2015
glennw pushed a commit to glennw/servo that referenced this pull request Jan 16, 2017
Fix replay crash on startup

<!-- 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/577)
<!-- Reviewable:end -->
@jdm jdm mentioned this pull request Nov 5, 2019
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

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