in _send, should Runtime.evaluate take "returnByValue": False?

[cjrh]

This is perhaps more of a question than an issue, but in my project, I need to add "returnByValue": False, at around line 72 in __init__.py:

(note, I'm adding line breaks only to make it easier to read)

def evaluate(self, javascript):
    """
    Evaluate JavaScript on the page
    """
    return self._send(
        {"method": "Runtime.evaluate", 
         "params": {"expression": javascript, 
            "returnByValue": False}})        # <---- This bit

Without this, I don't get back the full string contents of returned data, e.g. the results of Array maps and so on. I'm happy to make a PR to make this a setting (e.g. an extra param on evaluate, with default set to True), but just wanted to find out your thoughts?

[cjrh]

No, that wasn't it. There's something else going on. Sorry!