Revert of Use Window coordinates for IME composition bounds, auto resize

content/renderer/render_view_impl.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 "content/renderer/render_view_impl.h"
 
 #include <algorithm>
 #include <cmath>
 
 #include "base/auto_reset.h"
@@ skipping 2641 matching lines @@
     Send(new ViewHostMsg_RunFileChooser(routing_id_,
         file_chooser_completions_.front()->params));
   }
 }
 
 void RenderViewImpl::OnEnableAutoResize(const gfx::Size& min_size,
                                         const gfx::Size& max_size) {
   DCHECK(disable_scrollbars_size_limit_.IsEmpty());
   if (!webview())
     return;
-
   auto_resize_mode_ = true;
-  if (IsUseZoomForDSFEnabled()) {
-    webview()->enableAutoResizeMode(
-        gfx::ScaleToCeiledSize(min_size, device_scale_factor_),
-        gfx::ScaleToCeiledSize(max_size, device_scale_factor_));
-  } else {
-    webview()->enableAutoResizeMode(min_size, max_size);
-  }
+  webview()->enableAutoResizeMode(min_size, max_size);
 }
 
 void RenderViewImpl::OnDisableAutoResize(const gfx::Size& new_size) {
   DCHECK(disable_scrollbars_size_limit_.IsEmpty());
   if (!webview())
     return;
   auto_resize_mode_ = false;
   webview()->disableAutoResizeMode();
 
   if (!new_size.IsEmpty()) {
@@ skipping 564 matching lines @@
   return RenderWidget::GetTextInputType();
 }
 
 void RenderViewImpl::GetSelectionBounds(gfx::Rect* start, gfx::Rect* end) {
 #if defined(ENABLE_PLUGINS)
   if (focused_pepper_plugin_) {
     // TODO(kinaba) http://crbug.com/101101
     // Current Pepper IME API does not handle selection bounds. So we simply
     // use the caret position as an empty range for now. It will be updated
     // after Pepper API equips features related to surrounding text retrieval.
-    blink::WebRect caret(focused_pepper_plugin_->GetCaretBounds());
-    convertViewportToWindow(&caret);
+    gfx::Rect caret = focused_pepper_plugin_->GetCaretBounds();
     *start = caret;
     *end = caret;
     return;
   }
 #endif
   RenderWidget::GetSelectionBounds(start, end);
 }
 
 void RenderViewImpl::FocusChangeComplete() {
   RenderWidget::FocusChangeComplete();
   FOR_EACH_OBSERVER(RenderViewObserver, observers_, FocusChangeComplete());
 }
 
 void RenderViewImpl::GetCompositionCharacterBounds(
-    std::vector<gfx::Rect>* bounds_in_window) {
-  DCHECK(bounds_in_window);
-  bounds_in_window->clear();
+    std::vector<gfx::Rect>* bounds) {
+  DCHECK(bounds);
+  bounds->clear();
 
 #if defined(ENABLE_PLUGINS)
   if (focused_pepper_plugin_) {
     return;
   }
 #endif
 
   if (!webview())
     return;
   size_t start_offset = 0;
   size_t character_count = 0;
   if (!webview()->compositionRange(&start_offset, &character_count))
     return;
   if (character_count == 0)
     return;
 
   blink::WebFrame* frame = webview()->focusedFrame();
   if (!frame)
     return;
 
-  bounds_in_window->reserve(character_count);
+  bounds->reserve(character_count);
   blink::WebRect webrect;
   for (size_t i = 0; i < character_count; ++i) {
     if (!frame->firstRectForCharacterRange(start_offset + i, 1, webrect)) {
       DLOG(ERROR) << "Could not retrieve character rectangle at " << i;
-      bounds_in_window->clear();
+      bounds->clear();
       return;
     }
-    convertViewportToWindow(&webrect);
-    bounds_in_window->push_back(webrect);
+    bounds->push_back(webrect);
   }
 }
 
 void RenderViewImpl::GetCompositionRange(gfx::Range* range) {
 #if defined(ENABLE_PLUGINS)
   if (focused_pepper_plugin_) {
     return;
   }
 #endif
   RenderWidget::GetCompositionRange(range);
@@ skipping 405 matching lines @@
   if (IsUseZoomForDSFEnabled()) {
     compositor_->SetPaintedDeviceScaleFactor(device_scale_factor_);
     webview()->setZoomFactorForDeviceScaleFactor(
         device_scale_factor_);
   } else {
     webview()->setDeviceScaleFactor(device_scale_factor_);
   }
 }
 
 }  // namespace content