11 instances here: https://cs.chromium.org/search/?q=WrapUnique+file:%5Esrc/chrome/browser/ui/webui/+package:%5Echromium$&type=cs All of these look like we can replace the bare usage of "new" with MakeUnique. See here for docs on MakeUnique: https://cs.chromium.org/chromium/src/third_party/pdfium/third_party/base/ptr_util.h?q=ptr_util.h&dr=CSs&l=5
11 instances here: https://cs.chromium.org/search/?q=WrapUnique+file:%5Esrc/chrome/browser/ui/webui/+package:%5Echromium$&type=cs All of these look like we can replace the bare usage of "new" with MakeUnique. See here for docs on MakeUnique: https://cs.chromium.org/chromium/src/base/memory/ptr_util.h?dr=CSs&q=base/memory/ptr_&l=1
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/35e6d16c1b94cefc44aef0739b219ecdce2a20cc commit 35e6d16c1b94cefc44aef0739b219ecdce2a20cc Author: amberwon <amberwon@google.com> Date: Tue Jun 27 20:24:23 2017 Replace WrapUnique with MakeUnique BUG= 736896 Review-Url: https://codereview.chromium.org/2957863005 Cr-Commit-Position: refs/heads/master@{#482729} [modify] https://crrev.com/35e6d16c1b94cefc44aef0739b219ecdce2a20cc/chrome/browser/ui/webui/chromeos/first_run/first_run_actor.cc [modify] https://crrev.com/35e6d16c1b94cefc44aef0739b219ecdce2a20cc/chrome/browser/ui/webui/chromeos/login/l10n_util_unittest.cc [modify] https://crrev.com/35e6d16c1b94cefc44aef0739b219ecdce2a20cc/chrome/browser/ui/webui/options/content_settings_handler.cc [modify] https://crrev.com/35e6d16c1b94cefc44aef0739b219ecdce2a20cc/chrome/browser/ui/webui/settings/profile_info_handler.cc [modify] https://crrev.com/35e6d16c1b94cefc44aef0739b219ecdce2a20cc/chrome/browser/ui/webui/site_settings_helper.cc
Comment 1 by tommycli@chromium.org
, Jun 26 2017