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

Unified Diff: third_party/WebKit/Source/modules/webmidi/MIDIAccessInitializer.h

Issue 2116763002: Reland: Web MIDI: use mojom::blink::PermissionService directly to ask permission (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: for second attempt Created 4 years, 5 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: third_party/WebKit/Source/modules/webmidi/MIDIAccessInitializer.h
diff --git a/third_party/WebKit/Source/modules/webmidi/MIDIAccessInitializer.h b/third_party/WebKit/Source/modules/webmidi/MIDIAccessInitializer.h
index ed1e4e5e77c58664779907ecc1227ae2447711f8..9bb871dc2a9375427ca0758c638e3d28b6c80d36 100644
--- a/third_party/WebKit/Source/modules/webmidi/MIDIAccessInitializer.h
+++ b/third_party/WebKit/Source/modules/webmidi/MIDIAccessInitializer.h
@@ -12,6 +12,8 @@
#include "modules/webmidi/MIDIAccessorClient.h"
#include "modules/webmidi/MIDIOptions.h"
#include "modules/webmidi/MIDIPort.h"
+#include "third_party/WebKit/public/platform/modules/permissions/permission.mojom-blink.h"
+#include "third_party/WebKit/public/platform/modules/permissions/permission_status.mojom-blink.h"
#include "wtf/Vector.h"
#include <memory>
@@ -47,7 +49,7 @@ public:
return resolver->start();
}
- ~MIDIAccessInitializer() override;
+ ~MIDIAccessInitializer() override = default;
// Eager finalization to allow dispose() operation access
// other (non eager) heap objects.
@@ -61,23 +63,22 @@ public:
void didStartSession(bool success, const String& error, const String& message) override;
void didReceiveMIDIData(unsigned portIndex, const unsigned char* data, size_t length, double timeStamp) override { }
- void resolvePermission(bool allowed);
- SecurityOrigin* getSecurityOrigin() const;
-
private:
MIDIAccessInitializer(ScriptState*, const MIDIOptions&);
ExecutionContext* getExecutionContext() const;
ScriptPromise start();
- void dispose();
void contextDestroyed() override;
+ void onPermissionsUpdated(mojo::WTFArray<mojom::blink::PermissionStatus>);
+ void onPermissionUpdated(mojom::blink::PermissionStatus);
+
std::unique_ptr<MIDIAccessor> m_accessor;
Vector<PortDescriptor> m_portDescriptors;
MIDIOptions m_options;
- bool m_hasBeenDisposed;
- bool m_permissionResolved;
+
+ mojom::blink::PermissionServicePtr m_permissionService;
};
} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698