Migrate Blimp Linux Client to 0.6 APIs |
|||||
Issue descriptionCurrently the Linux client is using the old 0.5 class structure for a Blimp client. This needs to be ported over to the new 0.6 files. This involves something like: - Remove BlimpClientSessionLinux and pull display/event handling out. - Build a BlimpClientContext. - Add/use ConnectWithAssignment (method also being added in an integration test). - Build a BlimpContents. This might also require using a dummy IMEDelegate that auto-hides when it gets a ShowIME request. This might also be added during integration tests as well though.
,
Sep 8 2016
Wez can you triage? Marking this as blocked on the integration test. That should hopefully land this week.
,
Sep 8 2016
Assigning to Tommy, since he has been working with the Linux Client recently. Tommy, please take a look at the work breakdown in the bug description and let me know if you have questions!
,
Sep 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01 commit d890645241cfa41fe40e22a5cdf5cd9d6cef4f01 Author: steimel <steimel@chromium.org> Date: Thu Sep 29 16:59:47 2016 Migrating the Linux Blimp client away from using the deprecated BlimpClientSession and towards using BlimpClientContext. For this CL, we now create a BlimpClientContext and create a BlimpContents from that to send to the display manager, which replaces the individual features being sent before (e.g. TabControlFeature). blimp_main now handles some things that were being handled in the BlimpClientSession (e.g. creating a display manager and an IO thread) BUG= 645202 Review-Url: https://codereview.chromium.org/2363153002 Cr-Commit-Position: refs/heads/master@{#421851} [modify] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/BUILD.gn [add] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/app/linux/blimp_client_context_delegate_linux.cc [add] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/app/linux/blimp_client_context_delegate_linux.h [modify] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/app/linux/blimp_display_manager.cc [modify] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/app/linux/blimp_display_manager.h [add] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/app/linux/blimp_display_manager_delegate_main.cc [add] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/app/linux/blimp_display_manager_delegate_main.h [modify] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/app/linux/blimp_main.cc [modify] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/support/BUILD.gn [add] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/support/session/BUILD.gn [add] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/support/session/blimp_default_identity_provider.cc [add] https://crrev.com/d890645241cfa41fe40e22a5cdf5cd9d6cef4f01/blimp/client/support/session/blimp_default_identity_provider.h
,
Nov 14 2016
,
Dec 9 2016
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by dtrainor@chromium.org
, Sep 8 2016