New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 622848 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Nov 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Allow creating an Android Tab(.java) for Blimp without a ContentViewCore

Project Member Reported by siev...@chromium.org, Jun 23 2016

Issue description

Master bug to track what's needed for being able to do this in a reasonably good way.
 
Blockedon: 622847
Blocking this on 622847 for forwarding interesting stuff from the container view in a shared codepath.

Some thoughts:

There are also gesture event ACKs coming back. Looking at what this is currently used for, it's possible that Blimp does either not care about these, or could leave this as an implementation detail in its own layer.

There is also  crbug.com/617750  which eventually will be the interface to delegate back to the embedding View, such as adding popups. Maybe this should be an implementation detail in the blimp layer (and between its own container view) and how it wants to do its custom UI.



Blockedon: 620172
Blocking on 620172: I think eliminating ContentViewClient and moving functionality elsewhere as proposed over there moves things out of your way a bit more, since most of this doesn't apply to Blimp (but is implemented for the content layer). 
Thanks Daniel!  This sounds good.  We can have the Tab depend on classes that are shared (like ViewAndroid or another delegate/generic ui/ class), but not on ContentViewCore directly.
Blockedon: 624666
Blockedon: 624830
Cc: yus...@chromium.org
Blockedon: 626764
Blocking on 626764, which is about removing content_view_core.h *public* interface.
Because as long as it's there then the application can/will have native dependencies on CVC (like in tab_android.cc), which Blimp doesn't have.
Blockedon: -624666
Cc: boliu@chromium.org
Status: WontFix (was: Available)
Obsolete, WontFix.
Labels: Archive-Blimp
Blockedon: -624830 -622847 -620172 -626764

Sign in to add a comment