nit comment, I'd prefer to `void* data`, which is chromium coding style. If you agree, please apply all similar places in this PR.