Enable file path watcher for Android

base/files/file_path_watcher_linux.cc

 // Copyright (c) 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 "base/files/file_path_watcher.h"
 
 #include <errno.h>
 #include <string.h>
 #include <sys/inotify.h>
 #include <sys/ioctl.h>
@@ skipping 355 matching lines @@
   DCHECK(MessageLoopForIO::current());
 
   set_message_loop(base::MessageLoopProxy::current());
   delegate_ = delegate;
   target_ = path;
   MessageLoop::current()->AddDestructionObserver(this);
 
   std::vector<FilePath::StringType> comps;
   target_.GetComponents(&comps);
   DCHECK(!comps.empty());
-  for (std::vector<FilePath::StringType>::const_iterator comp(++comps.begin());
-       comp != comps.end(); ++comp) {
+  std::vector<FilePath::StringType>::const_iterator comp = comps.begin();
+  for (comp++; comp != comps.end(); comp++) {

[Mattias Nissler (ping if slow), 2012/08/27 12:53:02]

Style guide says to use preincrement:
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Preinc...

Also, you no longer need braces, as the loop body is only a single line now.

     watches_.push_back(WatchEntry(InotifyReader::kInvalidWatch, *comp));
   }
   watches_.push_back(WatchEntry(InotifyReader::kInvalidWatch,
                                 FilePath::StringType()));
   return UpdateWatches();
 }
 
 void FilePathWatcherImpl::Cancel() {
   if (!delegate_) {
     // Watch was never called, or the |message_loop_| thread is already gone.
@@ skipping 86 matching lines @@
 }
 
 }  // namespace
 
 FilePathWatcher::FilePathWatcher() {
   impl_ = new FilePathWatcherImpl();
 }
 
 }  // namespace files
 }  // namespace base