Remove second StopMonitor in UserInputMonitorLinux.

Cleanup double StopMonitor and unnecessary condition, leftover from mouse listener removal in https://crrev.com/2577573002.

Bug: 828864
Change-Id: Ia19c5bf4f81897f49b1c0eaf0f6d85a9cfff4984
Reviewed-on: https://chromium-review.googlesource.com/1007194Reviewed-by: Max Morin <maxmorin@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Marina Ciocea <marinaciocea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#550032}

media/base/user_input_monitor_linux.cc

@@ -105,7 +105,6 @@ UserInputMonitorLinuxCore::~UserInputMonitorLinuxCore() {
 void UserInputMonitorLinuxCore::WillDestroyCurrentMessageLoop() {
   DCHECK(io_task_runner_->BelongsToCurrentThread());
   StopMonitor();
-  StopMonitor();
 }
 
 size_t UserInputMonitorLinuxCore::GetKeyPressCount() const {
@@ -200,8 +199,6 @@ void UserInputMonitorLinuxCore::StopMonitor() {
     XFree(x_record_range_);
     x_record_range_ = NULL;
   }
-  if (x_record_range_)
-    return;
 
   // Context must be disabled via the control channel because we can't send
   // any X protocol traffic over the data channel while it's recording.