GitLab

This reverts commit 0aa36d96.

Reason for revert: landed prior to receiving LGTM, CL is NOT LGTM to gab@
https://chromium-review.googlesource.com/c/chromium/src/+/892582#message-1f3ca486683a18c2df48642825a99cee199034bc

Original change's description:
> Simple code cleanup.
> 
> Passed PrefRegistry and PrefRegistrySyncable by raw pointer instead of reference
> to scoped_ptr. PrefRegistry and PrefRegistrySyncable have an internal reference
> count. There is no need to use a double indirection when passing them around.
> 
> Used base::MakeRefCounted instead of naked 'new' operator.
> 
> Change-Id: I53a0fdfc99b7ef3dd32b191664520659c2585fa5
> Reviewed-on: https://chromium-review.googlesource.com/892582
> Reviewed-by: Scott Violet <sky@chromium.org>
> Commit-Queue: François Degros <fdegros@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#533087}

TBR=sky@chromium.org,gab@chromium.org,erg@chromium.org,fdegros@chromium.org

Change-Id: I082b80ef79afd99f093ac8327381ee1293351cfd
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/895362Reviewed-by: Gabriel Charette <gab@chromium.org>
Commit-Queue: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#533227}