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

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

Issue 2576843002: [win] Create ModuleDatabase and ModuleEventSinkImpl. (Closed)
Patch Set: Fix grt's nits on patchset 8. 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
new file mode 100644
index 0000000000000000000000000000000000000000..f06fe849f2d078fede9a862f1872b009b4015759
--- /dev/null
+++ b/chrome/browser/conflicts/module_event_sink_impl_win.h
@@ -0,0 +1,74 @@
+// Copyright 2016 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_;
sky 2017/01/05 00:30:45 base::Time?
chrisha 2017/01/06 20:35:47 Responded earlier.
+
+ // 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_

Powered by Google App Engine
This is Rietveld 408576698