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

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 c7b172078d7778472af64c4fe851a35bca15b73e..a71af583285105c945178858641a53f0937fdd38 100644
--- a/chrome/browser/resources/pdf/pdf.js
+++ b/chrome/browser/resources/pdf/pdf.js
@@ -202,8 +202,12 @@ function PDFViewer(browserApi) {
   this.plugin_.setAttribute('background-color', backgroundColor);
   this.plugin_.setAttribute('top-toolbar-height', topToolbarHeight);
 
-  if (!this.browserApi_.getStreamInfo().embedded)
+  if (this.browserApi_.getStreamInfo().embedded) {
+    this.plugin_.setAttribute('embedder-url',

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

I think top-level-url may be more accurate here (there may be several layers of
embedder, but we will only record the top level one.

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

Done in all the places.

+                              this.browserApi_.getStreamInfo().tabUrl);
+  } else {
     this.plugin_.setAttribute('full-frame', '');

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

optional: We could merge this with top-level-url and define empty as full-frame.
It would save parsing an additional param (though we would still probably want
to record a URL in the crash report). I'll leave that up to you.

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

Can we ever be embedded, but have an empty top-level URL somehow? I'm not 100%
sure, so I've left the two separate.

+  }
   document.body.appendChild(this.plugin_);
 
   // Setup the button event listeners.