Calls display_configuration_controller to rotate screen in accelerator_controller_delegate_aura

chrome/browser/extensions/display_info_provider_chromeos.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 "chrome/browser/extensions/display_info_provider_chromeos.h"
 
 #include <stdint.h>
 
 #include "ash/display/display_configuration_controller.h"
 #include "ash/display/resolution_notification_controller.h"
@@ skipping 457 matching lines @@
       display::Screen::GetScreen()->GetPrimaryDisplay();
 
   if (!ValidateParamsForDisplay(info, target, display_manager, primary.id(),
                                 error)) {
     return false;
   }
 
   // Process 'isPrimary' parameter.
   if (info.is_primary && *info.is_primary && target.id() != primary.id()) {
     display_configuration_controller->SetPrimaryDisplayId(
         display_id, true /* user_action */);

[oshima, 2017/03/01 00:47:31]

I believe all these user_actions aren't necessary either. Can you clean them up
in a separate CL?

[wutao, 2017/03/01 01:06:35]

Will clean this up in a separate CL.

   }
 
   // Process 'mirroringSourceId' parameter.
   if (info.mirroring_source_id) {
     bool mirror = !info.mirroring_source_id->empty();
     display_configuration_controller->SetMirrorMode(mirror,
                                                     true /* user_action */);
   }
 
   // Process 'overscan' parameter.
   if (info.overscan) {
     display_manager->SetOverscanInsets(
         display_id, gfx::Insets(info.overscan->top, info.overscan->left,
                                 info.overscan->bottom, info.overscan->right));
   }
 
   // Process 'rotation' parameter.
   if (info.rotation) {
     display_configuration_controller->SetDisplayRotation(
         display_id, DegreesToRotation(*info.rotation),
-        display::Display::ROTATION_SOURCE_ACTIVE, true /* user_action */);

[oshima, 2017/03/01 00:47:31]

Thanks, we changed this (used to be false) when we decided to use the API to
implement setting but didn't notice that we could clean this up.

[wutao, 2017/03/01 01:06:35]

Acknowledged.

+        display::Display::ROTATION_SOURCE_ACTIVE);
   }
 
   // Process new display origin parameters.
   gfx::Point new_bounds_origin = target.bounds().origin();
   if (info.bounds_origin_x)
     new_bounds_origin.set_x(*info.bounds_origin_x);
   if (info.bounds_origin_y)
     new_bounds_origin.set_y(*info.bounds_origin_y);
 
   if (new_bounds_origin != target.bounds().origin()) {
@@ skipping 374 matching lines @@
     touch_calibrator_.reset(new chromeos::TouchCalibratorController);
   return touch_calibrator_.get();
 }
 
 // static
 DisplayInfoProvider* DisplayInfoProvider::Create() {
   return new DisplayInfoProviderChromeOS();
 }
 
 }  // namespace extensions