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

Implement document.domain getter #8655

Merged
merged 1 commit into from Nov 24, 2015
Merged

Implement document.domain getter #8655

merged 1 commit into from Nov 24, 2015

Conversation

@jdm
Copy link
Member

jdm commented Nov 23, 2015

Rebased from #6840 + test fixes.

Review on Reviewable

@jdm
Copy link
Member Author

jdm commented Nov 23, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

📌 Commit 29e594c has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

Testing commit 29e594c with merge 262ec21...

bors-servo added a commit that referenced this pull request Nov 23, 2015
Implement document.domain getter

Rebased from #6840 + test fixes.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8655)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

💔 Test failed - linux-rel

@jdm
Copy link
Member Author

jdm commented Nov 23, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

Testing commit 29e594c with merge dc01b03...

bors-servo added a commit that referenced this pull request Nov 23, 2015
Implement document.domain getter

Rebased from #6840 + test fixes.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8655)
<!-- Reviewable:end -->
@Ms2ger
Copy link
Contributor

Ms2ger commented Nov 23, 2015

Any reason not submit the test upstream?

@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

💔 Test failed - linux-rel

@jdm jdm force-pushed the jdm:domain branch from 29e594c to 0e0e051 Nov 23, 2015
@jdm
Copy link
Member Author

jdm commented Nov 23, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

📌 Commit 0e0e051 has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

Testing commit 0e0e051 with merge 1fc21cf...

bors-servo added a commit that referenced this pull request Nov 23, 2015
Implement document.domain getter

Rebased from #6840 + test fixes.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8655)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 23, 2015

💔 Test failed - linux-rel

@jdm
Copy link
Member Author

jdm commented Nov 23, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Nov 24, 2015

Testing commit 0e0e051 with merge 0905088...

bors-servo added a commit that referenced this pull request Nov 24, 2015
Implement document.domain getter

Rebased from #6840 + test fixes.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8655)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 24, 2015

@bors-servo bors-servo merged commit 0e0e051 into servo:master Nov 24, 2015
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test successful
Details
@bors-servo bors-servo mentioned this pull request Nov 24, 2015
@Ms2ger Ms2ger deleted the jdm:domain branch Nov 24, 2015
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.