Move ash/ime to ui/chromeos/ime

ui/base/BUILD.gn

 # 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.
 
 import("//build/config/ui.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
 }
@@ skipping 877 matching lines @@
     ]
 
     deps += [
       "//ui/events",
       "//ui/events/platform",
     ]
   }
 
   if (is_chromeos) {
     sources += [
+      # TODO(pkotwicz): Move tests to their own target (crbug.com/432538) 
+      "../chromeos/ime/candidate_view_unittest.cc",
+      "../chromeos/ime/candidate_window_view_unittest.cc",
+      "../chromeos/ime/input_method_menu_item_unittest.cc",
+      "../chromeos/ime/input_method_menu_manager_unittest.cc",
       "../chromeos/network/network_state_notifier_unittest.cc",
       "../chromeos/touch_exploration_controller_unittest.cc",
     ]
     sources -= [
       "dragdrop/os_exchange_data_provider_aurax11_unittest.cc",
       "x/selection_requestor_unittest.cc",
     ]
     deps += [
       "//chromeos",
       "//ui/aura:test_support",
       "//ui/chromeos/resources",
       "//ui/chromeos/strings",
       "//ui/chromeos:ui_chromeos",
       "//ui/events:gesture_detection",
       "//ui/message_center:message_center",
+      "//ui/views",

[sky, 2014/11/14 17:56:04]

This is wrong and means there is a cycle between views and ui_base.

[oshima, 2014/11/14 19:52:11]

This is for the test target, so it won't introduce the cycle.

That's being said, I agree that even test target shouldn't depend no views, but
this target actually already depends on views (and aura) indirectly.
(ui_base_tests.gypi depends on ui_chrmoeos, which depends on views, and aura).

The plan is Peter will move these test to separate ui_chromeos_unittest and
remove these dependency.

+      "//ui/views:test_support",
     ]
   }
 }
 }
 # TODO(GYP) Mac (ui_unittest_bundle) and Android (ui_unittests_apk).