:Gijs (he/him) Assignee
	Description • 6 years ago

This is a footgun for people trying to just modify this list, and it shouldn't be necessary to manually track the length of the list.

	Andreas Tolfsen ❲:ato❳
	Comment 2 • 6 years ago
mozreview-review

Comment on attachment 8986728 [details]
Bug 1470100 - use a vector instead of an array for the list of prefs,

https://reviewboard.mozilla.org/r/252024/#review258442

Thanks for the patch!  I didn’t expect you to fix this, but much
appreciated.

	Mozilla Autoland
	Comment 3 • 6 years ago

We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again.

hg error in cmd: hg rebase -s ea823a7acea1d8ec9019a518f9aca5d1ec6650aa -d e95567b7138d: rebasing 469372:ea823a7acea1 "Bug 1470100 - use a vector instead of an array for the list of prefs, r=ato" (tip)
merging testing/geckodriver/src/prefs.rs
warning: conflicts while merging testing/geckodriver/src/prefs.rs! (edit, then use 'hg resolve --mark')
unresolved conflicts (see hg resolve, then hg rebase --continue)

	:Gijs (he/him) Assignee
	Comment 5 • 6 years ago

Comment on attachment 8986728 [details]
Bug 1470100 - use a vector instead of an array for the list of prefs,

Ugh, repushing this after rebasing was a bad idea...

	Pulsebot
	Comment 6 • 6 years ago

Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/fec9f43850d3
use a vector instead of an array for the list of prefs, r=ato

	Eliza Balazs [:ebalazs_]
	Comment 7 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/fec9f43850d3