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

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

Issue 2393513004: Convert app banners to use Mojo. (Closed)
Patch Set: Remove WebAppBannerPromptResult 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 327ccfd7a8de28705e711b5d2d29e61d826b903c..a3dab6a864844db650231aa7a63244a6d8da8059 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 {
@@ -2039,14 +2040,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