Re-introduce ScrollY trace event.

cc/layers/layer_impl.cc

 // Copyright 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 "cc/layers/layer_impl.h"
 
 #include "base/json/json_reader.h"
 #include "base/strings/stringprintf.h"
 #include "base/trace_event/trace_event.h"
 #include "base/trace_event/trace_event_argument.h"
@@ skipping 1304 matching lines @@
   if (scrollbar_layer->orientation() == HORIZONTAL) {
     float visible_ratio = clip_rect.width() / scroll_rect.width();
     scrollbar_needs_animation |=
         scrollbar_layer->SetCurrentPos(current_offset.x());
     scrollbar_needs_animation |=
         scrollbar_layer->SetMaximum(scroll_rect.width() - clip_rect.width());
     scrollbar_needs_animation |=
         scrollbar_layer->SetVisibleToTotalLengthRatio(visible_ratio);
   } else {
     float visible_ratio = clip_rect.height() / scroll_rect.height();
-    scrollbar_needs_animation |=
+    bool y_offset_did_change =
         scrollbar_layer->SetCurrentPos(current_offset.y());
+    scrollbar_needs_animation |= y_offset_did_change;
     scrollbar_needs_animation |=
         scrollbar_layer->SetMaximum(scroll_rect.height() - clip_rect.height());
     scrollbar_needs_animation |=
         scrollbar_layer->SetVisibleToTotalLengthRatio(visible_ratio);
+    // Record the current_offset.y() value so we can make sure
+    // BasicMouseWheelSmoothScrollGesture has proper scroll curves.

[enne (OOO), 2015/03/25 17:21:33]

What is BasicMouseWheelSmoothScrollGesture? I can't find any reference to that
in Chromium.

[wjmaclean, 2015/03/25 20:30:32]

Revised the comment to reflect the rationale given in the associated bug, namely
the desire to evaluate scroll latency via tracing.

+    if (y_offset_did_change && layer_tree_impl()->IsActiveTree())

[wjmaclean, 2015/03/25 20:30:32]

Will limit this to the InnerViewportScrollLayer() as per Paul's suggestion.

+      TRACE_EVENT1("cc", "scroll_offset_y", this->id(), current_offset.y());
   }
   if (scrollbar_needs_animation) {
     layer_tree_impl()->set_needs_update_draw_properties();
     // TODO(wjmaclean) The scrollbar animator for the pinch-zoom scrollbars
     // should activate for every scroll on the main frame, not just the
     // scrolls that move the pinch virtual viewport (i.e. trigger from
     // either inner or outer viewport).
     if (scrollbar_animation_controller_) {
       // Non-overlay scrollbars shouldn't trigger animations.
       if (scrollbar_layer->is_overlay_scrollbar())
@@ skipping 277 matching lines @@
   gfx::Transform scaled_draw_transform =
       draw_properties_.target_space_transform;
   scaled_draw_transform.Scale(SK_MScalar1 / scale, SK_MScalar1 / scale);
   gfx::Size scaled_content_bounds =
       gfx::ToCeiledSize(gfx::ScaleSize(content_bounds(), scale));
   return MathUtil::MapEnclosingClippedRect(scaled_draw_transform,
                                            gfx::Rect(scaled_content_bounds));
 }
 
 }  // namespace cc