nit comment on style, `base::StringValue* ` is chromium style, please try to follow.