Trusted plugin: Miscellaneous cleanup.

Trusted plugin: Miscellaneous cleanup.

This removes some unnecessary code in the trusted plugin. I broke this out to
its own change while fixing up other things.

BUG=239656
R=dmichael@chromium.org

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

[dmichael (off chromium), 2014-04-28 23:01:44]

lgtm

https://codereview.chromium.org/260053008/diff/1/ppapi/native_client/src/trus...
File ppapi/native_client/src/trusted/plugin/plugin.cc (right):

https://codereview.chromium.org/260053008/diff/1/ppapi/native_client/src/trus...
ppapi/native_client/src/trusted/plugin/plugin.cc:964: return
pp::URLUtil_Dev::Get()->DocumentCanRequest(this, pp::Var(url));
Hopefully this doesn't matter, but *if* this ever gets invoked on a background
thread, it's more likely to fail now. Though it was probably never thread-safe.

Might be worth adding a check that you're on the main thread.

[teravest, 2014-04-29 19:49:15]

On 2014/04/28 23:01:44, dmichael wrote:
> lgtm
> 
>
https://codereview.chromium.org/260053008/diff/1/ppapi/native_client/src/trus...
> File ppapi/native_client/src/trusted/plugin/plugin.cc (right):
> 
>
https://codereview.chromium.org/260053008/diff/1/ppapi/native_client/src/trus...
> ppapi/native_client/src/trusted/plugin/plugin.cc:964: return
> pp::URLUtil_Dev::Get()->DocumentCanRequest(this, pp::Var(url));
> Hopefully this doesn't matter, but *if* this ever gets invoked on a background
> thread, it's more likely to fail now. Though it was probably never
thread-safe.
> 
> Might be worth adding a check that you're on the main thread.

Done.

[teravest, 2014-04-29 20:12:49]

Committed patchset #2 manually as r266945 (presubmit successful).