Refactoring: Avoid redundant checks in SingletonFor

Refactoring: Avoid redundant checks in SingletonFor

SingletonFor(type, state) returns NULL and does nothing whenever SingletonFor(type) also return NULL. So checking this beforehand is redundant.

This is my first CL, so I also use it to get familiar with the system.

R=titzer@chromium.org
BUG=

Committed: https://crrev.com/9a65c0ef03c523f04fd35cf3e31033e99b79c622
Cr-Commit-Position: refs/heads/master@{#35285}

[titzer, 2016-04-06 08:37:41]

lgtm if description is updated as per in-person discussion

[clemensh, 2016-04-06 08:38:25]

Description was changed from

==========
Minor refactoring

This is my first CL, so I also use it to get familiar with the system.

R=titzer@chromium.org
BUG=
==========

to

==========
Refactoring: Avoid redundant checks in SingletonFor

SingletonFor(type, state) returns NULL and does nothing whenever
SingletonFor(type) also return NULL. So checking this beforehand is redundant.

This is my first CL, so I also use it to get familiar with the system.

R=titzer@chromium.org
BUG=
==========

[clemensh, 2016-04-06 08:38:50]

The CQ bit was checked by clemensh@google.com

[commit-bot: I haz the power, 2016-04-06 08:38:54]

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

[clemensh, 2016-04-06 08:48:20]

The CQ bit was unchecked by clemensh@google.com

[clemensh, 2016-04-06 08:50:45]

The CQ bit was checked by clemensh@google.com

[commit-bot: I haz the power, 2016-04-06 08:50:55]

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

[commit-bot: I haz the power, 2016-04-06 08:56:29]

Description was changed from

==========
Refactoring: Avoid redundant checks in SingletonFor

SingletonFor(type, state) returns NULL and does nothing whenever
SingletonFor(type) also return NULL. So checking this beforehand is redundant.

This is my first CL, so I also use it to get familiar with the system.

R=titzer@chromium.org
BUG=
==========

to

==========
Refactoring: Avoid redundant checks in SingletonFor

SingletonFor(type, state) returns NULL and does nothing whenever
SingletonFor(type) also return NULL. So checking this beforehand is redundant.

This is my first CL, so I also use it to get familiar with the system.

R=titzer@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2016-04-06 08:56:30]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-06 08:57:57]

Description was changed from

==========
Refactoring: Avoid redundant checks in SingletonFor

SingletonFor(type, state) returns NULL and does nothing whenever
SingletonFor(type) also return NULL. So checking this beforehand is redundant.

This is my first CL, so I also use it to get familiar with the system.

R=titzer@chromium.org
BUG=
==========

to

==========
Refactoring: Avoid redundant checks in SingletonFor

SingletonFor(type, state) returns NULL and does nothing whenever
SingletonFor(type) also return NULL. So checking this beforehand is redundant.

This is my first CL, so I also use it to get familiar with the system.

R=titzer@chromium.org
BUG=

Committed: https://crrev.com/9a65c0ef03c523f04fd35cf3e31033e99b79c622
Cr-Commit-Position: refs/heads/master@{#35285}
==========

[commit-bot: I haz the power, 2016-04-06 08:57:58]

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