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

Unified Diff: third_party/WebKit/Source/web/WebLocalFrameImpl.cpp

Issue 2393513004: Convert app banners to use Mojo. (Closed)
Patch Set: Rebase Created 4 years, 2 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/web/WebLocalFrameImpl.cpp
diff --git a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
index 91cd54ae1ec8bff1696e166fe3112d5eb6211eea..beb6d93da6f25572e2ffc38b21f23fd7041cd405 100644
--- a/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
+++ b/third_party/WebKit/Source/web/WebLocalFrameImpl.cpp
@@ -228,6 +228,7 @@
#include "wtf/PtrUtil.h"
#include <algorithm>
#include <memory>
+#include <utility>
namespace blink {
@@ -2050,14 +2051,16 @@ void WebLocalFrameImpl::sendOrientationChangeEvent() {
}
void WebLocalFrameImpl::willShowInstallBannerPrompt(
- int requestId,
+ mojo::ScopedMessagePipeHandle serviceHandle,
+ mojo::ScopedMessagePipeHandle eventHandle,
const WebVector<WebString>& platforms,
- WebAppBannerPromptReply* reply) {
+ mojom::AppBannerPromptReply* reply) {
if (!frame())
return;
AppBannerController::willShowInstallBannerPrompt(
- requestId, client()->appBannerClient(), frame(), platforms, reply);
+ frame(), std::move(serviceHandle), std::move(eventHandle), platforms,
+ reply);
}
void WebLocalFrameImpl::requestRunTask(WebSuspendableTask* task) const {

Powered by Google App Engine
This is Rietveld 408576698