Allow input & clipboard filters to be disabled without NULLing target stub.

remoting/protocol/input_filter.h

 // Copyright (c) 2012 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_PROTOCOL_INPUT_FILTER_H_
 #define REMOTING_PROTOCOL_INPUT_FILTER_H_
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "remoting/protocol/input_stub.h"
 
 namespace remoting {
 namespace protocol {
 
 // Forwards input events to |input_stub|, iif |input_stub| is not NULL.

[simonmorris, 2012/08/20 16:37:18]

"iff" doesn't apply now.

Is it feasible to require input_stub to be non-NULL?

[Wez, 2012/08/20 19:58:21]

Done.

+// Input forwarding may be disabled independently of the |input_stub| being
+// set.  InputFilters initially have input forwarding enabled.
 class InputFilter : public InputStub {
  public:
   InputFilter();
   explicit InputFilter(InputStub* input_stub);
   virtual ~InputFilter();
 
-  // Return the InputStub that events will be forwarded to.
-  InputStub* input_stub() { return input_stub_; }
-
   // Set the InputStub that events will be forwarded to.
   void set_input_stub(InputStub* input_stub) {
     input_stub_ = input_stub;
   }
 
+  // Enable/disable routing of events to the InputStub.
+  void set_enabled(bool enabled) {
+    enabled_ = enabled;
+  }
+  bool enabled() const {
+    return enabled_;
+  }
+
   // InputStub interface.
   virtual void InjectKeyEvent(const KeyEvent& event) OVERRIDE;
   virtual void InjectMouseEvent(const MouseEvent& event) OVERRIDE;
 
  private:
   InputStub* input_stub_;
+  bool enabled_;
 
   DISALLOW_COPY_AND_ASSIGN(InputFilter);
 };
 
 } // namespace protocol
 } // namespace remoting
 
 #endif // REMOTING_PROTOCOL_INPUT_FILTER_H_