PDFs viewed inside a <webview> should navigate the same as PDFs in tabs.

chrome/browser/resources/pdf/pdf.js

Index: chrome/browser/resources/pdf/pdf.js
diff --git a/chrome/browser/resources/pdf/pdf.js b/chrome/browser/resources/pdf/pdf.js
index c8753e14a6207dca698cd16144c70cb57dba82f9..86f00660064b37e2f559c85474f89594bb6b40d0 100644
--- a/chrome/browser/resources/pdf/pdf.js
+++ b/chrome/browser/resources/pdf/pdf.js
@@ -44,12 +44,12 @@ function getFilenameFromURL(url) {
  * @param {string} url The url to be opened in the current tab.
  */
 function onNavigateInCurrentTab(url) {
-  // Prefer the tabs API because it can navigate from one file:// URL to
-  // another.
-  if (chrome.tabs)

[Sam McNally, 2015/09/18 01:03:47]

Could we change this to
  if (chrome.tabs && this.browserApi_.getStreamInfo().tabId != -1)
    chrome.tabs.update(this.browserApi_.getStreamInfo().tabId, {url: url});
  else
    window.location.href = url;

This would allow file url links to work outside webviews and webview navigations
to not leak to the last-active tab. As far as I can tell, webviews can't
navigate to file urls so this should cover everything.

[wjmaclean, 2015/09/18 16:20:45]

Yes, but in this context |this| is a "Navigator" and does not have a
|browserApi_| member. I've plumbed one through, though a tad willy-nilly, as I'm
not sure what other implications this may have. Is there anything else that
needs to be done?

It does seem to solve both our cases.

I had originally attempted to look at chrome.tabs.getTabId, which didn't work
(for fairly obvious reasons), but I wasn't aware of browserApi, so thanks for
pointing it out.

-    chrome.tabs.update({url: url});
-  else
-    window.location.href = url;
+  // chrome.tabs.update should not be used here, since it blocks the ability
+  // of an app embedding a PDF viewer inside a webview to perform in-place
+  // navigations (e.g. to the current tab). In these cases chrome.tabs.update
+  // just refers the navigation to the current browser, causing the navigation
+  // to occur in a new tab in the browser window, which is unexpected.
+  window.location.href = url;
 }
 
 /**