Make DictionaryValue::DeepCopyWithoutEmptyChildren return a scoped_ptr

Make DictionaryValue::DeepCopyWithoutEmptyChildren return a scoped_ptr

BUG=none

Committed: https://crrev.com/6e04d5002ba4cff956df7f246c8bf3bfe0c8a6f2
Cr-Commit-Position: refs/heads/master@{#331223}

[Evan Stade, 2015-05-21 22:40:50]

estade@chromium.org changed reviewers:
+ brettw@chromium.org

[brettw, 2015-05-22 20:59:33]

LGTM with semicolons fixed.

https://codereview.chromium.org/1150863002/diff/1/base/values_unittest.cc
File base/values_unittest.cc (right):

https://codereview.chromium.org/1150863002/diff/1/base/values_unittest.cc#new...
base/values_unittest.cc:683: ;
Did you screw up search and replace for these places?

[Evan Stade, 2015-05-22 21:28:57]

https://codereview.chromium.org/1150863002/diff/1/base/values_unittest.cc
File base/values_unittest.cc (right):

https://codereview.chromium.org/1150863002/diff/1/base/values_unittest.cc#new...
base/values_unittest.cc:683: ;
On 2015/05/22 20:59:32, brettw wrote:
> Did you screw up search and replace for these places?

must have done

[Evan Stade, 2015-05-22 21:47:12]

The CQ bit was checked by estade@chromium.org

[Evan Stade, 2015-05-22 21:47:12]

The patchset sent to the CQ was uploaded after l-g-t-m from brettw@chromium.org
Link to the patchset: https://codereview.chromium.org/1150863002/#ps20001
(title: "remove semicolons")

[commit-bot: I haz the power, 2015-05-22 21:49:57]

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

[commit-bot: I haz the power, 2015-05-23 02:42:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-23 02:43:09]

Patchset 2 (id:??) landed as
https://crrev.com/6e04d5002ba4cff956df7f246c8bf3bfe0c8a6f2
Cr-Commit-Position: refs/heads/master@{#331223}