Re-enable TouchSelectionOriginatingFromWebpageTest with some fixes and logs

ui/views/touchui/touch_selection_controller_impl.cc

 // Copyright (c) 2013 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 "ui/views/touchui/touch_selection_controller_impl.h"
 
 #include "base/time/time.h"
 #include "grit/ui_resources.h"
 #include "grit/ui_strings.h"
 #include "ui/base/resource/resource_bundle.h"
@@ skipping 175 matching lines @@
     canvas->DrawImageInt(*GetHandleImage()->ToImageSkia(),
         kSelectionHandleHorizPadding, cursor_height());
   }
 
   virtual void OnGestureEvent(ui::GestureEvent* event) OVERRIDE {
     event->SetHandled();
     switch (event->type()) {
       case ui::ET_GESTURE_SCROLL_BEGIN:
         widget_->SetCapture(this);
         controller_->SetDraggingHandle(this);
-        drag_offset_ = event->y() - cursor_height() -
+        drag_offset_ = event->y() - cursor_height() +

[varunjain, 2013/08/30 16:52:08]

While you are here, mind adding a comment on drag_offset_? You can refer to the
ASCII figure at the top of the file and say something like drag_offset_ is the
offset between scroll event position and the cursor position reported to the
client_view_.

[mohsen, 2013/08/30 18:26:39]

Done.

             kSelectionHandleVerticalDragOffset;
         break;
       case ui::ET_GESTURE_SCROLL_UPDATE: {
         gfx::Point drag_pos(event->location().x(),
             event->location().y() - drag_offset_);
         controller_->SelectionHandleDragged(drag_pos);
         break;
       }
       case ui::ET_GESTURE_SCROLL_END:
       case ui::ET_SCROLL_FLING_START:
@@ skipping 323 matching lines @@
 }
 
 ui::TouchSelectionController* ViewsTouchSelectionControllerFactory::create(
     ui::TouchEditable* client_view) {
   if (switches::IsTouchEditingEnabled())
     return new views::TouchSelectionControllerImpl(client_view);
   return NULL;
 }
 
 }  // namespace views