Currently Success takes a pointer, since the return value is optional. However, that makes the case where there is a return value annoying if it's not a complex EncodableValue that was already constructed, since you have to write things like:

EncodableValue response(true);
result->Success(&response);

We should add an overload that would allow for the more natural

result->Success(EncodableValue(true));