[headless] Initial skeleton of headless/public/

headless/public/web_contents.h

+// Copyright 2015 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 HEADLESS_PUBLIC_WEB_CONTENTS_H_
+#define HEADLESS_PUBLIC_WEB_CONTENTS_H_
+
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+#include "headless/public/headless_export.h"
+#include "url/gurl.h"
+
+class SkBitmap;
+
+namespace headless {
+
+class WebFrame;
+
+class HEADLESS_EXPORT WebContents {
+ public:
+  class Observer {
+   public:
+    // Will be called on browser thread.
+    virtual void DocumentOnLoadCompletedInMainFrame() = 0;
+
+   protected:
+    Observer(WebContents* web_contents);

[alex clarke (OOO till 29th), 2015/11/19 18:17:54]

I note we are not providing the implementation in this patch, was that intended?

nit: explicit
ultra nit: Should this take a scoped_ptr<ObserverImpl> instead of a naked
pointer?  That way it's obvious it takes ownership.

[altimin, 2015/11/19 18:36:24]

Yes, it is.

I didn't really understand the bit about scoped_ptr<ObserverImpl>.

[alex clarke (OOO till 29th), 2015/12/01 20:07:30]

I was assuming it took ownership of |web_contents|, but you probably don't want
to do that :)

+    virtual ~Observer();
+
+   private:
+    class ObserverImpl;
+    scoped_ptr<ObserverImpl> observer_;
+  };
+
+  // Returns main frame for web page.
+  // Should be called on browser main thread.
+  virtual scoped_ptr<WebFrame> main_frame();
+
+  // Requests browser tab to nagivate to given url.
+  // Should be called on browser main thread.
+  virtual void OpenURL(const GURL& url);

[alex clarke (OOO till 29th), 2015/11/19 18:17:54]

This won't always succeed, how are we going to report errors?

[altimin, 2015/11/19 18:36:24]

In bright and not so distant future observer will have more methods, not just
DocumentOnLoadCompleted().

+
+  using ScreenshotCallback = base::Callback<void(scoped_ptr<SkBitmap>)>;
+  // Requests an image of web contents.
+  // Should be called on browser main thread.
+  virtual void GetScreenshot(const ScreenshotCallback& callback);
+
+ protected:
+  virtual ~WebContents() {}
+};
+
+}  // namespace headless
+
+#endif  // HEADLESS_PUBLIC_WEB_CONTENTS_H_