autowebcompat icon indicating copy to clipboard operation
autowebcompat copied to clipboard

Published 20 hours ago • verified publisher icon marco-c

Add other browsers

Open marco-c opened this issue 7 years ago • 10 comments

The more browsers we have, the more screenshots we can collect

marco-c avatar Apr 13 '18 18:04 marco-c

@marco-c i had a couple of doubts regarding this. Is this the same as adding a webdriver for a particular browser and then updating collect.py so that this driver is also run alongside chrome and firefox ?

Apart from that , after we do add a browser won't we have to update data_inconsitencies.py and some functions in utils.py to accommodate these new set of images?

Shashi456 avatar Apr 14 '18 18:04 Shashi456

Yes, we will have to add a new webdriver and use it in collect.py and update all the places where we were just expecting Firefox and Chrome.

marco-c avatar Apr 14 '18 18:04 marco-c

So should we add a task list in this Issue ?

Shashi456 avatar Apr 14 '18 18:04 Shashi456

@marco-c having taken a look into it, I've found that safari and edge both are dependent on their respective os and cannot be run cross-platform. Should we still pursue adding browsers like opera? or should we add safari and edge driver irrespective of the there not being cross-platform support.

Shashi456 avatar Apr 16 '18 18:04 Shashi456

@marco-c having taken a look into it, I've found that safari and edge both are dependent on their respective os and cannot be run cross-platform. Should we still pursue adding browsers like opera? or should we add safari and edge driver irrespective of the there not being cross-platform support.

Opera uses Chromium, so there will be no difference between Opera and Chrome. Yes, I think we should try to add Safari and Edge when we are running the crawler on a supported OS.

marco-c avatar Apr 18 '18 23:04 marco-c

safari is available for windows. So windows supports edge and safari both officially. We can also use wine if we want to use it in linux imo.

Vedarth avatar Apr 19 '18 02:04 Vedarth

Is it still available? I thought they stopped supporting Windows.

marco-c avatar Apr 19 '18 08:04 marco-c

I've tried to run Safari through selenium on Windows , they use an extension which is only compatible with Safari installed on a MacOs.

Shashi456 avatar Apr 19 '18 08:04 Shashi456

Although selenium supports safaridriver in windows, Apple has dropped support :( so It might not work properly. Sorry for the confusion. I think checking the os and choosing edge or safari based on that should be the way to go.

Vedarth avatar Apr 19 '18 08:04 Vedarth

Probably not so valuable to add Edge after all: https://blog.mozilla.org/blog/2018/12/06/goodbye-edge/ :cry:

But maybe we can use it anyway so we have more diverse screenshots and we're more likely to have incompatibilities in our training set.

marco-c avatar Dec 07 '18 16:12 marco-c