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

Side by Side Diff: ios/chrome/browser/ui/reading_list/offline_page_native_content.h

Issue 2627093003: Reuse context menu in StaticHTMLViewController (Closed)
Patch Set: fix DEPS 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef IOS_CHROME_BROWSER_UI_READING_LIST_OFFLINE_PAGE_NATIVE_CONTENT_H_ 5 #ifndef IOS_CHROME_BROWSER_UI_READING_LIST_OFFLINE_PAGE_NATIVE_CONTENT_H_
6 #define IOS_CHROME_BROWSER_UI_READING_LIST_OFFLINE_PAGE_NATIVE_CONTENT_H_ 6 #define IOS_CHROME_BROWSER_UI_READING_LIST_OFFLINE_PAGE_NATIVE_CONTENT_H_
7 7
8 #import "ios/chrome/browser/ui/static_content/static_html_native_content.h" 8 #import "ios/chrome/browser/ui/static_content/static_html_native_content.h"
9 9
10 class GURL; 10 class GURL;
11 11
12 namespace web { 12 namespace web {
13 class BrowserState; 13 class BrowserState;
14 class WebState; 14 class WebState;
15 } 15 }
16 16
17 @protocol ContextMenuDelegate;
17 @protocol UrlLoader; 18 @protocol UrlLoader;
18 19
19 // A |StaticHtmlNativeContent| that displays offline pages located in the 20 // A |StaticHtmlNativeContent| that displays offline pages located in the
20 // application data folder that have been distilled by DOMdistiller. 21 // application data folder that have been distilled by DOMdistiller.
21 @interface OfflinePageNativeContent : StaticHtmlNativeContent 22 @interface OfflinePageNativeContent : StaticHtmlNativeContent
22 23
23 // Initialization method. 24 // Initialization method.
24 // |loader| is the loader to use to follow navigation. Cannot be nil. 25 // |loader| is the loader to use to follow navigation. Cannot be nil.
25 // |browserState| is the user browser state and must not be null. 26 // |browserState| is the user browser state and must not be null.
26 // |URL| is the url of the page. The format of the URL must be 27 // |URL| is the url of the page. The format of the URL must be
27 // chrome://offline/distillation_id/page.html 28 // chrome://offline/distillation_id/page.html
28 // If |URL| contain a virtual URL in its query params, this will be returned by 29 // If |URL| contain a virtual URL in its query params, this will be returned by
29 // the |OfflinePageNativeContent virtualURL| method. 30 // the |OfflinePageNativeContent virtualURL| method.
30 - (instancetype)initWithLoader:(id<UrlLoader>)loader 31 - (instancetype)initWithLoader:(id<UrlLoader>)loader
31 browserState:(web::BrowserState*)browserState 32 browserState:(web::BrowserState*)browserState
32 webState:(web::WebState*)webState 33 webState:(web::WebState*)webState
33 URL:(const GURL&)URL NS_DESIGNATED_INITIALIZER; 34 URL:(const GURL&)URL
35 contextMenuDelegate:(id<ContextMenuDelegate>)contextMenuDelegate
Eugene But (OOO till 7-30) 2017/01/13 07:16:26 Would it make sense to change this to property ins
Olivier 2017/01/13 10:20:25 Removed it. Added to the existing delegate.
36 NS_DESIGNATED_INITIALIZER;
34 37
35 - (instancetype)initWithLoader:(id<UrlLoader>)loader 38 - (instancetype)initWithLoader:(id<UrlLoader>)loader
36 staticHTMLViewController:(StaticHtmlViewController*)HTMLViewController 39 staticHTMLViewController:(StaticHtmlViewController*)HTMLViewController
37 URL:(const GURL&)URL NS_UNAVAILABLE; 40 URL:(const GURL&)URL NS_UNAVAILABLE;
38 41
39 @end 42 @end
40 43
41 #endif // IOS_CHROME_BROWSER_UI_READING_LIST_OFFLINE_PAGE_NATIVE_CONTENT_H_ 44 #endif // IOS_CHROME_BROWSER_UI_READING_LIST_OFFLINE_PAGE_NATIVE_CONTENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698