[9] RFR 8159488 "Deprivilege java.xml.crypto" and 8161171 "Missed the make/common/Modules.gmk file when integrating JDK-8154191"

[Sean Mullan]

On 07/13/2016 08:10 PM, Valerie Peng wrote:
> Sean,
>
> Can you please review the following two webrevs?
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8159488
> Webrev: http://cr.openjdk.java.net/~valeriep/8159488/

Looks good except for this one:

  127         // needed for reading Certs
  128         permission java.io.FilePermission "<<ALL FILES>>","read";

Why is that needed?

>
> While making changes for 8159488, I noticed a problem with my earlier
> putback of 8154191 - the top level Modules.gmk was not integrated.
> So, I filed 8161171: Missed the make/common/Modules.gmk file when
> integrating JDK-8154191.
> Can you also review this? It's essentially the same change as the one
> reviewed.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8161171
> Webrev: http://cr.openjdk.java.net/~valeriep/8161171/webrev.00/

I'll skip this since Mandy already reviewed that one.

--Sean