Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(305)

Unified Diff: headless/public/headless_web_contents.h

Issue 2863953003: Headless: Remove obsolete Mojo code and some other obsolete APIs (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: headless/public/headless_web_contents.h
diff --git a/headless/public/headless_web_contents.h b/headless/public/headless_web_contents.h
index 8986358bdb80bdb68a95936416b6dcddc98a8a78..7f98ebd5f65a3d7fe90de4bb0867118b24d44502 100644
--- a/headless/public/headless_web_contents.h
+++ b/headless/public/headless_web_contents.h
@@ -83,13 +83,6 @@ class HEADLESS_EXPORT HeadlessWebContents {
// Returns the headless tab socket for JS -> C++ if one was created.
virtual HeadlessTabSocket* GetHeadlessTabSocket() const = 0;
- // Returns the FrameTreeNode Id associated with the |devtools_agent_host_id|
- // if any.
- // TODO(alexclarke): Remove this, it doesn't work as expected.
- virtual bool GetFrameTreeNodeIdForDevToolsAgentHostId(
- const std::string& devtools_agent_host_id,
- int* frame_tree_node_id) const = 0;
-
private:
friend class HeadlessWebContentsImpl;
HeadlessWebContents() {}
@@ -111,23 +104,6 @@ class HEADLESS_EXPORT HeadlessWebContents::Builder {
// Specify the initial window size (default is configured in browser options).
Builder& SetWindowSize(const gfx::Size& size);
- // DEPRECATED. Specify an embedder provided Mojo service to be installed. The
- // |service_factory| callback is called on demand by Mojo to instantiate the
- // service if a client asks for it.
- // TODO(alexclarke): Remove AddMojoService.
- template <typename Interface>
- Builder& AddMojoService(
- const base::Callback<void(mojo::InterfaceRequest<Interface>)>&
- service_factory) {
- return AddMojoService(
- Interface::Name_,
- base::Bind(&Builder::ForwardToServiceFactory<Interface>,
- service_factory));
- }
- Builder& AddMojoService(const std::string& service_name,
- const base::Callback<void(
- mojo::ScopedMessagePipeHandle)>& service_factory);
-
// Whether or not a headless tab socket should be created, to allow JS -> C++
// embedder communications.
Builder& CreateTabSocket(bool create_tab_socket);

Powered by Google App Engine
This is Rietveld 408576698