HTML titlebars for v2 apps.

chrome/renderer/extensions/app_window_custom_bindings.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/renderer/extensions/app_window_custom_bindings.h"
 
 #include <string>
 
 #include "base/string_util.h"
 #include "chrome/common/extensions/extension_action.h"
 #include "chrome/common/extensions/extension_messages.h"
 #include "chrome/common/url_constants.h"
+#include "chrome/renderer/extensions/chrome_v8_context.h"
 #include "chrome/renderer/extensions/dispatcher.h"
 #include "chrome/renderer/extensions/extension_helper.h"
 #include "content/public/renderer/render_thread.h"
 #include "content/public/renderer/render_view.h"
 #include "grit/renderer_resources.h"
+#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebURLRequest.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFrame.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebNavigationPolicy.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebView.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebWindowFeatures.h"
-#include "third_party/WebKit/Source/WebKit/chromium/public/platform/WebURLRequest.h"
 #include "v8/include/v8.h"
 #include "webkit/glue/webkit_glue.h"
 
 #include "content/public/renderer/render_view.h"
 #include "content/public/renderer/render_view_visitor.h"
 
 namespace extensions {
 
+class DidCreateDocumentElementObserver : public content::RenderViewObserver {
+ public:
+  DidCreateDocumentElementObserver(
+      content::RenderView* view, Dispatcher* dispatcher, bool inject_titlebar)
+      : content::RenderViewObserver(view), dispatcher_(dispatcher),
+        inject_titlebar_(inject_titlebar) {
+  }
+  virtual void DidCreateDocumentElement(WebKit::WebFrame* frame) {

[Mihai Parparita -not on Chrome, 2012/08/30 01:13:47]

Nit: OVERRIDE

[jeremya, 2012/08/30 02:21:02]

Done.

+    v8::HandleScope handle_scope;
+    DCHECK(frame);
+    DCHECK(dispatcher_);
+    ChromeV8Context* v8_context =
+        dispatcher_->v8_context_set().GetByV8Context(
+            frame->mainWorldScriptContext());
+
+    if (!v8_context)
+      return;
+    v8::Context::Scope context_scope(v8_context->v8_context());
+
+    v8::Local<v8::Value> inject_app_titlebar =
+        v8::Local<v8::Value>::New(
+          v8_context->module_system()->RequireForJsInner(
+              v8::String::New("injectAppTitlebar")));
+    if (inject_app_titlebar.IsEmpty())
+      return;
+    v8::Local<v8::Value> value =
+        v8::Local<v8::Object>::Cast(inject_app_titlebar)->Get(

[abarth-chromium, 2012/08/29 23:51:20]

How do you know this case is legit?  We should probably check for whether its an
object before casting it.

[jeremya, 2012/08/30 02:21:02]

Done.

+            v8::String::New("didCreateDocumentElement"));
+    DCHECK(!value.IsEmpty() && value->IsFunction());

[abarth-chromium, 2012/08/29 23:51:20]

How do you know these conditions are met?  These are potentially arbitrary
objects from the page.

[jeremya, 2012/08/30 02:21:02]

I believe the v8 extensions system is sufficient to ensure that this function is
really the function I want to call (it comes from the module system, not
generally from the page). These are really just sanity checks :) (I hope! Adding
koz, who wrote the module system, to double check.)

+    v8::Local<v8::Function> function = v8::Local<v8::Function>::Cast(value);
+    v8::Handle<v8::Value> argv[1];
+    argv[0] = v8::Boolean::New(inject_titlebar_);
+    frame->callFunctionEvenIfScriptDisabled(function,
+                                            v8::Object::New(),
+                                            arraysize(argv),
+                                            argv);
+  }
+
+ private:
+  Dispatcher* dispatcher_;
+  bool inject_titlebar_;
+};
+
 AppWindowCustomBindings::AppWindowCustomBindings(Dispatcher* dispatcher)
     : ChromeV8Extension(dispatcher) {
-  RouteStaticFunction("GetView", &GetView);
+  RouteFunction("GetView",
+      base::Bind(&AppWindowCustomBindings::GetView,
+                 base::Unretained(this)));
 }
 
 namespace {
 class FindViewByID : public content::RenderViewVisitor {
  public:
   explicit FindViewByID(int route_id) : route_id_(route_id), view_(NULL) {
   }
 
   content::RenderView* view() { return view_; }
 
   // Returns false to terminate the iteration.
   virtual bool Visit(content::RenderView* render_view) {
     if (render_view->GetRoutingID() == route_id_) {
       view_ = render_view;
       return false;
     }
     return true;
   }
 
  private:
   int route_id_;
   content::RenderView* view_;
 };
 }  // namespace
 
 v8::Handle<v8::Value> AppWindowCustomBindings::GetView(
     const v8::Arguments& args) {
   // TODO(jeremya): convert this to IDL nocompile to get validation, and turn
   // these argument checks into CHECK().
-  if (args.Length() != 1)
+  if (args.Length() != 2)
     return v8::Undefined();
 
   if (!args[0]->IsInt32())
     return v8::Undefined();
 
+  if (!args[1]->IsBoolean())
+    return v8::Undefined();
+
   int view_id = args[0]->Int32Value();
 
+  bool inject_titlebar = args[1]->BooleanValue();
+
   if (view_id == MSG_ROUTING_NONE)
     return v8::Undefined();
 
   // TODO(jeremya): there exists a direct map of routing_id -> RenderView as
   // ChildThread::current()->ResolveRoute(), but ResolveRoute returns an
   // IPC::Listener*, which RenderView doesn't inherit from (RenderViewImpl
   // does, indirectly, via RenderWidget, but the link isn't exposed outside of
   // content/.)
   FindViewByID view_finder(view_id);
   content::RenderView::ForEach(&view_finder);
   content::RenderView* view = view_finder.view();
   if (!view)
     return v8::Undefined();
 
+  new DidCreateDocumentElementObserver(view, dispatcher(), inject_titlebar);

[abarth-chromium, 2012/08/29 23:51:20]

I assume someone else manages the lifetime of this object?

[jeremya, 2012/08/30 02:21:02]

See content/public/renderer/render_view_observer.h. The object is by default
destroyed when the RenderView goes away.

+
   // TODO(jeremya): it doesn't really make sense to set the opener here, but we
   // need to make sure the security origin is set up before returning the DOM
   // reference. A better way to do this would be to have the browser pass the
   // opener through so opener_id is set in RenderViewImpl's constructor.
   content::RenderView* render_view = GetCurrentRenderView();
   if (!render_view)
     return v8::Undefined();
   WebKit::WebFrame* opener = render_view->GetWebView()->mainFrame();
   WebKit::WebFrame* frame = view->GetWebView()->mainFrame();
   frame->setOpener(opener);
   content::RenderThread::Get()->Send(
       new ExtensionHostMsg_ResumeRequests(view->GetRoutingID()));
 
   v8::Local<v8::Value> window = frame->mainWorldScriptContext()->Global();
   return window;
 }
 
 }  // namespace extensions