Some easy linked_ptr removal from extensions/common/

Some easy linked_ptr removal from extensions/common/

BUG=556939
Test=None, internal change.

Committed: https://crrev.com/d5a387ce8c4d17179d64e958c61c9eb00addccf0
Cr-Commit-Position: refs/heads/master@{#432063}

[lazyboy, 2016-11-12 03:57:09]

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

[commit-bot: I haz the power, 2016-11-12 03:57:20]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-12 04:58:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-12 04:58:18]

Dry run: This issue passed the CQ dry run.

[lazyboy, 2016-11-14 22:31:50]

lazyboy@chromium.org changed reviewers:
+ asargent@chromium.org

[asargent_no_longer_on_chrome, 2016-11-15 00:41:46]

lgtm

[lazyboy, 2016-11-15 00:59:13]

The CQ bit was checked by lazyboy@chromium.org

[commit-bot: I haz the power, 2016-11-15 01:00:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-15 02:34:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-15 02:34:40]

Failed to apply patch for extensions/common/event_filter.cc:
While running git apply --index -p1;
  error: patch failed: extensions/common/event_filter.cc:7
  error: extensions/common/event_filter.cc: patch does not apply

Patch:       extensions/common/event_filter.cc
Index: extensions/common/event_filter.cc
diff --git a/extensions/common/event_filter.cc
b/extensions/common/event_filter.cc
index
2a16678cbdf2026aa6e327bd73861f27fb8f7490..fa587016ba3480f1a7f9c9c24b54b08d4c898422
100644
--- a/extensions/common/event_filter.cc
+++ b/extensions/common/event_filter.cc
@@ -7,6 +7,7 @@
 #include <string>
 #include <utility>
 
+#include "base/memory/ptr_util.h"
 #include "components/url_matcher/url_matcher_factory.h"
 #include "ipc/ipc_message.h"
 
@@ -67,8 +68,8 @@ EventFilter::MatcherID EventFilter::AddEventMatcher(
         std::make_pair((*it)->id(), id));
   }
   id_to_event_name_[id] = event_name;
-  event_matchers_[event_name][id] = linked_ptr<EventMatcherEntry>(
-      new EventMatcherEntry(std::move(matcher), &url_matcher_,
condition_sets));
+  event_matchers_[event_name][id] = base::MakeUnique<EventMatcherEntry>(
+      std::move(matcher), &url_matcher_, condition_sets);
   return id;
 }

[commit-bot: I haz the power, 2016-11-15 03:01:48]

Description was changed from

==========
Some easy linked_ptr removal from extensions/common/

BUG=556939
Test=None, internal change.
==========

to

==========
Some easy linked_ptr removal from extensions/common/

BUG=556939
Test=None, internal change.

Committed: https://crrev.com/d5a387ce8c4d17179d64e958c61c9eb00addccf0
Cr-Commit-Position: refs/heads/master@{#432063}
==========

[commit-bot: I haz the power, 2016-11-15 03:01:49]

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

[limasdf, 2016-11-15 10:45:55]

limasdf@gmail.com changed reviewers:
+ limasdf@gmail.com

[limasdf, 2016-11-15 10:45:56]

@lazy, awesome. I can't wait to see how you gonna treat linked_ptr in
e/b/api/declarative/rules_registry.h :)

[lazyboy, 2016-11-15 21:12:30]

On 2016/11/15 10:45:56, limasdf wrote:
> @lazy, awesome. I can't wait to see how you gonna treat linked_ptr in
> e/b/api/declarative/rules_registry.h :)

I think we'd need to use scoped_refptr for api::events::Rule, wdyt?
(If that's what you were wondering about).