Limit PDF helper extension to print preview only

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 188 matching lines @@
     });
   }).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)
+  if (location.search && location.ancestorOrigins[0] === 'chrome://print')

[raymes, 2016/10/11 06:24:44]

Do we need the window.location.search check too?

[robwu, 2016/10/11 21:05:38]

Yes. When loaded inside the plugin location.ancestorOrigins.length === 0, which
is indistinguishable from being loaded in the top-level frame.

     return createBrowserApiForStandaloneExtension();
 
   return createBrowserApiForMimeHandlerView();
 }