touch: Return an enum from OnTouchEvent.

views/view.cc

 // Copyright (c) 2010 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 "views/view.h"
 
 #include <algorithm>
 #ifndef NDEBUG
 #include <iostream>
 #endif
@@ skipping 522 matching lines @@
       ConvertPointToScreen(this, &location);
       ShowContextMenu(location, true);
     }
   } else {
     OnMouseReleased(e, canceled);
   }
   // WARNING: we may have been deleted.
 }
 
 #if defined(TOUCH_UI)
-bool View::ProcessTouchEvent(const TouchEvent& e) {
+View::TouchStatus View::ProcessTouchEvent(const TouchEvent& e) {
   // TODO(rjkroege): Implement a grab scheme similar to as
   // as is found in MousePressed.
-  const bool result = OnTouchEvent(e);
-  return result;
+  const TouchStatus status = OnTouchEvent(e);

[sky, 2011/01/18 17:16:50]

return OnTouchEvent(e);
Also, we generally don't use const in cases like this.

[sadrul, 2011/01/18 18:48:20]

Noted, and fixed. Thanks.

+  return status;
 }
 #endif
 
 void View::AddChildView(View* v) {
   AddChildView(static_cast<int>(child_views_.size()), v);
 }
 
 void View::AddChildView(int index, View* v) {
   CHECK(v != this) << "You cannot add a view as its own child";
 
@@ skipping 729 matching lines @@
 void View::OnMouseMoved(const MouseEvent& e) {
 }
 
 void View::OnMouseEntered(const MouseEvent& e) {
 }
 
 void View::OnMouseExited(const MouseEvent& e) {
 }
 
 #if defined(TOUCH_UI)
-bool View::OnTouchEvent(const TouchEvent& event) {
+View::TouchStatus View::OnTouchEvent(const TouchEvent& event) {
   DVLOG(1) << "visited the OnTouchEvent";
-  return false;
+  return TOUCH_STATUS_UNKNOWN;
 }
 #endif
 
 void View::SetMouseHandler(View *new_mouse_handler) {
   // It is valid for new_mouse_handler to be NULL
   if (parent_)
     parent_->SetMouseHandler(new_mouse_handler);
 }
 
 void View::SetVisible(bool flag) {
@@ skipping 221 matching lines @@
   possible_drag = false;
   start_pt = gfx::Point();
 }
 
 void View::DragInfo::PossibleDrag(const gfx::Point& p) {
   possible_drag = true;
   start_pt = p;
 }
 
 }  // namespace