Index: content/browser/frame_host/navigator_impl.cc |
diff --git a/content/browser/frame_host/navigator_impl.cc b/content/browser/frame_host/navigator_impl.cc |
index 09f927e3793d8cba56b48111f6d02953393357b6..1247668afc0903fc43c00423266a38e3a0d5786b 100644 |
--- a/content/browser/frame_host/navigator_impl.cc |
+++ b/content/browser/frame_host/navigator_impl.cc |
@@ -766,18 +766,10 @@ void NavigatorImpl::RequestOpenURL( |
params.user_gesture = user_gesture; |
if (render_frame_host->web_ui()) { |
- // Web UI pages sometimes want to override the page transition type for |
- // link clicks (e.g., so the new tab page can specify AUTO_BOOKMARK for |
- // automatically generated suggestions). We don't override other types |
- // like TYPED because they have different implications (e.g., autocomplete). |
- if (ui::PageTransitionCoreTypeIs( |
- params.transition, ui::PAGE_TRANSITION_LINK)) |
- params.transition = render_frame_host->web_ui()->GetLinkTransitionType(); |
Charlie Reis
2016/11/04 23:31:59
Sanity check: No other actual WebUI pages rely on
Devlin
2016/11/05 01:23:17
No other WebUI ever sets a transition type (and th
|
- |
- // Note also that we hide the referrer for Web UI pages. We don't really |
- // want web sites to see a referrer of "chrome://blah" (and some |
- // chrome: URLs might have search terms or other stuff we don't want to |
- // send to the site), so we send no referrer. |
+ // Note that we hide the referrer for Web UI pages. We don't really want |
+ // web sites to see a referrer of "chrome://blah" (and some chrome: URLs |
+ // might have search terms or other stuff we don't want to send to the |
+ // site), so we send no referrer. |
params.referrer = Referrer(); |
// Navigations in Web UI pages count as browser-initiated navigations. |
@@ -831,17 +823,10 @@ void NavigatorImpl::RequestTransferURL( |
// navigation. See https://crbug.com/495161. |
bool is_renderer_initiated = true; |
if (render_frame_host->web_ui()) { |
- // Web UI pages sometimes want to override the page transition type for |
- // link clicks (e.g., so the new tab page can specify AUTO_BOOKMARK for |
- // automatically generated suggestions). We don't override other types |
- // like TYPED because they have different implications (e.g., autocomplete). |
- if (ui::PageTransitionCoreTypeIs(page_transition, ui::PAGE_TRANSITION_LINK)) |
- page_transition = render_frame_host->web_ui()->GetLinkTransitionType(); |
- |
- // Note also that we hide the referrer for Web UI pages. We don't really |
- // want web sites to see a referrer of "chrome://blah" (and some |
- // chrome: URLs might have search terms or other stuff we don't want to |
- // send to the site), so we send no referrer. |
+ // Note that we hide the referrer for Web UI pages. We don't really want |
+ // web sites to see a referrer of "chrome://blah" (and some chrome: URLs |
+ // might have search terms or other stuff we don't want to send to the |
+ // site), so we send no referrer. |
referrer_to_use = Referrer(); |
// Navigations in Web UI pages count as browser-initiated navigations. |