Move SearchProvider implementation into Blink and make it a no-op.

third_party/WebKit/Source/core/frame/External.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef External_h
+#define External_h
+
+#include "bindings/core/v8/ScriptWrappable.h"
+#include "platform/heap/Handle.h"
+
+namespace blink {
+
+class External : public GarbageCollected<External>, public ScriptWrappable {
+    DEFINE_WRAPPERTYPEINFO();

[jochen (gone - plz use gerrit), 2016/07/27 07:34:35]

wtf_make_noncopiable?

[dcheng, 2016/07/27 08:01:59]

That's true for all things that derive from GarbageCollected<T>. We don't have a
strong convention of doing this elsewhere (in fact, most code usually doesn't
spell this out), so I think for consistency, we should leave it out here?

+public:
+    External() { }

[jochen (gone - plz use gerrit), 2016/07/27 07:34:35]

= default?

[dcheng, 2016/07/27 08:01:59]

Done, but any particular reason for doing so? It's more characters to write it
out, and I think it won't give this class a trivial constructor anyway:
http://stackoverflow.com/questions/20828907/the-new-keyword-default-in-c11

+
+    void AddSearchProvider() { }
+    void IsSearchProviderInstalled() { }
+
+    DEFINE_INLINE_TRACE() { }
+};
+
+} // namespace blink
+
+#endif // External_h