Treats the returned value from ui::TextInputClient as DIP

ui/base/ime/text_input_client.h

 // 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.
 
 #ifndef UI_BASE_IME_TEXT_INPUT_CLIENT_H_
 #define UI_BASE_IME_TEXT_INPUT_CLIENT_H_
 
 #include "base/basictypes.h"
 #include "base/i18n/rtl.h"
 #include "base/strings/string16.h"
@@ skipping 53 matching lines @@
   // TEXT_INPUT_TYPE_NONE at runtime.
   virtual ui::TextInputType GetTextInputType() const = 0;
 
   // Returns current text input mode. It could be changed and even becomes
   // TEXT_INPUT_MODE_DEFAULT at runtime.
   virtual ui::TextInputMode GetTextInputMode() const = 0;
 
   // Returns if the client supports inline composition currently.
   virtual bool CanComposeInline() const = 0;
 
-  // Returns current caret (insertion point) bounds relative to the screen
+  // Returns current caret (insertion point) bounds in the universal screen
   // coordinates. If there is selection, then the selection bounds will be
   // returned.
+  // Note: On Windows, the returned value is supposed to be DIP (Density
+  // Independent Pixel).
+  // TODO(ime): Have a clear spec whether the returned value is DIP or not.
+  // http://crbug.com/360334
   virtual gfx::Rect GetCaretBounds() const = 0;
 
-  // Retrieves the composition character boundary rectangle relative to the
+  // Retrieves the composition character boundary rectangle in the universal
   // screen coordinates. The |index| is zero-based index of character position
   // in composition text.
   // Returns false if there is no composition text or |index| is out of range.
   // The |rect| is not touched in the case of failure.
+  // Note: On Windows, the returned value is supposed to be DIP
+  // (Density Independent Pixel).
+  // TODO(ime): Have a clear spec whether the returned value is DIP or not.
+  // http://crbug.com/360334
   virtual bool GetCompositionCharacterBounds(uint32 index,
                                              gfx::Rect* rect) const = 0;
 
   // Returns true if there is composition text.
   virtual bool HasCompositionText() const = 0;
 
   // Document content operations ----------------------------------------------
 
   // Retrieves the UTF-16 based character range containing accessibled text in
   // the View. It must cover the composition and selection range.
@@ skipping 45 matching lines @@
 
   // Deletes the current selection plus the specified number of characters
   // before and after the selection or caret. This function should be used
   // instead of calling DeleteRange with GetSelectionRange, because
   // GetSelectionRange may not be the latest value due to asynchronous of IPC
   // between browser and renderer.
   virtual void ExtendSelectionAndDelete(size_t before, size_t after) = 0;
 
   // Ensure the caret is within |rect|.  |rect| is in screen coordinates and
   // may extend beyond the bounds of this TextInputClient.
+  // Note: On Windows, the returned value is supposed to be DIP (Density
+  // Independent Pixel).
+  // TODO(ime): Have a clear spec whether the returned value is DIP or not.
+  // http://crbug.com/360334
   virtual void EnsureCaretInRect(const gfx::Rect& rect) = 0;
 
   // Called when IME shows a candidate window.
   virtual void OnCandidateWindowShown() = 0;
   // Called when IME updates any appearance of the current candidate window.
   virtual void OnCandidateWindowUpdated() = 0;
   // Called when IME hides the candidate window.
   virtual void OnCandidateWindowHidden() = 0;
 };
 
 }  // namespace ui
 
 #endif  // UI_BASE_IME_TEXT_INPUT_CLIENT_H_