Don't reset scroll position of contents on every ScrollView layout.

ui/views/controls/scroll_view.cc

Index: ui/views/controls/scroll_view.cc
diff --git a/ui/views/controls/scroll_view.cc b/ui/views/controls/scroll_view.cc
index 8a0c2590e8d2eae11644a63eddd61455fc6c0fa6..86a8cc3f75039ffb42c2db19e94c53ab97490459 100644
--- a/ui/views/controls/scroll_view.cc
+++ b/ui/views/controls/scroll_view.cc
@@ -313,8 +313,7 @@ void ScrollView::Layout() {
       content_width = std::max(content_width - GetScrollBarWidth(), 0);
       content_height = contents()->GetHeightForWidth(content_width);
     }
-    if (contents()->bounds().size() != gfx::Size(content_width, content_height))
-      contents()->SetBounds(0, 0, content_width, content_height);
+    contents()->SetSize(gfx::Size(content_width, content_height));

[sky, 2016/11/16 20:12:57]

It would be nice if we were consistent and used contents_ and not contents(). I
think it's this block that uses contents() rahter than contents_.

[Evan Stade, 2016/11/17 01:14:00]

(also done)

   }
 
   // Most views will want to auto-fit the available space. Most of them want to