[Remoting Android] JniGlDisplayHandler calls invalidate() on UI thread

remoting/client/jni/display_updater_factory.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 REMOTING_CLIENT_JNI_DISPLAY_UPDATER_FACTORY_H_
 #define REMOTING_CLIENT_JNI_DISPLAY_UPDATER_FACTORY_H_
 
 #include <memory>
 
 #include "base/macros.h"
 
 namespace remoting {
 
 namespace protocol {
 class CursorShapeStub;
 class VideoRenderer;
 }  // namespace protocol
 
 // Interface for creating objects to update image (desktop frame or cursor
 // shape) to display. Factory functions can be called on any thread but the
 // returned object should be used on the network thread.
 class DisplayUpdaterFactory {
  public:
   virtual ~DisplayUpdaterFactory() {}

[Yuwei, 2016/09/12 20:49:06]

Tried to make this protected but `display_handler_.reset(raw_handler)` requires
the destructor.

BTW we may also remove this interface? Since now we only have one renderer.

 
   virtual std::unique_ptr<protocol::CursorShapeStub>
   CreateCursorShapeStub() = 0;
   virtual std::unique_ptr<protocol::VideoRenderer> CreateVideoRenderer() = 0;
+  virtual void Destroy() = 0;
 
  protected:
   DisplayUpdaterFactory() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(DisplayUpdaterFactory);
 };
 
 }  // namespace remoting
 
 #endif  // REMOTING_CLIENT_JNI_DISPLAY_UPDATER_FACTORY_H_