Refactoring display configuration state to allow generic state objects

ui/display/chromeos/x11/native_display_event_dispatcher_x11.cc

 // Copyright 2014 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.
 
 #include "ui/display/chromeos/x11/native_display_event_dispatcher_x11.h"
+#include "ui/display/chromeos/x11/display_mode_x11.h"
+#include "ui/display/chromeos/x11/display_snapshot_x11.h"
 
 #include <X11/extensions/Xrandr.h>
 
 namespace ui {
 
 NativeDisplayEventDispatcherX11::NativeDisplayEventDispatcherX11(
     NativeDisplayDelegateX11::HelperDelegate* delegate,
     int xrandr_event_base)
     : delegate_(delegate), xrandr_event_base_(xrandr_event_base) {}
 
@@ skipping 21 matching lines @@
     return POST_DISPATCH_PERFORM_DEFAULT;
 
   const bool connected = (action == RR_Connected);
   VLOG(1) << "Received RRNotify_OutputChange event:"
           << " output=" << output_change_event->output
           << " crtc=" << output_change_event->crtc
           << " mode=" << output_change_event->mode
           << " action=" << (connected ? "connected" : "disconnected");
 
   bool found_changed_output = false;
-  const std::vector<OutputConfigurator::OutputSnapshot>& cached_outputs =
+  const std::vector<DisplaySnapshot*>& cached_outputs =
       delegate_->GetCachedOutputs();
-  for (std::vector<OutputConfigurator::OutputSnapshot>::const_iterator it =
+  for (std::vector<DisplaySnapshot*>::const_iterator it =
            cached_outputs.begin();
        it != cached_outputs.end();
        ++it) {
-    if (it->output == output_change_event->output) {
-      if (connected && it->crtc == output_change_event->crtc &&
-          it->current_mode == output_change_event->mode) {
+    const DisplaySnapshotX11* x11_output =
+        static_cast<const DisplaySnapshotX11*>(*it);
+    const DisplayModeX11* x11_mode =
+        static_cast<const DisplayModeX11*>(x11_output->current_mode());
+
+    if (x11_output->output() == output_change_event->output) {
+      if (connected && x11_output->crtc() == output_change_event->crtc &&
+          x11_mode->mode_id() == output_change_event->mode) {
         VLOG(1) << "Ignoring event describing already-cached state";
         return POST_DISPATCH_PERFORM_DEFAULT;
       }
       found_changed_output = true;
       break;
     }
   }
 
   if (!connected && !found_changed_output) {
     VLOG(1) << "Ignoring event describing already-disconnected output";
     return POST_DISPATCH_PERFORM_DEFAULT;
   }
 
   delegate_->NotifyDisplayObservers();
 
   return POST_DISPATCH_PERFORM_DEFAULT;
 }
 
 }  // namespace ui