Allow the user to tap the selected editable again after rotation to trigger another zoom/center anim

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index ca1bb655aab848afc9f9f5954d22426a36961c66..eabb94b62a264233c37a885903077e3bdb72fb8e 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -665,6 +665,7 @@ RenderViewImpl::RenderViewImpl(RenderViewImplParams* params)
       cached_has_main_frame_horizontal_scrollbar_(false),
       cached_has_main_frame_vertical_scrollbar_(false),
       has_scrolled_focused_editable_node_into_rect_(false),
+      is_orientation_changed_(false),
       notification_provider_(NULL),
       push_messaging_dispatcher_(NULL),
       geolocation_dispatcher_(NULL),
@@ -1262,7 +1263,8 @@ void RenderViewImpl::OnMoveCaret(const gfx::Point& point) {
 void RenderViewImpl::OnScrollFocusedEditableNodeIntoRect(
     const gfx::Rect& rect) {
   if (has_scrolled_focused_editable_node_into_rect_ &&
-      rect == rect_for_scrolled_focused_editable_node_) {
+      rect == rect_for_scrolled_focused_editable_node_ &&
+      !is_orientation_changed_) {
     return;
   }
 
@@ -1270,6 +1272,7 @@ void RenderViewImpl::OnScrollFocusedEditableNodeIntoRect(
   if (!element.isNull() && IsEditableNode(element)) {
     rect_for_scrolled_focused_editable_node_ = rect;
     has_scrolled_focused_editable_node_into_rect_ = true;
+    is_orientation_changed_ = false;
     webview()->scrollFocusedNodeIntoRect(rect);
   }
 }
@@ -3347,6 +3350,7 @@ void RenderViewImpl::OnMediaPlayerActionAt(const gfx::Point& location,
 
 void RenderViewImpl::OnOrientationChangeEvent(int orientation) {
   // Screen has rotated. 0 = default (portrait), 90 = one turn right, and so on.
+  is_orientation_changed_ = true;

[jdduke (slow), 2014/04/15 14:36:00]

Why not just set |has_scrolled_focused_editable_node_into_rect_| as false here?

[jdduke (slow), 2014/04/16 14:51:40]

Until we have a solution to the multiple resize problem (crbug.com/363182),
let's go ahead and just set |has_scrolled_focused_editable_node_into_rect_ =
false| here.  Then at least, if the user taps the editable again, it will scroll
and zoom the editable into view properly.  The rest of the changes won't be
necessary.

   FOR_EACH_OBSERVER(RenderViewObserver,
                     observers_,
                     OrientationChangeEvent(orientation));