Properly decode IDN in interstitials

chrome/browser/interstitials/security_interstitial_page.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_PAGE_H_
 #define CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_PAGE_H_
 
 #include "base/strings/string16.h"
 #include "content/public/browser/interstitial_page_delegate.h"
 #include "url/gurl.h"
@@ skipping 21 matching lines @@
     CMD_OPEN_HELP_CENTER = 3,
     CMD_OPEN_DIAGNOSTIC = 4,
     // Primary button actions
     CMD_RELOAD = 5,
     CMD_OPEN_DATE_SETTINGS = 6,
     CMD_OPEN_LOGIN = 7,
     // Safe Browsing Extended Reporting
     CMD_DO_REPORT = 8,
     CMD_DONT_REPORT = 9,
     CMD_OPEN_REPORTING_PRIVACY = 10,
+    // Used by tests
+    CMD_TEXT_FOUND = 11,
+    CMD_TEXT_NOT_FOUND = 12

[meacer, 2015/03/25 01:08:40]

How about making these negative values? That way the page could just ignore them
by checking for values >=0. It's also possible that we'll add new actions which
will shift these values in the future (I don't think that's going to break
anything, but better to be safe).

[estark, 2015/03/25 04:23:49]

Done.

   };
 
   SecurityInterstitialPage(content::WebContents* web_contents,
                            const GURL& url);
   ~SecurityInterstitialPage() override;
 
   // Creates an interstitial and shows it.
   virtual void Show();
 
   // Prevents creating the actual interstitial view for testing.
@@ skipping 23 matching lines @@
   // Once shown, |interstitial_page| takes ownership of this
   // SecurityInterstitialPage instance.
   content::InterstitialPage* interstitial_page_;
   // Whether the interstitial should create a view.
   bool create_view_;
 
   DISALLOW_COPY_AND_ASSIGN(SecurityInterstitialPage);
 };
 
 #endif  // CHROME_BROWSER_INTERSTITIALS_SECURITY_INTERSTITIAL_PAGE_H_