Preliminary interactive layout of window manager's demo_launcher

mojo/examples/html_viewer/html_document_view.h

 // Copyright 2014 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 MOJO_EXAMPLES_HTML_VIEWER_HTML_DOCUMENT_VIEW_H_
 #define MOJO_EXAMPLES_HTML_VIEWER_HTML_DOCUMENT_VIEW_H_
 
 #include "base/compiler_specific.h"
 #include "base/memory/weak_ptr.h"
 #include "mojo/public/cpp/application/lazy_interface_ptr.h"
 #include "mojo/public/interfaces/service_provider/service_provider.mojom.h"
+#include "mojo/services/public/cpp/view_manager/node_observer.h"
 #include "mojo/services/public/cpp/view_manager/view_observer.h"
 #include "mojo/services/public/interfaces/navigation/navigation.mojom.h"
 #include "mojo/services/public/interfaces/network/url_loader.mojom.h"
 #include "third_party/WebKit/public/web/WebFrameClient.h"
 #include "third_party/WebKit/public/web/WebViewClient.h"
 
 namespace mojo {
 
 namespace view_manager {
 class Node;
 class ViewManager;
 class View;
 }
 
 namespace examples {
 
 // A view for a single HTML document.
 class HTMLDocumentView : public blink::WebViewClient,
                          public blink::WebFrameClient,
-                         public view_manager::ViewObserver {
+                         public view_manager::ViewObserver,
+                         public view_manager::NodeObserver {
  public:
   HTMLDocumentView(ServiceProvider* service_provider,
                    view_manager::ViewManager* view_manager);
   virtual ~HTMLDocumentView();
 
   void AttachToNode(view_manager::Node* node);
 
   void Load(URLResponsePtr response);
 
  private:
@@ skipping 16 matching lines @@
       const blink::WebString& stack_trace) OVERRIDE;
   virtual void didNavigateWithinPage(
       blink::WebLocalFrame* frame,
       const blink::WebHistoryItem& history_item,
       blink::WebHistoryCommitType commit_type) OVERRIDE;
 
   // ViewObserver methods:
   virtual void OnViewInputEvent(view_manager::View* view,
                                 const EventPtr& event) OVERRIDE;
 
+  // NodeObserver methods:
+  virtual void OnNodeBoundsChanged(view_manager::Node* node,
+                                   const gfx::Rect&,

[sky, 2014/07/14 19:15:27]

Google C++ style is to always have names for arguments, even if not used. Same
comment applies throughout this patch and you want the same in the definitions.

[hansmuller, 2014/07/14 22:16:42]

Done.

+                                   const gfx::Rect&) OVERRIDE;
+
   void Repaint();
 
   view_manager::ViewManager* view_manager_;
   view_manager::View* view_;
   blink::WebView* web_view_;
   bool repaint_pending_;
   LazyInterfacePtr<navigation::NavigatorHost> navigator_host_;
 
   base::WeakPtrFactory<HTMLDocumentView> weak_factory_;
   DISALLOW_COPY_AND_ASSIGN(HTMLDocumentView);
 };
 
 }  // namespace examples
 }  // namespace mojo
 
 #endif  // MOJO_EXAMPLES_HTML_VIEWER_HTML_DOCUMENT_VIEW_H_