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

Unified Diff: content/browser/web_contents/web_contents_impl.h

Issue 2086423005: Using WebContents::UpdateTitleForEntry() instead of NavigationEntry::SetTitle() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Improving comment and removing the NOTREACHED. Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: content/browser/web_contents/web_contents_impl.h
diff --git a/content/browser/web_contents/web_contents_impl.h b/content/browser/web_contents/web_contents_impl.h
index dca061eb4860f88f288e60f3deb76983d3d818f6..c99ec23578b0b192a42750af0c2c90dfa3f46e14 100644
--- a/content/browser/web_contents/web_contents_impl.h
+++ b/content/browser/web_contents/web_contents_impl.h
@@ -278,6 +278,8 @@ class CONTENT_EXPORT WebContentsImpl
bool IsFullAccessibilityModeForTesting() const override;
const PageImportanceSignals& GetPageImportanceSignals() const override;
const base::string16& GetTitle() const override;
+ void UpdateTitleForEntry(NavigationEntry* entry,
+ const base::string16& title) override;
int32_t GetMaxPageID() override;
int32_t GetMaxPageIDForSiteInstance(SiteInstance* site_instance) override;
SiteInstanceImpl* GetSiteInstance() const override;
@@ -803,9 +805,6 @@ class CONTENT_EXPORT WebContentsImpl
// So |find_request_manager_| can be accessed for testing.
friend class FindRequestManagerTest;
- // So InterstitialPageImpl can access SetIsLoading.
- friend class InterstitialPageImpl;
-
// TODO(brettw) TestWebContents shouldn't exist!
friend class TestWebContents;
@@ -997,17 +996,6 @@ class CONTENT_EXPORT WebContentsImpl
// have begun, to prevent any races in updating RenderView::next_page_id.
void UpdateMaxPageIDIfNecessary(RenderViewHost* rvh);
- // Saves the given title to the navigation entry and does associated work. It
- // will update history and the view for the new title, and also synthesize
- // titles for file URLs that have none (so we require that the URL of the
- // entry already be set).
- //
- // This is used as the backend for state updates, which include a new title,
- // or the dedicated set title message. It returns true if the new title is
- // different and was therefore updated.
- bool UpdateTitleForEntry(NavigationEntryImpl* entry,
- const base::string16& title);
-
// Helper for CreateNewWidget/CreateNewFullscreenWidget.
void CreateNewWidget(int32_t render_process_id,
int32_t route_id,

Powered by Google App Engine
This is Rietveld 408576698