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

Side by Side Diff: third_party/WebKit/public/web/WebMediaIPH.h

Issue 2943983003: chrome/blink: Add functionality for in-product help for media elements. (Closed)
Patch Set: Created 3 years, 6 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef WebMediaIPH_h
chrishtr 2017/06/23 00:39:45 InProductHelp, here and everywhere else in the CL.
Khushal 2017/06/28 05:32:38 Done.
6 #define WebMediaIPH_h
7
8 #include "public/platform/WebCommon.h"
9
10 namespace blink {
11 struct WebRect;
12
13 class BLINK_EXPORT WebMediaIPHClient {
14 public:
15 virtual ~WebMediaIPHClient() {}
16
17 // Notifies that the IPH widget was hidden.
18 virtual void DidHideMediaIPHWidget() = 0;
19 };
20
21 // WebMediaIPH provides the functionality to display in-product help UI for
22 // media elements.
23 class BLINK_EXPORT WebMediaIPH {
chrishtr 2017/06/23 00:39:45 Can you adjust add these methods to WebViewClient
Khushal 2017/06/23 02:11:06 I'm just following the advice on RenderViewImpl wh
chrishtr 2017/06/27 18:22:20 Ok then move them to WebFrameClient?
24 public:
25 virtual ~WebMediaIPH() {}
26
27 // Binds to the client to be notified of changes to the IPH UI.
28 virtual void BindToClient(WebMediaIPHClient*) = 0;
29
30 // Requests that an IPH widget be shown at the provided location.
31 virtual void ShowMediaDownloadIPH(const WebRect&) = 0;
32
33 // Requests that the IPH widget being shown currently is dismissed.
34 virtual void HideMediaDownloadIPH() = 0;
35 };
36
37 } // namespace blink
38
39 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698