Hide knowledge of webkit::ppapi::PluginDelegate from chrome. This is part of moving ppapi implement…

Hide knowledge of webkit::ppapi::PluginDelegate from chrome. This is part of moving ppapi implementation from webkit/plugins/ppapi to content/renderer.

Chrome used four methods on that interface for PDFs.
-DidStartLoading/DidStopLoading: I just exposed these methods from RenderViewImpl on the RenderView interface
-SetContentRestriction: I moved this functionality completely into chrome. This allows us to get rid of all the content restrictions related methods in the content API, and to move content_restriction.h out to chrome.
-SaveURLAs: exposed the internal SaveURL method on WebContents

BUG=263054
R=scottmg@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=213027

[scottmg, 2013-07-23 00:34:16]

lgtm, but maybe get some a pepper person to look too?

https://codereview.chromium.org/19800005/diff/18003/chrome/renderer/pepper/pp...
File chrome/renderer/pepper/ppb_pdf_impl.cc (right):

https://codereview.chromium.org/19800005/diff/18003/chrome/renderer/pepper/pp...
chrome/renderer/pepper/ppb_pdf_impl.cc:357: content::Referrer
referrer(frame->document().url(),
i don't know anything about the lifetimes here, is it possible for the frame to
be gone?

[jam, 2013-07-23 00:42:03]

https://codereview.chromium.org/19800005/diff/18003/chrome/renderer/pepper/pp...
File chrome/renderer/pepper/ppb_pdf_impl.cc (right):

https://codereview.chromium.org/19800005/diff/18003/chrome/renderer/pepper/pp...
chrome/renderer/pepper/ppb_pdf_impl.cc:357: content::Referrer
referrer(frame->document().url(),
On 2013/07/23 00:34:16, scottmg wrote:
> i don't know anything about the lifetimes here, is it possible for the frame
to
> be gone?

this is code I moved from PepperPluginDelegateImpl::SaveURLAs, so there should
be no change in behavior.

https://codereview.chromium.org/19800005/diff/18003/content/renderer/pepper/p...

[jam, 2013-07-23 00:55:00]

On 2013/07/23 00:34:16, scottmg wrote:
> lgtm, but maybe get some a pepper person to look too?

for most of the changes I will do in that directory, I'll send the change to a
pepper person. but this stuff is just basic refactorings of code that I wrote,
so I think your review will be enough :)

[scottmg, 2013-07-23 00:56:10]

On 2013/07/23 00:55:00, jam wrote:
> On 2013/07/23 00:34:16, scottmg wrote:
> > lgtm, but maybe get some a pepper person to look too?
> 
> for most of the changes I will do in that directory, I'll send the change to a
> pepper person. but this stuff is just basic refactorings of code that I wrote,
> so I think your review will be enough :)

yup, fine with me.