chromeos: Fix display failures when going to mirrored mode.

chromeos/display/output_configurator.cc

 // 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.
 
 #include "chromeos/display/output_configurator.h"
 
 #include <X11/Xlib.h>
 #include <X11/extensions/Xrandr.h>
 #include <X11/extensions/XInput2.h>
 
@@ skipping 83 matching lines @@
   // "Projecting" is defined as having more than 1 output connected while at
   // least one of them is an internal output.
   return has_internal_output && (connected_output_count > 1);
 }
 
 }  // namespace
 
 OutputConfigurator::ModeInfo::ModeInfo()
     : width(0),
       height(0),
-      interlaced(false) {}
+      interlaced(false),
+      refresh_rate(0.0) {}
 
-OutputConfigurator::ModeInfo::ModeInfo(int width, int height, bool interlaced)
+OutputConfigurator::ModeInfo::ModeInfo(int width,
+                                       int height,
+                                       bool interlaced,
+                                       float refresh_rate)
     : width(width),
       height(height),
-      interlaced(interlaced) {}
+      interlaced(interlaced),
+      refresh_rate(refresh_rate) {}
 
 OutputConfigurator::CoordinateTransformation::CoordinateTransformation()
     : x_scale(1.0),
       x_offset(0.0),
       y_scale(1.0),
       y_offset(0.0) {}
 
 OutputConfigurator::OutputSnapshot::OutputSnapshot()
     : output(None),
       crtc(None),
@@ skipping 45 matching lines @@
   }
 }
 
 // static
 const OutputConfigurator::ModeInfo* OutputConfigurator::GetModeInfo(
     const OutputSnapshot& output,
     RRMode mode) {
   if (mode == None)
     return NULL;
 
-  std::map<RRMode, ModeInfo>::const_iterator it = output.mode_infos.find(mode);
+  ModeInfoMap::const_iterator it = output.mode_infos.find(mode);
   if (it == output.mode_infos.end()) {
     LOG(WARNING) << "Unable to find info about mode " << mode
                  << " for output " << output.output;
     return NULL;
   }
   return &it->second;
 }
 
+// static
+RRMode OutputConfigurator::FindOutputModeMatchingSize(
+    const OutputSnapshot& output,
+    int width,
+    int height) {
+  RRMode found = None;
+  float best_rate = 0;
+  bool non_interlaced_found = false;
+  for (ModeInfoMap::const_iterator it = output.mode_infos.begin();
+       it != output.mode_infos.end(); ++it) {
+    RRMode mode = it->first;
+    const ModeInfo& info = it->second;
+
+    if (info.width == width && info.height == height) {
+      if (info.interlaced) {
+        if (non_interlaced_found)
+          continue;
+      } else {
+        // Reset the best rate if the non interlaced is
+        // found the first time.
+        if (!non_interlaced_found)
+          best_rate = info.refresh_rate;
+        non_interlaced_found = true;
+      }
+      if (info.refresh_rate < best_rate)
+        continue;
+
+      found = mode;
+      best_rate = info.refresh_rate;
+    }
+  }
+  return found;
+}
+
 OutputConfigurator::OutputConfigurator()
     : state_controller_(NULL),
       mirroring_controller_(NULL),
