Skip to content
This repository has been archived by the owner on May 30, 2023. It is now read-only.

Ghostdriver Development Branch: pull request #2 #293

Closed
wants to merge 7 commits into from
Closed

Ghostdriver Development Branch: pull request #2 #293

wants to merge 7 commits into from

Conversation

detro
Copy link
Collaborator

@detro detro commented Jul 12, 2012

Hi,

this pull request covers 2 Issues that are "dear" to the GhostDriver project (it can't procede without this actually).

Also, 2/3 minor commits to fix little details around our code base (tiny but annoying stuff).

As usual, everything comes with tests.

Doesn't need a reference to the Config object (for now).
This addresses [Issue #151](http://code.google.com/p/phantomjs/issues/detail?id=151).

Summary of the new API:
- page.pages[]
- page.pagesWindowName[]
- page.getPage(windowName)
- page.windowName
- page.onPageCreated = function(newPage) { ... }

The page object created by the user holds responsibility of the "child" pages it creates.
If a page closes (i.e. window.close()) or a call to "page.pages[i].release()" is done,
the array "page.pages[]" will automatically update to contain only the pages still open.
@ariya
Copy link
Owner

ariya commented Jul 13, 2012

Half-way testing and merged.

@detro
Copy link
Collaborator Author

detro commented Jul 13, 2012

Why half way? ;)


Ivan De Marino
Coder, Cook, Cyclist, Gamer

Sent while standing on one leg

On 13 Jul 2012, at 18:00, Ariya Hidayat
reply@reply.github.com
wrote:

Half-way testing and merged.


Reply to this email directly or view it on GitHub:
#293 (comment)

@ariya
Copy link
Owner

ariya commented Jul 13, 2012

Because time is not always in your favor.

@detro
Copy link
Collaborator Author

detro commented Jul 13, 2012

Oh, I didn't mean in "that" way.
Take your time: I was just wondering why you would merge those changes
gradually instead of one shot.
Do you do some specific testing that I'm not aware of?
Can I do something to make it easier on you when merging/reviewing?


Ivan De Marino
Coder, Cook, Cyclist, Gamer

Sent while standing on one leg

On 13 Jul 2012, at 23:53, Ariya Hidayat
reply@reply.github.com
wrote:

Because time is not always in your favor.


Reply to this email directly or view it on GitHub:
#293 (comment)

@ariya
Copy link
Owner

ariya commented Jul 14, 2012

Nothing particular, I just had to stop midway due to some other commitment.

@ariya
Copy link
Owner

ariya commented Jul 15, 2012

Merged. Thanks!

@ariya ariya closed this Jul 15, 2012
detro added a commit that referenced this pull request Jan 5, 2014
CHANGELOG for v1.1.0 (https://github.com/detro/ghostdriver/issues?labels=1.1.0&state=closed)

JavaScript Driver (Core)
* ENHANCEMENT: `/maximize` window will set the window size to 1336x768,
currently most common resolution online (see http://gs.statcounter.com/#resolution-ww-monthly-201307-201312)
* ENHANCEMENT #275: Implemented Browser and Network (HAR) Logging types
* FIXED #284: Attempt to wait for Page to Load if input causes form submit
* FIXED #291: Throw exception when attempting to set invalid timeout value
* FIXED #259: Fix issue regarding mouse clicks
* ENHANCEMENT #290: Enabled support for "Keep Alive" HTTP connections
* ENHANCEMENT #262: Allow access to PhantomJS API from WebDriver (Driver part)
* ENHANCEMENT #293: Import Selenium 2.39.0 WebDriver Atoms

Java Binding
* MINOR #251: Minor compilation issues for Binding
* ENHANCEMENT #262: Allow access to PhantomJS API from WebDriver (Java Binding part)

Tested using GhostDriver validation tests (https://github.com/detro/ghostdriver/tree/master/test).

#11877
detro added a commit that referenced this pull request Jan 19, 2014
CHANGELOG for v1.1.0 (https://github.com/detro/ghostdriver/issues?labels=1.1.0&state=closed)

JavaScript Driver (Core)
* ENHANCEMENT: `/maximize` window will set the window size to 1336x768,
currently most common resolution online (see http://gs.statcounter.com/#resolution-ww-monthly-201307-201312)
* ENHANCEMENT #275: Implemented Browser and Network (HAR) Logging types
* FIXED #284: Attempt to wait for Page to Load if input causes form submit
* FIXED #291: Throw exception when attempting to set invalid timeout value
* FIXED #259: Fix issue regarding mouse clicks
* ENHANCEMENT #290: Enabled support for "Keep Alive" HTTP connections
* ENHANCEMENT #262: Allow access to PhantomJS API from WebDriver (Driver part)
* ENHANCEMENT #293: Import Selenium 2.39.0 WebDriver Atoms

Java Binding
* MINOR #251: Minor compilation issues for Binding
* ENHANCEMENT #262: Allow access to PhantomJS API from WebDriver (Java Binding part)

Tested using GhostDriver validation tests (https://github.com/detro/ghostdriver/tree/master/test).

#11877
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants