Makes IViewManager::DeleteNode take the server_change_id

mojo/services/public/cpp/view_manager/lib/view_manager_synchronizer.cc

 // Copyright 2014 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 "mojo/services/public/cpp/view_manager/lib/view_manager_synchronizer.h"
 
 #include "base/bind.h"
 #include "base/message_loop/message_loop.h"
 #include "mojo/public/cpp/application/application.h"
 #include "mojo/public/cpp/application/connect.h"
@@ skipping 217 matching lines @@
  public:
   DestroyViewTreeNodeTransaction(Id node_id,
                                  ViewManagerSynchronizer* synchronizer)
       : ViewManagerTransaction(synchronizer),
         node_id_(node_id) {}
   virtual ~DestroyViewTreeNodeTransaction() {}
 
  private:
   // Overridden from ViewManagerTransaction:
   virtual void DoCommit() OVERRIDE {
-    GetAndAdvanceNextServerChangeId();
-    service()->DeleteNode(node_id_, ActionCompletedCallback());
+    service()->DeleteNode(node_id_,
+                          GetAndAdvanceNextServerChangeId(),
+                          ActionCompletedCallback());
   }
   virtual void DoActionCompleted(bool success) OVERRIDE {
     // TODO(beng): recovery?
   }
 
   const Id node_id_;
   DISALLOW_COPY_AND_ASSIGN(DestroyViewTreeNodeTransaction);
 };
 
 class AddChildTransaction : public ViewManagerTransaction {
@@ skipping 580 matching lines @@
 // ViewManager, public:
 
 // static
 void ViewManager::Create(Application* application,
                          ViewManagerDelegate* delegate) {
   application->AddService<ViewManagerSynchronizer>(delegate);
 }
 
 }  // namespace view_manager
 }  // namespace mojo