Use the virtual URL of the page in the Page Info Bubble

Use the virtual URL of the page in the Page Info Bubble

BUG=612710
Committed: https://crrev.com/903170ee962abea0e5b71fc3836ef5fad8c2ea27
Cr-Commit-Position: refs/heads/master@{#397020}

[meacer, 2016-05-27 23:46:54]

Description was changed from

==========
Use virtual URL instead of URL when showing page info bubble.

BUG=612939
==========

to

==========
Use the virtual URL of the page in the Page Info Bubble

BUG=612939
==========

[meacer, 2016-05-27 23:49:22]

meacer@chromium.org changed reviewers:
+ sky@chromium.org

[meacer, 2016-05-27 23:49:22]

sky: can you please take a look?

[sky, 2016-05-31 13:23:32]

I can't view the associated bug. Can you make it so I can, or send me
direct email about the issue?

On Fri, May 27, 2016 at 4:49 PM,  <meacer@chromium.org> wrote:
> Reviewers: sky
> CL: https://codereview.chromium.org/1995813002/
>
> Message:
> sky: can you please take a look?
>
> Description:
> Use the virtual URL of the page in the Page Info Bubble
>
> BUG=612939
>
> Base URL: https://chromium.googlesource.com/chromium/src.git@master
>
> Affected files (+34, -33 lines):
> M chrome/browser/chromeos/login/ui/simple_web_view_dialog.h
> M chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
> M chrome/browser/ui/browser_dialogs.h
> M chrome/browser/ui/browser_window.h
> M chrome/browser/ui/cocoa/browser_window_cocoa.h
> M chrome/browser/ui/cocoa/browser_window_cocoa.mm
> M chrome/browser/ui/cocoa/location_bar/location_icon_decoration.mm
> M
> chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
> M
>
chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
> M chrome/browser/ui/views/browser_dialogs_views_mac.cc
> M chrome/browser/ui/views/frame/browser_view.h
> M chrome/browser/ui/views/frame/browser_view.cc
> M chrome/browser/ui/views/frame/web_app_left_header_view_ash.cc
> M chrome/browser/ui/views/location_bar/location_bar_view.h
> M chrome/browser/ui/views/location_bar/location_icon_view.cc
> M chrome/browser/ui/views/toolbar/toolbar_view.h
> M chrome/browser/ui/views/toolbar/toolbar_view.cc
> M chrome/test/base/test_browser_window.h
>
>
> Index: chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
> diff --git a/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
> b/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
> index
>
7c8b192d0f497c2a4668d9ca9e056dffc9f63f6e..38bae20e4cab5a80fd1b49ab2258a11b064ed8a7
> 100644
> --- a/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
> +++ b/chrome/browser/chromeos/login/ui/simple_web_view_dialog.cc
> @@ -303,7 +303,7 @@
> SimpleWebViewDialog::GetContentSettingBubbleModelDelegate() {
>
> void SimpleWebViewDialog::ShowWebsiteSettings(
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info) {
> NOTIMPLEMENTED();
> // TODO (markusheintz@): implement this
> Index: chrome/browser/chromeos/login/ui/simple_web_view_dialog.h
> diff --git a/chrome/browser/chromeos/login/ui/simple_web_view_dialog.h
> b/chrome/browser/chromeos/login/ui/simple_web_view_dialog.h
> index
>
3e3a7892f3deafb86a328e2f99be7026e3d898bc..1c415c55909e6c5391c147292e63c7cb4a63a275
> 100644
> --- a/chrome/browser/chromeos/login/ui/simple_web_view_dialog.h
> +++ b/chrome/browser/chromeos/login/ui/simple_web_view_dialog.h
> @@ -85,7 +85,7 @@ class SimpleWebViewDialog : public views::ButtonListener,
> override;
> void ShowWebsiteSettings(
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info)
> override;
>
> Index: chrome/browser/ui/browser_dialogs.h
> diff --git a/chrome/browser/ui/browser_dialogs.h
> b/chrome/browser/ui/browser_dialogs.h
> index
>
968999d49c37cb66c99b0db8e0e246661ba6e8d3..c45ceb2e57e9aee9ea2efec9196b41fcaeefd763
> 100644
> --- a/chrome/browser/ui/browser_dialogs.h
> +++ b/chrome/browser/ui/browser_dialogs.h
> @@ -98,7 +98,7 @@ void ShowWebsiteSettingsBubbleViewsAtPoint(
> const gfx::Point& anchor_point,
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info);
>
> // Show a Views bookmark bubble at the given point. This occurs when the
> Index: chrome/browser/ui/browser_window.h
> diff --git a/chrome/browser/ui/browser_window.h
> b/chrome/browser/ui/browser_window.h
> index
>
43837dd1013dc07308226ea54ae6d2868ad79aad..42b7483aa7ccb4c42c36cedc6fcea0d39d762375
> 100644
> --- a/chrome/browser/ui/browser_window.h
> +++ b/chrome/browser/ui/browser_window.h
> @@ -279,14 +279,14 @@ class BrowserWindow : public ui::BaseWindow {
> // indicating that it's time to redraw everything.
> virtual void UserChangedTheme() = 0;
>
> - // Shows the website settings using the specified information. |url| is
> the
> - // url of the page/frame the info applies to, |ssl| is the SSL information
> for
> - // that page/frame. If |show_history| is true, a section showing how many
> - // times that URL has been visited is added to the page info.
> + // Shows the website settings using the specified information.
> |virtual_url|
> + // is the virtual url of the page/frame the info applies to, |ssl| is the
> SSL
> + // information for that page/frame. If |show_history| is true, a section
> + // showing how many times that URL has been visited is added to the page
> info.
> virtual void ShowWebsiteSettings(
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo&
> security_info) = 0;
>
> Index: chrome/browser/ui/cocoa/browser_window_cocoa.h
> diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.h
> b/chrome/browser/ui/cocoa/browser_window_cocoa.h
> index
>
f03c48b8730d38d15c6507871bc36966e2421eea..d5d354b648b757f0bf874e11d5943570e5282d21
> 100644
> --- a/chrome/browser/ui/cocoa/browser_window_cocoa.h
> +++ b/chrome/browser/ui/cocoa/browser_window_cocoa.h
> @@ -128,7 +128,7 @@ class BrowserWindowCocoa
> void ShowWebsiteSettings(
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info)
> override;
> void ShowAppMenu() override;
> Index: chrome/browser/ui/cocoa/browser_window_cocoa.mm
> diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm
> b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
> index
>
779a79d2a0e3d7d980d363ae85af56e5a706c75a..b1e11470030258bdcd4b759cca9f5850396ab797
> 100644
> --- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm
> +++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm
> @@ -671,9 +671,9 @@ void BrowserWindowCocoa::UserChangedTheme() {
> void BrowserWindowCocoa::ShowWebsiteSettings(
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info) {
> - WebsiteSettingsUIBridge::Show(window(), profile, web_contents, url,
> + WebsiteSettingsUIBridge::Show(window(), profile, web_contents,
> virtual_url,
> security_info);
> }
>
> Index: chrome/browser/ui/cocoa/location_bar/location_icon_decoration.mm
> diff --git
> a/chrome/browser/ui/cocoa/location_bar/location_icon_decoration.mm
> b/chrome/browser/ui/cocoa/location_bar/location_icon_decoration.mm
> index
>
549b4a29b1b6d501bc09f54ac10ee7366167dcf8..c48a346f86217956def7c6070977c1eca88e26ac
> 100644
> --- a/chrome/browser/ui/cocoa/location_bar/location_icon_decoration.mm
> +++ b/chrome/browser/ui/cocoa/location_bar/location_icon_decoration.mm
> @@ -124,7 +124,7 @@ bool LocationIconDecoration::OnMousePressed(NSRect
> frame, NSPoint location) {
> ChromeSecurityStateModelClient::FromWebContents(tab);
> DCHECK(security_model_client);
>
> - chrome::ShowWebsiteSettings(browser, tab, nav_entry->GetURL(),
> + chrome::ShowWebsiteSettings(browser, tab, nav_entry->GetVirtualURL(),
> security_model_client->GetSecurityInfo());
> return true;
> }
> Index:
> chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
> diff --git
>
a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
>
b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
> index
>
69a1a99846ffaaf468f80bb2fac6773f58a1f807..cea66d366d3eab472d9cbb03034f46c7496b5985
> 100644
> ---
>
a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
> +++
>
b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.h
> @@ -113,7 +113,7 @@ class WebsiteSettingsUIBridge : public
> content::WebContentsObserver,
> // Creates a |WebsiteSettingsBubbleController| and displays the UI. |parent|
> // is the currently active window. |profile| points to the currently active
> // profile. |web_contents| points to the WebContents that wraps the
> currently
> - // active tab. |url| is the GURL of the currently active
> + // active tab. |virtual_url| is the virtual GURL of the currently active
> // tab. |security_info| is the
> // |security_state::SecurityStateModel::SecurityInfo| of
> // the connection to the website in the currently active tab.
> @@ -121,7 +121,7 @@ class WebsiteSettingsUIBridge : public
> content::WebContentsObserver,
> gfx::NativeWindow parent,
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info);
>
> void set_bubble_controller(
> Index:
>
chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
> diff --git
>
a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
>
b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
> index
>
19e83f29513e30c7a0f443ecdeb3645a8d83beef..bdf3826376a83a29641e07e7eb20337d9769e046
> 100644
> ---
>
a/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
> +++
>
b/chrome/browser/ui/cocoa/website_settings/website_settings_bubble_controller.mm
> @@ -1113,12 +1113,12 @@ void WebsiteSettingsUIBridge::Show(
> gfx::NativeWindow parent,
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info) {
> if (chrome::ToolkitViewsWebUIDialogsEnabled()) {
> chrome::ShowWebsiteSettingsBubbleViewsAtPoint(
> gfx::ScreenPointFromNSPoint(AnchorPointForWindow(parent)), profile,
> - web_contents, url, security_info);
> + web_contents, virtual_url, security_info);
> return;
> }
>
> Index: chrome/browser/ui/views/browser_dialogs_views_mac.cc
> diff --git a/chrome/browser/ui/views/browser_dialogs_views_mac.cc
> b/chrome/browser/ui/views/browser_dialogs_views_mac.cc
> index
>
73dd8e6fb29c6e6941f83bc311d9793599e426bb..f30448add01b2e69586eb22d5804f244625825ef
> 100644
> --- a/chrome/browser/ui/views/browser_dialogs_views_mac.cc
> +++ b/chrome/browser/ui/views/browser_dialogs_views_mac.cc
> @@ -24,26 +24,26 @@ void ShowWebsiteSettingsBubbleViewsAtPoint(
> const gfx::Point& anchor_point,
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info) {
> WebsiteSettingsPopupView::ShowPopup(
> - nullptr, gfx::Rect(anchor_point, gfx::Size()), profile, web_contents, url,
> - security_info);
> + nullptr, gfx::Rect(anchor_point, gfx::Size()), profile, web_contents,
> + virtual_url, security_info);
> }
>
> void ShowBookmarkBubbleViewsAtPoint(const gfx::Point& anchor_point,
> gfx::NativeView parent,
> bookmarks::BookmarkBubbleObserver* observer,
> Browser* browser,
> - const GURL& url,
> + const GURL& virtual_url,
> bool already_bookmarked) {
> // The Views dialog may prompt for sign in.
> std::unique_ptr<BubbleSyncPromoDelegate> delegate(
> new BookmarkBubbleSignInDelegate(browser));
>
> - BookmarkBubbleView::ShowBubble(nullptr, gfx::Rect(anchor_point,
> gfx::Size()),
> - parent, observer, std::move(delegate),
> - browser->profile(), url, already_bookmarked);
> + BookmarkBubbleView::ShowBubble(
> + nullptr, gfx::Rect(anchor_point, gfx::Size()), parent, observer,
> + std::move(delegate), browser->profile(), virtual_url, already_bookmarked);
> }
>
> ui::TableModel* ShowTaskManagerViews(Browser* browser) {
> Index: chrome/browser/ui/views/frame/browser_view.cc
> diff --git a/chrome/browser/ui/views/frame/browser_view.cc
> b/chrome/browser/ui/views/frame/browser_view.cc
> index
>
2eb4e5135e4df4ad6b7d16d114f9312699730ab9..62b520d646888bd41757a10edb23a45786203fa7
> 100644
> --- a/chrome/browser/ui/views/frame/browser_view.cc
> +++ b/chrome/browser/ui/views/frame/browser_view.cc
> @@ -1304,7 +1304,7 @@ void BrowserView::UserChangedTheme() {
> void BrowserView::ShowWebsiteSettings(
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info) {
> // Some browser windows have a location icon embedded in the frame. Try to
> // use that if it exists. If it doesn't exist, use the location icon from
> @@ -1314,7 +1314,7 @@ void BrowserView::ShowWebsiteSettings(
> popup_anchor = GetLocationBarView()->location_icon_view()->GetImageView();
>
> WebsiteSettingsPopupView::ShowPopup(popup_anchor, gfx::Rect(), profile,
> - web_contents, url, security_info);
> + web_contents, virtual_url, security_info);
> }
>
> void BrowserView::ShowAppMenu() {
> Index: chrome/browser/ui/views/frame/browser_view.h
> diff --git a/chrome/browser/ui/views/frame/browser_view.h
> b/chrome/browser/ui/views/frame/browser_view.h
> index
>
a2ac8bc97385755c74d5c4f2c4e8379f6f1a265b..40275c48841ba7d54b6731394024f0e6d78d8444
> 100644
> --- a/chrome/browser/ui/views/frame/browser_view.h
> +++ b/chrome/browser/ui/views/frame/browser_view.h
> @@ -335,7 +335,7 @@ class BrowserView : public BrowserWindow,
> void ShowWebsiteSettings(
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info)
> override;
> void ShowAppMenu() override;
> Index: chrome/browser/ui/views/frame/web_app_left_header_view_ash.cc
> diff --git a/chrome/browser/ui/views/frame/web_app_left_header_view_ash.cc
> b/chrome/browser/ui/views/frame/web_app_left_header_view_ash.cc
> index
>
2fb92c3e0cd20452ca9806f219efcc254bb720fb..82b7e8961a23b170fd54de5ff4fba6ba4ddc5f70
> 100644
> --- a/chrome/browser/ui/views/frame/web_app_left_header_view_ash.cc
> +++ b/chrome/browser/ui/views/frame/web_app_left_header_view_ash.cc
> @@ -94,6 +94,6 @@ void WebAppLeftHeaderView::ShowWebsiteSettings() const {
> DCHECK(security_model_client);
>
> chrome::ShowWebsiteSettings(browser_view_->browser(), tab,
> - nav_entry->GetURL(),
> + nav_entry->GetVirtualURL(),
> security_model_client->GetSecurityInfo());
> }
> Index: chrome/browser/ui/views/location_bar/location_bar_view.h
> diff --git a/chrome/browser/ui/views/location_bar/location_bar_view.h
> b/chrome/browser/ui/views/location_bar/location_bar_view.h
> index
>
0a7163906552dcfef4dca8bfe6216227cc6f805e..dcf35343b4817ed2cd83e3ecfdf044ae83e4fd05
> 100644
> --- a/chrome/browser/ui/views/location_bar/location_bar_view.h
> +++ b/chrome/browser/ui/views/location_bar/location_bar_view.h
> @@ -96,7 +96,7 @@ class LocationBarView : public LocationBar,
> // Shows permissions and settings for the given web contents.
> virtual void ShowWebsiteSettings(
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo&
> security_info) = 0;
>
> Index: chrome/browser/ui/views/location_bar/location_icon_view.cc
> diff --git a/chrome/browser/ui/views/location_bar/location_icon_view.cc
> b/chrome/browser/ui/views/location_bar/location_icon_view.cc
> index
>
df9c61c33675933f20fab5474f8935b468090ce2..d2027eba190827ede2a61c8d10f0fc17a4c39243
> 100644
> --- a/chrome/browser/ui/views/location_bar/location_icon_view.cc
> +++ b/chrome/browser/ui/views/location_bar/location_icon_view.cc
> @@ -146,7 +146,7 @@ bool LocationIconView::OnActivate() {
> DCHECK(model_client);
>
> location_bar_->delegate()->ShowWebsiteSettings(
> - contents, entry->GetURL(), model_client->GetSecurityInfo());
> + contents, entry->GetVirtualURL(), model_client->GetSecurityInfo());
> return true;
> }
>
> Index: chrome/browser/ui/views/toolbar/toolbar_view.cc
> diff --git a/chrome/browser/ui/views/toolbar/toolbar_view.cc
> b/chrome/browser/ui/views/toolbar/toolbar_view.cc
> index
>
ce2d355eb89eb27dc6af285020a3a5cd9655ecb2..129796ae9a9f56f98962953adb213b2a04d77342
> 100644
> --- a/chrome/browser/ui/views/toolbar/toolbar_view.cc
> +++ b/chrome/browser/ui/views/toolbar/toolbar_view.cc
> @@ -394,9 +394,10 @@ ToolbarView::GetContentSettingBubbleModelDelegate() {
>
> void ToolbarView::ShowWebsiteSettings(
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info) {
> - chrome::ShowWebsiteSettings(browser_, web_contents, url, security_info);
> + chrome::ShowWebsiteSettings(browser_, web_contents, virtual_url,
> + security_info);
> }
>
> PageActionImageView* ToolbarView::CreatePageActionImageView(
> Index: chrome/browser/ui/views/toolbar/toolbar_view.h
> diff --git a/chrome/browser/ui/views/toolbar/toolbar_view.h
> b/chrome/browser/ui/views/toolbar/toolbar_view.h
> index
>
46f14bb291e2d8566241bca5c9796420c8294e52..122ea168ad6164a7637660289a3e53d2dafa6b04
> 100644
> --- a/chrome/browser/ui/views/toolbar/toolbar_view.h
> +++ b/chrome/browser/ui/views/toolbar/toolbar_view.h
> @@ -120,7 +120,7 @@ class ToolbarView : public views::AccessiblePaneView,
> override;
> void ShowWebsiteSettings(
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info)
> override;
>
> Index: chrome/test/base/test_browser_window.h
> diff --git a/chrome/test/base/test_browser_window.h
> b/chrome/test/base/test_browser_window.h
> index
>
2c75121fa8826fddc382045b31f64defb57e6434..374d5a06533cafc60872b39198a161dbe0fc9b40
> 100644
> --- a/chrome/test/base/test_browser_window.h
> +++ b/chrome/test/base/test_browser_window.h
> @@ -119,7 +119,7 @@ class TestBrowserWindow : public BrowserWindow {
> void ShowWebsiteSettings(
> Profile* profile,
> content::WebContents* web_contents,
> - const GURL& url,
> + const GURL& virtual_url,
> const security_state::SecurityStateModel::SecurityInfo& security_info)
> override {}
> void CutCopyPaste(int command_id) override {}
>
>

-- 
You received this message because you are subscribed to the Google Groups
"Chromium-reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[meacer, 2016-05-31 17:11:40]

Description was changed from

==========
Use the virtual URL of the page in the Page Info Bubble

BUG=612939
==========

to

==========
Use the virtual URL of the page in the Page Info Bubble

BUG=612710
==========

[meacer, 2016-05-31 17:12:56]

Ah, sorry, I put the wrong bug number in this CL. The correct one is
https://crbug.com/612710 which isn't view restricted.

[sky, 2016-05-31 19:16:53]

LGTM

[meacer, 2016-05-31 19:24:54]

The CQ bit was checked by meacer@chromium.org

[commit-bot: I haz the power, 2016-05-31 19:25:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995813002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995813002/40001

[commit-bot: I haz the power, 2016-05-31 19:42:31]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-05-31 19:42:33]

Try jobs failed on following builders:
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[meacer, 2016-05-31 23:47:52]

The CQ bit was checked by meacer@chromium.org

[meacer, 2016-05-31 23:47:52]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/1995813002/#ps60001
(title: "Fix Mac build")

[commit-bot: I haz the power, 2016-05-31 23:48:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1995813002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1995813002/60001

[commit-bot: I haz the power, 2016-06-01 01:46:35]

Description was changed from

==========
Use the virtual URL of the page in the Page Info Bubble

BUG=612710
==========

to

==========
Use the virtual URL of the page in the Page Info Bubble

BUG=612710
==========

[commit-bot: I haz the power, 2016-06-01 01:46:39]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-06-01 01:48:53]

Description was changed from

==========
Use the virtual URL of the page in the Page Info Bubble

BUG=612710
==========

to

==========
Use the virtual URL of the page in the Page Info Bubble

BUG=612710

Committed: https://crrev.com/903170ee962abea0e5b71fc3836ef5fad8c2ea27
Cr-Commit-Position: refs/heads/master@{#397020}
==========

[commit-bot: I haz the power, 2016-06-01 01:48:55]

Patchset 4 (id:??) landed as
https://crrev.com/903170ee962abea0e5b71fc3836ef5fad8c2ea27
Cr-Commit-Position: refs/heads/master@{#397020}