{"comment":{"url":"https://api.github.com/repos/jirkadanek/googleapis_auth/comments/9605710","html_url":"https://github.com/jirkadanek/googleapis_auth/commit/8cc0967c9719a09c11b931202d97f5df2df8113c#commitcomment-9605710","id":9605710,"user":{"login":"sgjesse","id":1343914,"avatar_url":"https://avatars.githubusercontent.com/u/1343914?v=3","gravatar_id":"","url":"https://api.github.com/users/sgjesse","html_url":"https://github.com/sgjesse","followers_url":"https://api.github.com/users/sgjesse/followers","following_url":"https://api.github.com/users/sgjesse/following{/other_user}","gists_url":"https://api.github.com/users/sgjesse/gists{/gist_id}","starred_url":"https://api.github.com/users/sgjesse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sgjesse/subscriptions","organizations_url":"https://api.github.com/users/sgjesse/orgs","repos_url":"https://api.github.com/users/sgjesse/repos","events_url":"https://api.github.com/users/sgjesse/events{/privacy}","received_events_url":"https://api.github.com/users/sgjesse/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"8cc0967c9719a09c11b931202d97f5df2df8113c","created_at":"2015-02-05T15:47:00Z","updated_at":"2015-02-05T15:47:00Z","body":"Thank you for the patch.\r\n\r\nI have a number of comments:\r\n\r\n1. The argument name `hostedDomain` seems a bit long, and non-obvious to me. How about just `domain`?\r\n\r\n2. The updated methods in `lib/auth_browser.dart` needs updated dartdoc explaining the added argument.\r\n\r\n3. Can the `hd` argument be a list, or only one domain?\r\n\r\n4. The `hd` parameter also need to be added [here](https://github.com/dart-lang/googleapis_auth/blob/master/lib/src/oauth2_flows/auth_code.dart#L133) for console applications.\r\n\r\n5. Please add tests for this. The tests  in `test/oauth2_flow/implicit/` test the browser case."}}