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

Side by Side Diff: chrome/browser/ssl/captive_portal_blocking_page.h

Issue 318213002: Add custom interstitial for captive portals. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Expand browser tests Created 6 years 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright 2014 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 CHROME_BROWSER_SSL_CAPTIVE_PORTAL_BLOCKING_PAGE_H_
6 #define CHROME_BROWSER_SSL_CAPTIVE_PORTAL_BLOCKING_PAGE_H_
7
8 #include <string>
9
10 #include "base/macros.h"
11 #include "chrome/browser/interstitials/security_interstitial_page.h"
12
13 class GURL;
14
15 namespace content{
16 class WebContents;
17 }
18
19 // This class is responsible for showing/hiding the interstitial page that is
20 // shown when a captive portal and an SSL error is detected.
21 // It deletes itself when the interstitial page is closed.
22 //
23 // This class should only be used on the UI thread because its implementation
24 // uses captive_portal::CaptivePortalService, which can only be accessed on the
25 // UI thread.
mmenke 2014/12/09 22:42:26 + "Only used when ENABLE_CAPTIVE_PORTAL_DETECTION
meacer 2014/12/10 22:48:02 Done.
26 class CaptivePortalBlockingPage : public SecurityInterstitialPage {
27 public:
28 // Interstitial type, for testing.
29 static const void* kTypeForTesting;
30
31 CaptivePortalBlockingPage(content::WebContents* web_contents,
32 const GURL& request_url);
33 ~CaptivePortalBlockingPage() override;
34
35 // SecurityInterstitialPage method:
36 const void* GetTypeForTesting() const override;
37
38 protected:
39 // SecurityInterstitialPage implementation:
mmenke 2014/12/09 22:42:26 nit: Should be consistent. You use "method:" abo
meacer 2014/12/10 22:48:02 Done.
40 void PopulateInterstitialStrings(
41 base::DictionaryValue* load_time_data) override;
42 bool ShouldCreateNewNavigation() const override;
43
44 // InterstitialPageDelegate implementation.
45 void CommandReceived(const std::string& command) override;
46
47 private:
48 DISALLOW_COPY_AND_ASSIGN(CaptivePortalBlockingPage);
49 };
50
51 #endif // CHROME_BROWSER_SSL_CAPTIVE_PORTAL_BLOCKING_PAGE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698