Remove unused locks in mus ContextProvider classes

Remove unused locks in mus ContextProvider classes

BUG=584497
Committed: https://crrev.com/9ea37153effa7c6221608d21a53b43c5f70046f9
Cr-Commit-Position: refs/heads/master@{#391110}

[Fady Samuel, 2016-05-02 22:08:56]

Description was changed from

==========
Remove unused locks in mus ContextProvider classes

BUG=none
==========

to

==========
Remove unused locks in mus ContextProvider classes

BUG=584497
==========

[Fady Samuel, 2016-05-02 22:09:24]

fsamuel@chromium.org changed reviewers:
+ danakj@chromium.org, sky@chromium.org

[Fady Samuel, 2016-05-02 22:09:25]

+danakj@: please review
+sky@ for OWNERS

[Fady Samuel, 2016-05-02 22:09:49]

The CQ bit was checked by fsamuel@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-05-02 22:10:10]

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

[danakj, 2016-05-02 22:10:29]

LGTM

https://codereview.chromium.org/1936423002/diff/1/components/mus/public/cpp/l...
File components/mus/public/cpp/lib/context_provider.cc (right):

https://codereview.chromium.org/1936423002/diff/1/components/mus/public/cpp/l...
components/mus/public/cpp/lib/context_provider.cc:57: return nullptr;
NOTREACHED() as well and comment like in AwRenderThreadContextProvider?

[Fady Samuel, 2016-05-02 22:20:10]

PTAL Scott

https://codereview.chromium.org/1936423002/diff/1/components/mus/public/cpp/l...
File components/mus/public/cpp/lib/context_provider.cc (right):

https://codereview.chromium.org/1936423002/diff/1/components/mus/public/cpp/l...
components/mus/public/cpp/lib/context_provider.cc:57: return nullptr;
On 2016/05/02 22:10:28, danakj wrote:
> NOTREACHED() as well and comment like in AwRenderThreadContextProvider?

Done.

[sky, 2016-05-02 22:40:13]

LGTM

[Fady Samuel, 2016-05-02 22:40:58]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-05-02 22:40:58]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org
Link to the patchset: https://codereview.chromium.org/1936423002/#ps20001
(title: "Addressed Dana's nit")

[commit-bot: I haz the power, 2016-05-02 22:41:49]

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

[danakj, 2016-05-02 22:45:20]

https://codereview.chromium.org/1936423002/diff/20001/components/mus/surfaces...
File components/mus/surfaces/surfaces_context_provider.cc (right):

https://codereview.chromium.org/1936423002/diff/20001/components/mus/surfaces...
components/mus/surfaces/surfaces_context_provider.cc:97: NOTREACEHD();
typo

[Fady Samuel, 2016-05-02 22:48:54]

The CQ bit was unchecked by fsamuel@chromium.org

[Fady Samuel, 2016-05-02 22:51:37]

CQ'ing again.

https://codereview.chromium.org/1936423002/diff/20001/components/mus/surfaces...
File components/mus/surfaces/surfaces_context_provider.cc (right):

https://codereview.chromium.org/1936423002/diff/20001/components/mus/surfaces...
components/mus/surfaces/surfaces_context_provider.cc:97: NOTREACEHD();
On 2016/05/02 22:45:20, danakj wrote:
> typo

Doh! I should've tested compiled locally first. Fixed. Thanks for catching this!

[Fady Samuel, 2016-05-02 22:52:48]

The CQ bit was checked by fsamuel@chromium.org

[Fady Samuel, 2016-05-02 22:52:48]

The patchset sent to the CQ was uploaded after l-g-t-m from danakj@chromium.org,
sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1936423002/#ps40001
(title: "Fixed typo")

[commit-bot: I haz the power, 2016-05-02 22:54:31]

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

[commit-bot: I haz the power, 2016-05-03 00:04:15]

Description was changed from

==========
Remove unused locks in mus ContextProvider classes

BUG=584497
==========

to

==========
Remove unused locks in mus ContextProvider classes

BUG=584497
==========

[commit-bot: I haz the power, 2016-05-03 00:04:15]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-03 00:06:04]

Description was changed from

==========
Remove unused locks in mus ContextProvider classes

BUG=584497
==========

to

==========
Remove unused locks in mus ContextProvider classes

BUG=584497

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

[commit-bot: I haz the power, 2016-05-03 00:06:05]

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