Invalidate the page state when restoring WebUIs

chrome/browser/ui/webui/md_history_ui.cc

 // Copyright 2015 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.
 
 #include "chrome/browser/ui/webui/md_history_ui.h"
 
 #include "base/command_line.h"
 #include "build/build_config.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/signin/signin_manager_factory.h"
@@ skipping 14 matching lines @@
 #include "components/search/search.h"
 #include "components/signin/core/browser/signin_manager.h"
 #include "components/strings/grit/components_strings.h"
 #include "content/public/browser/web_ui.h"
 #include "content/public/browser/web_ui_data_source.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 
 namespace {
 
-content::WebUIDataSource* CreateMdHistoryUIHTMLSource(Profile* profile) {
+content::WebUIDataSource* CreateMdHistoryUIHTMLSource(Profile* profile,
+                                                      bool use_test_title) {
   PrefService* prefs = profile->GetPrefs();
 
   content::WebUIDataSource* source =
       content::WebUIDataSource::Create(chrome::kChromeUIHistoryHost);
 
   // Localized strings (alphabetical order).
   source->AddLocalizedString("bookmarked", IDS_HISTORY_ENTRY_BOOKMARKED);
   source->AddLocalizedString("cancel", IDS_CANCEL);
   source->AddLocalizedString("clearBrowsingData",
                              IDS_CLEAR_BROWSING_DATA_TITLE);
@@ skipping 43 matching lines @@
   source->AddLocalizedString("removeFromHistory", IDS_HISTORY_REMOVE_PAGE);
   source->AddLocalizedString("removeSelected",
                              IDS_HISTORY_REMOVE_SELECTED_ITEMS);
   source->AddLocalizedString("searchPrompt", IDS_MD_HISTORY_SEARCH_PROMPT);
   source->AddLocalizedString("searchResult", IDS_HISTORY_SEARCH_RESULT);
   source->AddLocalizedString("searchResults", IDS_HISTORY_SEARCH_RESULTS);
   source->AddLocalizedString("signInButton", IDS_MD_HISTORY_SIGN_IN_BUTTON);
   source->AddLocalizedString("signInPromo", IDS_MD_HISTORY_SIGN_IN_PROMO);
   source->AddLocalizedString("signInPromoDesc",
                              IDS_MD_HISTORY_SIGN_IN_PROMO_DESC);
-  source->AddLocalizedString("title", IDS_HISTORY_TITLE);
+  if (use_test_title)
+    source->AddString("title", "MD History");
+  else
+    source->AddLocalizedString("title", IDS_HISTORY_TITLE);
 
   source->AddString(
       "sidebarFooter",
       l10n_util::GetStringFUTF16(
           IDS_HISTORY_OTHER_FORMS_OF_HISTORY,
           l10n_util::GetStringUTF16(
               IDS_SETTINGS_CLEAR_DATA_WEB_HISTORY_URL_IN_HISTORY)));
 
   bool allow_deleting_history =
       prefs->GetBoolean(prefs::kAllowDeletingBrowserHistory);
@@ skipping 74 matching lines @@
 #endif  // BUILDFLAG(USE_VULCANIZE)
 
   source->SetDefaultResource(IDR_MD_HISTORY_HISTORY_HTML);
   source->SetJsonPath("strings.js");
 
   return source;
 }
 
 }  // namespace
 
+bool MdHistoryUI::use_test_title = false;

[Dan Beam, 2016/09/21 06:19:17]

should this technically be g_use_test_title_?

+
 MdHistoryUI::MdHistoryUI(content::WebUI* web_ui) : WebUIController(web_ui) {
   Profile* profile = Profile::FromWebUI(web_ui);
   web_ui->AddMessageHandler(new BrowsingHistoryHandler());
   web_ui->AddMessageHandler(new MetricsHandler());
 
   if (search::IsInstantExtendedAPIEnabled()) {
     web_ui->AddMessageHandler(new browser_sync::ForeignSessionHandler());
     web_ui->AddMessageHandler(new HistoryLoginHandler());
   }
 
-  content::WebUIDataSource::Add(profile, CreateMdHistoryUIHTMLSource(profile));
+  content::WebUIDataSource::Add(
+      profile,
+      CreateMdHistoryUIHTMLSource(profile, MdHistoryUI::use_test_title));

[Dan Beam, 2016/09/21 06:19:17]

I don't technically think you need this MdHistoryUI prefix

 }
 
 MdHistoryUI::~MdHistoryUI() {}
 
 // static
 bool MdHistoryUI::IsEnabled(Profile* profile) {
   return base::FeatureList::IsEnabled(features::kMaterialDesignHistory) &&
          !base::CommandLine::ForCurrentProcess()->HasSwitch(
              switches::kHistoryEnableGroupByDomain) &&
          !profile->IsSupervised();
 }
 
 // static
 void MdHistoryUI::SetEnabledForTesting(bool enabled) {
   std::unique_ptr<base::FeatureList> feature_list(new base::FeatureList);
 
   const std::string& name = features::kMaterialDesignHistory.name;
   feature_list->InitializeFromCommandLine(enabled ? name : "",
                                           enabled ? "" : name);
   base::FeatureList::ClearInstanceForTesting();
   base::FeatureList::SetInstance(std::move(feature_list));
 }
 
 // static
 base::RefCountedMemory* MdHistoryUI::GetFaviconResourceBytes(
     ui::ScaleFactor scale_factor) {
   return ResourceBundle::GetSharedInstance().LoadDataResourceBytesForScale(
       IDR_HISTORY_FAVICON, scale_factor);
 }