hudelgado added the comment:
I took a look at it and have a some more questions.
The class MacOSX(BaseBrowser) seems to be broken because it uses 'OpenUrl'
which doesn't work in most recent versions of applescript, also that class
doesn't seem to be used anywhere, is it
hudelgado added the comment:
Ok, I'm thinking in also replace the os.popen calls with subprocess.Popen to
keep consistency with existing tests.
Do you think it's a good approach? Or should be better to just add some tests
around the current impl
