kazurayam / chromedriverfactory

A Java library that enables you to launch Selenium ChromeDriver using an existing user Profile. That let you to carry cookies over multiple HTTP sessions via user Profile.
0 stars 0 forks source link

List or Set? #26

Closed kazurayam closed 2 years ago

kazurayam commented 2 years ago

v0.2.7, ChromeDriverFactory has this signature.

abstract void addAllChromePreferencesModifiers(
            List<ChromePreferencesModifier> chromePreferencesModifierList)

Here I use List, but why? I should rather use Set here.

kazurayam commented 2 years ago

I thought about it. I think it should be List rather than Set. List is easier than Set for users to add xxxx. Set is strict in that it rejects adding the same entry into it.