Bug 1580453 Comment 1 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

ous binary blob on a key not recognised as configuration by Marionette,
this arbitrary JSON structure gets passed around internally at great
expense (since it is copied to every new content browser with
`initialProcessData`.)

I have a suspicion the function is a leftover from the days before
capabilities negotiation was done in geckodriver. Since it doesn’t
do any matching it would be prudent to rename/rewrite it to the
effect it’s clear it only deals with picking up configuration passed
by geckodriver.
I should also mention that the way `Capabilities._match()` is
written, if a client provides an enormous binary blob on a key not
recognised as configuration by Marionette, this arbitrary JSON
structure gets passed around internally at great expense (since it
is copied to every new content browser with `initialProcessData`.)

I have a suspicion the function is a leftover from the days before
capabilities negotiation was done in geckodriver. Since it doesn’t
do any matching it would be prudent to rename/rewrite it to the
effect it’s clear it only deals with picking up configuration passed
by geckodriver.

(__Edit__: Part of the comment went mysteriously missing.)

Back to Bug 1580453 Comment 1