Add TrimmedAppID to info service.

Add TrimmedAppID to info service.

Also refactor info service to share common 'augmented' methods in
the service/info package itself, ala service/datastore.

Split out from CL: https://chromiumcodereview.appspot.com/1537883002

R=dnj@chromium.org
BUG=550684
Committed: https://github.com/luci/gae/commit/48d5fbded2f1bc5cfa24d377c180d871a238351e

[dnj (Google), 2016-06-11 02:47:38]

dnj@google.com changed reviewers:
+ dnj@google.com

[dnj (Google), 2016-06-11 02:47:39]

lgtm w/ comment

https://codereview.chromium.org/2062613002/diff/1/service/info/wrapper.go
File service/info/wrapper.go (right):

https://codereview.chromium.org/2062613002/diff/1/service/info/wrapper.go#new...
service/info/wrapper.go:19: func (i infoImpl) MustNamespace(namespace string)
context.Context {
Any reason *not* to make this a pointer receiver?

[iannucci, 2016-06-14 00:01:09]

https://chromiumcodereview.appspot.com/2062613002/diff/1/service/info/wrapper.go
File service/info/wrapper.go (right):

https://chromiumcodereview.appspot.com/2062613002/diff/1/service/info/wrapper...
service/info/wrapper.go:19: func (i infoImpl) MustNamespace(namespace string)
context.Context {
On 2016/06/11 02:47:38, dnj (Google) wrote:
> Any reason *not* to make this a pointer receiver?

double deref for no reason? additional opportunities for nils?

[iannucci, 2016-06-14 00:01:26]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2016-06-14 00:01:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2062613002/1

[commit-bot: I haz the power, 2016-06-14 00:07:12]

Description was changed from

==========
Add TrimmedAppID to info service.

Also refactor info service to share common 'augmented' methods in
the service/info package itself, ala service/datastore.

Split out from CL: https://chromiumcodereview.appspot.com/1537883002

R=dnj@chromium.org
BUG=550684
==========

to

==========
Add TrimmedAppID to info service.

Also refactor info service to share common 'augmented' methods in
the service/info package itself, ala service/datastore.

Split out from CL: https://chromiumcodereview.appspot.com/1537883002

R=dnj@chromium.org
BUG=550684

Committed:
https://github.com/luci/gae/commit/48d5fbded2f1bc5cfa24d377c180d871a238351e
==========

[commit-bot: I haz the power, 2016-06-14 00:07:13]

Committed patchset #1 (id:1) as
https://github.com/luci/gae/commit/48d5fbded2f1bc5cfa24d377c180d871a238351e