Record the PDF and top level URL when the PDF plugin crashes.

chrome/browser/resources/pdf/browser_api.js

Index: chrome/browser/resources/pdf/browser_api.js
diff --git a/chrome/browser/resources/pdf/browser_api.js b/chrome/browser/resources/pdf/browser_api.js
index 53f14b5460e770d538a8ff1eacd57811510b8bcd..dacbac3b984f72cc141e8fe34c07724c923535a3 100644
--- a/chrome/browser/resources/pdf/browser_api.js
+++ b/chrome/browser/resources/pdf/browser_api.js
@@ -154,6 +154,12 @@ function createBrowserApiForMimeHandlerView() {
   }).then(function(streamInfo) {
     let manageZoom = !streamInfo.embedded && streamInfo.tabId != -1;

[raymes, 2016/09/05 04:11:47]

It does feel a bit weird that we pass in whether the the PDF is embedded in the
streamInfo but we don't pass the URL of the embedder (we could even consider
merging those two parameters - empty implies top level frame). Perhaps we should
just do that? I think it would happen here:
https://cs.chromium.org/chromium/src/chrome/browser/renderer_host/chrome_reso...

This would reduce the indirection here (this file is already full of indirection
as I'm sure you've noticed!).

Saying that if it's a lot more complicated to do that, this seems ok. I'll let
you take a look and decide.

     return new Promise(function(resolve, reject) {
+      if (streamInfo.tabId != -1) {
+        chrome.tabs.get(streamInfo.tabId, function(tab) {
+          streamInfo.tabUrl = tab.url;
+          resolve();
+        });
+      }

[raymes, 2016/09/05 04:11:47]

Hmm this doesn't look quite right to me. resolve() will get called here and then
again after we setZoomSettings, is that right? I think we may want to chain
another .then() before we get to this function which adds the new parameter in.

I might be missing something though!

[Lei Zhang, 2016/09/07 00:12:38]

Uhh, no idea what I'm doing exactly. Let's take another stab at this...

       if (!manageZoom) {
         resolve();
         return;
@@ -185,6 +191,7 @@ function createBrowserApiForStandaloneExtension() {
     }
     chrome.tabs.getCurrent(function(tab) {
       streamInfo.tabId = tab.id;
+      streamInfo.tabUrl = tab.url;
       resolve();
     });
   }).then(function() { return BrowserApi.create(streamInfo, false); });