New issue
Advanced search Search tips

Issue 656833 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Aug 20
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug

Blocked on:
issue 623989



Sign in to add a comment

[Remoting Android] Sorting out Chromoting.java code using promise and events

Project Member Reported by yuweih@chromium.org, Oct 18 2016

Issue description

Currently the Chromoting class is messy, with callbacks and listeners everywhere. Since there are promise and event available in our code base. To improve code readability and maintainability, we can use base.Promise to rewrite async sequences and chromoting.Event to rewrite event listener bindings.

I think the good time for us to start refactoring these is after N SDK is rolled-in (and stabilized) and we are able to write lambdas.
 

Comment 1 by yuweih@chromium.org, Oct 18 2016

Labels: M-56

Comment 2 by yuweih@chromium.org, Oct 20 2016

Blockedon: 623989
Labels: -M-56 M-57

Comment 4 by joedow@chromium.org, Jan 27 2017

Labels: -M-57
Status: WontFix (was: Assigned)

Sign in to add a comment