Add TabControlFeature to BlimpClientContextImpl

blimp/client/public/contents/blimp_contents.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 BLIMP_CLIENT_PUBLIC_CONTENTS_BLIMP_CONTENTS_H_
 #define BLIMP_CLIENT_PUBLIC_CONTENTS_BLIMP_CONTENTS_H_
 
 #include "base/macros.h"
 #include "base/supports_user_data.h"
+#include "ui/gfx/geometry/size.h"
 #include "url/gurl.h"
 
 namespace blimp {
 namespace client {
 
 class BlimpContentsObserver;
 class BlimpNavigationController;
 
 // BlimpContents is the core class in blimp client which is responsible for
 // rendering web pages in a rectangular area. The interaction with between the
 // client and engine is encapsulated behind this interface through the use of
 // the navigation controller. Each BlimpContents has exactly one
 // BlimpNavigationController which can be used to load URLs, navigate
 // backwards/forward etc. See blimp_navigation_controller.h for more details.
 class BlimpContents : public base::SupportsUserData {
  public:
   // Retrives the navigation controller that controls all navigation related
   // to this BlimpContents.
   virtual BlimpNavigationController& GetNavigationController() = 0;
 
   // Enables adding and removing observers to this BlimpContents.
   virtual void AddObserver(BlimpContentsObserver* observer) = 0;
   virtual void RemoveObserver(BlimpContentsObserver* observer) = 0;
 
+  // Pushes the size and scale information to the engine, which will affect the
+  // web content display area for all tabs.
+  virtual void SetSizeAndScale(const gfx::Size& size,

[nyquist, 2016/08/16 20:41:04]

Could this possibly be moved to the impl instead? I think size and scale can
come from the BlimpView. If so, remember to move the comment over.

[Menglin, 2016/08/16 21:10:09]

impl has this function now. it overrides this the virtual function here. If we
only need to call it on BlimpContentsImpl, i can remove the virtual function
from here

[nyquist, 2016/08/16 21:41:40]

yeah, you should be able to do that.

[Menglin, 2016/08/17 19:48:46]

Done.

+                               float device_pixel_ratio) = 0;
+
  protected:
   BlimpContents() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(BlimpContents);
 };
 
 }  // namespace client
 }  // namespace blimp
 
 #endif  // BLIMP_CLIENT_PUBLIC_CONTENTS_BLIMP_CONTENTS_H_