+      is_panel_fitting_enabled_(false),
       configure_display_(base::chromeos::IsRunningOnChromeOS()),
       xrandr_event_base_(0),
       output_state_(STATE_INVALID),
       power_state_(DISPLAY_POWER_ALL_ON) {
 }
 
 OutputConfigurator::~OutputConfigurator() {}
 
 void OutputConfigurator::SetDelegateForTesting(scoped_ptr<Delegate> delegate) {
   delegate_ = delegate.Pass();
   configure_display_ = true;
 }
 
 void OutputConfigurator::SetInitialDisplayPower(DisplayPowerState power_state) {
   DCHECK_EQ(output_state_, STATE_INVALID);
   power_state_ = power_state;
 }
 
 void OutputConfigurator::Init(bool is_panel_fitting_enabled) {
+  is_panel_fitting_enabled_ = is_panel_fitting_enabled;
   if (!configure_display_)
     return;
 
   if (!delegate_)
     delegate_.reset(new RealOutputConfiguratorDelegate());
-  delegate_->SetPanelFittingEnabled(is_panel_fitting_enabled);
 }
 
 void OutputConfigurator::Start(uint32 background_color_argb) {
   if (!configure_display_)
     return;
 
   delegate_->GrabServer();
   delegate_->InitXRandRExtension(&xrandr_event_base_);
 
-  std::vector<OutputSnapshot> outputs =
-      delegate_->GetOutputs(state_controller_);
+  std::vector<OutputSnapshot> outputs = GetOutputs();
   if (outputs.size() > 1 && background_color_argb)
     delegate_->SetBackgroundColor(background_color_argb);
   const OutputState new_state = GetOutputState(outputs, power_state_);
   const bool success = EnterStateOrFallBackToSoftwareMirroring(
       new_state, power_state_, outputs);
 
   // Force the DPMS on chrome startup as the driver doesn't always detect
   // that all displays are on when signing out.
   delegate_->ForceDPMSOn();
   delegate_->UngrabServer();
   delegate_->SendProjectingStateToPowerManager(IsProjecting(outputs));
   NotifyObservers(success, new_state);
 }
 
 void OutputConfigurator::Stop() {
   configure_display_ = false;
 }
 
 bool OutputConfigurator::SetDisplayPower(DisplayPowerState power_state,
                                          int flags) {
   if (!configure_display_)
     return false;
 
   VLOG(1) << "SetDisplayPower: power_state="
           << DisplayPowerStateToString(power_state) << " flags=" << flags;
   if (power_state == power_state_ && !(flags & kSetDisplayPowerForceProbe))
     return true;
 
   delegate_->GrabServer();
-  std::vector<OutputSnapshot> outputs =
-      delegate_->GetOutputs(state_controller_);
+  std::vector<OutputSnapshot> outputs = GetOutputs();
 
   const OutputState new_state = GetOutputState(outputs, power_state);
   bool attempted_change = false;
   bool success = false;
 
   bool only_if_single_internal_display =
       flags & kSetDisplayPowerOnlyIfSingleInternalDisplay;
   bool single_internal_display = outputs.size() == 1 && outputs[0].is_internal;
   if (single_internal_display || !only_if_single_internal_display) {
     success = EnterStateOrFallBackToSoftwareMirroring(
@@ skipping 20 matching lines @@
   if (output_state_ == new_state) {
     // Cancel software mirroring if the state is moving from
     // STATE_DUAL_EXTENDED to STATE_DUAL_EXTENDED.
     if (mirroring_controller_ && new_state == STATE_DUAL_EXTENDED)
       mirroring_controller_->SetSoftwareMirroring(false);
     NotifyObservers(true, new_state);
     return true;
   }
 
   delegate_->GrabServer();
-  std::vector<OutputSnapshot> outputs =
-      delegate_->GetOutputs(state_controller_);
+  std::vector<OutputSnapshot> outputs = GetOutputs();
   const bool success = EnterStateOrFallBackToSoftwareMirroring(
       new_state, power_state_, outputs);
   delegate_->UngrabServer();
 
   NotifyObservers(success, new_state);
   return success;
 }
 
 bool OutputConfigurator::Dispatch(const base::NativeEvent& event) {
   if (!configure_display_)
@@ skipping 105 matching lines @@
   } else {
     configure_timer_.reset(new base::OneShotTimer<OutputConfigurator>());
     configure_timer_->Start(
         FROM_HERE,
         base::TimeDelta::FromMilliseconds(kConfigureDelayMs),
         this,
         &OutputConfigurator::ConfigureOutputs);
   }
 }
 
+std::vector<OutputConfigurator::OutputSnapshot>
+OutputConfigurator::GetOutputs() {
+  std::vector<OutputSnapshot> outputs = delegate_->GetOutputs();
+
+  // Set |selected_mode| fields.
+  for (size_t i = 0; i < outputs.size(); ++i) {
+    OutputSnapshot* output = &outputs[i];
+    if (output->has_display_id) {
+      int width = 0, height = 0;
+      if (state_controller_ &&
+          state_controller_->GetResolutionForDisplayId(
+              output->display_id, &width, &height)) {
+        output->selected_mode =
+            FindOutputModeMatchingSize(*output, width, height);
+      }
+    }
+    // Fall back to native mode.
+    if (output->selected_mode == None)
+      output->selected_mode = output->native_mode;
+  }
+
+  // Set |mirror_mode| fields.
+  if (outputs.size() == 2) {
+    bool one_is_internal = outputs[0].is_internal;
+    bool two_is_internal = outputs[1].is_internal;
+    int internal_outputs = (one_is_internal ? 1 : 0) +
+        (two_is_internal ? 1 : 0);
+    DCHECK_LT(internal_outputs, 2);
+    LOG_IF(WARNING, internal_outputs == 2)
+        << "Two internal outputs detected.";
+
+    bool can_mirror = false;
+    for (int attempt = 0; !can_mirror && attempt < 2; ++attempt) {
+      // Try preserving external output's aspect ratio on the first attempt.
+      // If that fails, fall back to the highest matching resolution.
+      bool preserve_aspect = attempt == 0;
+
+      if (internal_outputs == 1) {
+        if (one_is_internal) {
+          can_mirror = FindOrCreateMirrorMode(&outputs[0], &outputs[1],
+              is_panel_fitting_enabled_, preserve_aspect);
+        } else {
+          DCHECK(two_is_internal);
+          can_mirror = FindOrCreateMirrorMode(&outputs[1], &outputs[0],
+              is_panel_fitting_enabled_, preserve_aspect);
+        }
+      } else {  // if (internal_outputs == 0)
+        // No panel fitting for external outputs, so fall back to exact match.
+        can_mirror = FindOrCreateMirrorMode(&outputs[0], &outputs[1], false,
+                                            preserve_aspect);
+        if (!can_mirror && preserve_aspect) {
+          // FindOrCreateMirrorMode will try to preserve aspect ratio of
+          // what it thinks is external display, so if it didn't succeed
+          // with one, maybe it will succeed with the other.  This way we
+          // will have the correct aspect ratio on at least one of them.
+          can_mirror = FindOrCreateMirrorMode(&outputs[1], &outputs[0],
+              false, preserve_aspect);
+        }
+      }
+    }
+  }
+
+  return outputs;
+}
+
+bool OutputConfigurator::FindOrCreateMirrorMode(
+    OutputSnapshot* internal_output,
+    OutputSnapshot* external_output,
+    bool try_creating,
+    bool preserve_aspect) {
+  const ModeInfo* internal_native_info =
+      GetModeInfo(*internal_output, internal_output->native_mode);
+  const ModeInfo* external_native_info =
+      GetModeInfo(*external_output, external_output->native_mode);
+  if (!internal_native_info || !external_native_info)
+    return false;
+
+  // Check if some external output resolution can be mirrored on internal.
+  // Prefer the modes in the order that X sorts them, assuming this is the order
+  // in which they look better on the monitor.
+  for (ModeInfoMap::const_iterator external_it =
+       external_output->mode_infos.begin();
+       external_it != external_output->mode_infos.end(); ++external_it) {
+    const ModeInfo& external_info = external_it->second;
+    bool is_native_aspect_ratio =
+        external_native_info->width * external_info.height ==
+        external_native_info->height * external_info.width;
+    if (preserve_aspect && !is_native_aspect_ratio)
+      continue;  // Allow only aspect ratio preserving modes for mirroring.
+
+    // Try finding an exact match.
+    for (ModeInfoMap::const_iterator internal_it =
+         internal_output->mode_infos.begin();
+         internal_it != internal_output->mode_infos.end(); ++internal_it) {
+      const ModeInfo& internal_info = internal_it->second;
+      if (internal_info.width == external_info.width &&
+          internal_info.height == external_info.height &&
+          internal_info.interlaced == external_info.interlaced) {
+        internal_output->mirror_mode = internal_it->first;
+        external_output->mirror_mode = external_it->first;
+        return true;  // Mirror mode found.
+      }
+    }
+
+    // Try to create a matching internal output mode by panel fitting.
+    if (try_creating) {
+      // We can downscale by 1.125, and upscale indefinitely. Downscaling looks
+      // ugly, so, can fit == can upscale. Also, internal panels don't support
+      // fitting interlaced modes.
+      bool can_fit =
+          internal_native_info->width >= external_info.width &&
+          internal_native_info->height >= external_info.height &&
+          !external_info.interlaced;
+      if (can_fit) {
+        RRMode mode = external_it->first;
+        delegate_->AddOutputMode(internal_output->output, mode);
+        internal_output->mode_infos.insert(std::make_pair(mode, external_info));

[Daniel Erat, 2013/09/17 15:53:27]

(Side note: Apart from this line, which is the actual bug fix, I haven't made
any intentional changes to this method, apart from changing it to operate on
OutputSnapshots instead of using Xlib structures.)

+        internal_output->mirror_mode = mode;
+        external_output->mirror_mode = mode;
+        return true;  // Mirror mode created.
+      }
+    }
+  }
+
+  return false;
+}
+
 void OutputConfigurator::ConfigureOutputs() {
   configure_timer_.reset();
 
   delegate_->GrabServer();
-  std::vector<OutputSnapshot> outputs =
-      delegate_->GetOutputs(state_controller_);
+  std::vector<OutputSnapshot> outputs = GetOutputs();
   const OutputState new_state = GetOutputState(outputs, power_state_);
   const bool success = EnterStateOrFallBackToSoftwareMirroring(
       new_state, power_state_, outputs);
   delegate_->UngrabServer();
 
   NotifyObservers(success, new_state);
   delegate_->SendProjectingStateToPowerManager(IsProjecting(outputs));
 }
 
 void OutputConfigurator::NotifyObservers(bool success,
@@ skipping 262 matching lines @@
   float width_ratio = static_cast<float>(mirror_mode_info->width) /
       static_cast<float>(native_mode_info->width);
   float height_ratio = static_cast<float>(mirror_mode_info->height) /
       static_cast<float>(native_mode_info->height);
 
   area_ratio = width_ratio * height_ratio;
   return area_ratio;
 }
 
 }  // namespace chromeos