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

chrome/browser/resources/pdf/browser_api.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 'use strict';
 
 /**
  * Returns a promise that will resolve to the default zoom factor.
  * @param {!Object} streamInfo The stream object pointing to the data contained
  *     in the PDF.
@@ skipping 134 matching lines @@
 
 /**
  * Creates a BrowserApi for an extension running as a mime handler.
  * @return {Promise<BrowserApi>} A promise to a BrowserApi instance constructed
  *     using the mimeHandlerPrivate API.
  */
 function createBrowserApiForMimeHandlerView() {
   return new Promise(function(resolve, reject) {
     chrome.mimeHandlerPrivate.getStreamInfo(resolve);
   }).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;
       }
       chrome.tabs.setZoomSettings(
           streamInfo.tabId, {mode: 'manual', scope: 'per-tab'}, resolve);
     }).then(function() { return BrowserApi.create(streamInfo, manageZoom); });
   });
 }
 
@@ skipping 11 matching lines @@
     embedded: window.parent != window,
     tabId: -1,
   };
   return new Promise(function(resolve, reject) {
     if (!chrome.tabs) {
       resolve();
       return;
     }
     chrome.tabs.getCurrent(function(tab) {
       streamInfo.tabId = tab.id;
+      streamInfo.tabUrl = tab.url;
       resolve();
     });
   }).then(function() { return BrowserApi.create(streamInfo, false); });
 }
 
 /**
  * Returns a promise that will resolve to a BrowserApi instance.
  * @return {Promise<BrowserApi>} A promise to a BrowserApi instance for the
  *     current environment.
  */
 function createBrowserApi() {
   if (window.location.search)
     return createBrowserApiForStandaloneExtension();
 
   return createBrowserApiForMimeHandlerView();
 }