Removing TODO for It2Me Confirmation Dialog

remoting/host/it2me/it2me_confirmation_dialog_android.cc

Index: remoting/host/it2me/it2me_confirmation_dialog_android.cc
diff --git a/remoting/host/it2me/it2me_confirmation_dialog_android.cc b/remoting/host/it2me/it2me_confirmation_dialog_android.cc
index fba9a043a3dfbb9298656000af0bdc85f612a4cb..40a6d38e08650d022a3767b4c3231f19edd87919 100644
--- a/remoting/host/it2me/it2me_confirmation_dialog_android.cc
+++ b/remoting/host/it2me/it2me_confirmation_dialog_android.cc
@@ -4,10 +4,15 @@
 
 #include "remoting/host/it2me/it2me_confirmation_dialog.h"
 
+#include <memory>
+
+#include "base/logging.h"
+
 namespace remoting {
 
 std::unique_ptr<It2MeConfirmationDialog>
 It2MeConfirmationDialogFactory::Create() {
+  NOTIMPLEMENTED();

[Sergey Ulanov, 2016/09/19 20:56:35]

Should this be NOTREACHED()? The calling code doesn't expect nullptr result.
OTOH we shouldn't be compiling host on android, so I'm not sure we even need
this file.

[joedow, 2016/09/20 03:23:41]

I added it to make things simpler if we decided to proceed with the Android host
in the future, since you and Lambros are both suggesting I remove it then I am
happy to do so.

   return nullptr;
 }