[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org

Committed: https://crrev.com/ae1e2192c4d9bc962ecc4e0b82a0bd5ba153577c
Cr-Commit-Position: refs/heads/master@{#425479}

[ssid, 2016-10-14 01:01:28]

Description was changed from

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org
==========

to

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

[ssid, 2016-10-14 01:02:36]

Patchset #1 (id:1) has been deleted

[ssid, 2016-10-14 01:02:56]

Description was changed from

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
==========

to

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org
==========

[ssid, 2016-10-14 02:00:15]

ptal, thanks.

[Primiano Tucci (use gerrit), 2016-10-14 09:20:57]

I had no idea he android::ResourceManager thing happened :)
this LGTM (+oysteine@ FYI)
Maybe, since we are still in time, can we just change the name of that thing to
AndroidResourceManager? We never used ::scope things so far, maybe we can stay
consistent

[ssid, 2016-10-14 13:36:29]

On 2016/10/14 09:20:57, Primiano Tucci wrote:
> I had no idea he android::ResourceManager thing happened :)
> this LGTM (+oysteine@ FYI)
> Maybe, since we are still in time, can we just change the name of that thing
to
> AndroidResourceManager? We never used ::scope things so far, maybe we can stay
> consistent

Sorry I'll keep you in loop. We had meeting with Reed since I just found him in
MTV. He suggested this should be accounted from outside skia since they don't
keep track.
I added it yesterday. The issue is there are so many resources managers and even
in cc there are different in implementations. We use scoped names in cc dump
providers in general. we have  lot of names with cc::

[Primiano Tucci (use gerrit), 2016-10-14 14:18:15]

On 2016/10/14 13:36:29, ssid wrote:
> On 2016/10/14 09:20:57, Primiano Tucci wrote:
> > I had no idea he android::ResourceManager thing happened :)
> > this LGTM (+oysteine@ FYI)
> > Maybe, since we are still in time, can we just change the name of that thing
> to
> > AndroidResourceManager? We never used ::scope things so far, maybe we can
stay
> > consistent
> 
> Sorry I'll keep you in loop. We had meeting with Reed since I just found him
in
> MTV. He suggested this should be accounted from outside skia since they don't
> keep track.
> I added it yesterday. The issue is there are so many resources managers and
even
> in cc there are different in implementations. We use scoped names in cc dump
> providers in general. we have  lot of names with cc::

ah ok then

[ssid, 2016-10-14 18:11:51]

The CQ bit was checked by ssid@chromium.org

[commit-bot: I haz the power, 2016-10-14 18:12:36]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 19:46:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-14 19:46:02]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[ssid, 2016-10-14 21:08:33]

The CQ bit was checked by ssid@chromium.org

[commit-bot: I haz the power, 2016-10-14 21:09:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-14 21:52:39]

Description was changed from

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org
==========

to

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org
==========

[commit-bot: I haz the power, 2016-10-14 21:52:40]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-10-14 21:55:52]

Description was changed from

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org
==========

to

==========
[tracing] Whitelist android ResourceManagerImpl dump provider for field trials.

BUG=654640
R=primiano@chromium.org

Committed: https://crrev.com/ae1e2192c4d9bc962ecc4e0b82a0bd5ba153577c
Cr-Commit-Position: refs/heads/master@{#425479}
==========

[commit-bot: I haz the power, 2016-10-14 21:55:53]

Patchset 1 (id:??) landed as
https://crrev.com/ae1e2192c4d9bc962ecc4e0b82a0bd5ba153577c
Cr-Commit-Position: refs/heads/master@{#425479}