Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1157)

Unified Diff: chrome/browser/conflicts/module_event_sink_impl_win.h

Issue 2634073002: Revert of [win] Create ModuleDatabase and ModuleEventSinkImpl. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/conflicts/module_event_sink_impl_win.h
diff --git a/chrome/browser/conflicts/module_event_sink_impl_win.h b/chrome/browser/conflicts/module_event_sink_impl_win.h
deleted file mode 100644
index 5f5b7ced00e94ee3bfab732b59ce75d1a5613b6f..0000000000000000000000000000000000000000
--- a/chrome/browser/conflicts/module_event_sink_impl_win.h
+++ /dev/null
@@ -1,74 +0,0 @@
-// Copyright 2017 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 CHROME_BROWSER_CONFLICTS_MODULE_EVENT_SINK_IMPL_WIN_H_
-#define CHROME_BROWSER_CONFLICTS_MODULE_EVENT_SINK_IMPL_WIN_H_
-
-#include <stdint.h>
-
-#include "base/process/process_handle.h"
-#include "chrome/common/conflicts/module_event_sink_win.mojom.h"
-#include "content/public/common/process_type.h"
-
-class ModuleDatabase;
-
-// Implementation of the mojom::ModuleEventSink interface. This is the endpoint
-// in the browser process. This redirects calls to the singleton ModuleDatabase
-// object.
-class ModuleEventSinkImpl : public mojom::ModuleEventSink {
- public:
- // Creates a service endpoint that forwards notifications from the remote
- // |process| of the provided |process_type| to the provided |module_database|.
- // The |module_database| must outlive this object.
- ModuleEventSinkImpl(base::ProcessHandle process,
- content::ProcessType process_type,
- ModuleDatabase* module_database);
- ~ModuleEventSinkImpl() override;
-
- // Factory function for use with service_manager::InterfaceRegistry. This
- // creates a concrete implementation of mojom::ModuleDatabase interface in the
- // current process, for the remote process represented by the provided
- // |request|. This should only be called on the UI thread.
- static void Create(base::ProcessHandle process,
- content::ProcessType process_type,
- ModuleDatabase* module_database,
- mojom::ModuleEventSinkRequest request);
-
- // mojom::ModuleEventSink implementation:
- void OnModuleEvent(mojom::ModuleEventType event_type,
- uint64_t load_address) override;
-
- bool in_error() const { return in_error_; }
-
- private:
- friend class ModuleEventSinkImplTest;
-
- // OnModuleEvent disptaches to these two functions depending on the event
- // type.
- void OnModuleLoad(uint64_t load_address);
- void OnModuleUnload(uint64_t load_address);
-
- // A handle to the process on the other side of the pipe.
- base::ProcessHandle process_;
-
- // The module database this forwards events to. The |module_database| must
- // outlive this object.
- ModuleDatabase* module_database_;
-
- // The process ID of the remote process on the other end of the pipe. This is
- // forwarded along to the ModuleDatabase for each call.
- uint32_t process_id_;
-
- // The creation time of the process. Combined with process_id_ this uniquely
- // identifies a process.
- uint64_t creation_time_;
-
- // Indicates whether or not this connection is in an error mode. If true then
- // all communication from the remote client is silently dropped.
- bool in_error_;
-
- DISALLOW_COPY_AND_ASSIGN(ModuleEventSinkImpl);
-};
-
-#endif // CHROME_BROWSER_CONFLICTS_MODULE_EVENT_SINK_IMPL_WIN_H_
« no previous file with comments | « chrome/browser/conflicts/module_database_win_unittest.cc ('k') | chrome/browser/conflicts/module_event_sink_impl_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698