Clarify that WebContents::GetLastCommittedURL returns the virtual URL.

content/public/browser/web_contents.h

Index: content/public/browser/web_contents.h
diff --git a/content/public/browser/web_contents.h b/content/public/browser/web_contents.h
index 90b647efcbf43132033298205d9ce8a0837945d8..7474725b61227aeadc3587769150ab619239e2bf 100644
--- a/content/public/browser/web_contents.h
+++ b/content/public/browser/web_contents.h
@@ -210,9 +210,9 @@ class WebContents : public PageNavigator,
   // page (active) but the security context is of the old page (last committed).
   virtual const GURL& GetVisibleURL() const = 0;
 
-  // Gets the last committed URL. It represents the current page that is
-  // displayed in this WebContents. It represents the current security
-  // context.
+  // Gets the last committed URL. This is the URL that would be displayed in the

[Charlie Reis, 2016/06/20 17:33:06]

We probably want to be more explicit about the difference between "visible,"
"virtual," and "displayed," relative to GetVisibleURL.

Instead, let's say:
Gets the virtual URL of the last committed page in this WebContents.  Virtual
URLs are meant to be displayed to the user (e.g., they include the
"view-source:" prefix for view source URLs, unlike NavigationEntry::GetURL). 
The last committed page is the current security context and the content that is
actually displayed within the tab.  See also GetVisibleURL above, which may
differ from this URL.

And let's update the beginning of the GetVisibleURL comment to say:
Gets the virtual URL currently being displayed...

[dominickn, 2016/06/20 23:56:06]

Done.

+  // URL bar for the current page displayed in this WebContents. It represents
+  // the current security context.
   virtual const GURL& GetLastCommittedURL() const = 0;
 
   // Return the currently active RenderProcessHost and RenderViewHost. Each of