Remove interface_impl.h and error_handler.h usage from chromium

content/browser/permissions/permission_service_impl.h

 // 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.
 
 #ifndef CONTENT_BROWSER_PERMISSIONS_PERMISSION_SERVICE_IMPL_H_
 #define CONTENT_BROWSER_PERMISSIONS_PERMISSION_SERVICE_IMPL_H_
 
 #include "base/id_map.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "content/browser/permissions/permission_service_context.h"
 #include "content/common/permission_service.mojom.h"
 #include "third_party/mojo/src/mojo/public/cpp/bindings/binding.h"
-#include "third_party/mojo/src/mojo/public/cpp/bindings/error_handler.h"
 
 namespace content {
 
 enum class PermissionType;
 
 // Implements the PermissionService Mojo interface.
 // This service can be created from a RenderFrameHost or a RenderProcessHost.
 // It is owned by a PermissionServiceContext.
 // It receives at PermissionServiceContext instance when created which allows it
 // to have some information about the current context. That enables the service
 // to know whether it can show UI and have knowledge of the associated
 // WebContents for example.
-class PermissionServiceImpl : public PermissionService,
-                              public mojo::ErrorHandler {
+class PermissionServiceImpl : public PermissionService {
  public:
   ~PermissionServiceImpl() override;
 
   // Clear pending operations currently run by the service. This will be called
   // by PermissionServiceContext when it will need the service to clear its
   // state for example, if the frame changes.
   void CancelPendingOperations();
 
  protected:
   friend PermissionServiceContext;
@@ skipping 38 matching lines @@
                          const PermissionStatusCallback& callback) override;
   void RevokePermission(PermissionName permission,
                         const mojo::String& origin,
                         const PermissionStatusCallback& callback) override;
   void GetNextPermissionChange(
       PermissionName permission,
       const mojo::String& origin,
       PermissionStatus last_known_status,
       const PermissionStatusCallback& callback) override;
 
-  // mojo::ErrorHandler
-  void OnConnectionError() override;

[yzshen1, 2015/07/29 00:01:02]

I wonder who does set_error_handler() using this class as an error handler. I
couldn't find it.

+  void OnConnectionError();
 
   void OnRequestPermissionResponse(int request_id, PermissionStatus status);
 
   PermissionStatus GetPermissionStatusFromName(PermissionName permission,
                                                const GURL& origin);
   PermissionStatus GetPermissionStatusFromType(PermissionType type,
                                                const GURL& origin);
   void ResetPermissionStatus(PermissionType type, const GURL& origin);
 
   void OnPermissionStatusChanged(int pending_subscription_id,
                                  PermissionStatus status);
 
   RequestsMap pending_requests_;
   SubscriptionsMap pending_subscriptions_;
   // context_ owns |this|.
   PermissionServiceContext* context_;
   mojo::Binding<PermissionService> binding_;
   base::WeakPtrFactory<PermissionServiceImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(PermissionServiceImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_PERMISSIONS_PERMISSION_SERVICE_IMPL_H_