Reland of: Refactor process_resources.py to use aapt's --extra-packages

Reland #2 Refactor process_resources.py to use aapt's --extra-packages

Without --extra-packages, we now use the same code-path
to generate the current R.java as well as dependent R.java files.
Previously, we were combing aapt's R.java with our own template.

Our custom template no longer generating assignments for non-array
styleable fields within onResourcesLoaded() (matches aapt's behaviour)

Reason for reland:
Fixed GYP issue & verified all webrtc targets compile.

TBR=jbudorick
BUG=none

Committed: https://crrev.com/f3c8dfe0190ad701a9339436ff1d7f3b1e556e7d
Cr-Commit-Position: refs/heads/master@{#401624}

[agrieve, 2016-06-23 15:19:31]

Created Reland of land: Refactor process_resources.py to use aapt's
--extra-packages

[agrieve, 2016-06-23 15:24:45]

Description was changed from

==========
Reland of land: Refactor process_resources.py to use aapt's --extra-packages
(patchset #1 id:1 of https://codereview.chromium.org/2090303003/ )

Reason for revert:
asdf

Original issue's description:
> Revert of Reland: Refactor process_resources.py to use aapt's --extra-packages
(patchset #1 id:1 of https://codereview.chromium.org/2083383003/ )
> 
> Reason for revert:
> Broke webrtc GYP rules.
> 
> Original issue's description:
> > Reland: Refactor process_resources.py to use aapt's --extra-packages
> >
> > Reason for Reland:
> > - No longer generating assignments for non-array styleable fields within
> >   onResourcesLoaded() (matches aapt's behaviour)
> >
> > We actually never pass --include-all-resources, so this is just
> > refactoring dead code. I think it's an improvement though,
> > as the new comments explain more of why things work the way they do.
> >
> > TBR=agrieve
> > BUG=none
> >
> > Committed: https://crrev.com/c04a207f591c1cae455fee6c9c7d8b0dbca20abe
> > Cr-Commit-Position: refs/heads/master@{#401355}
> 
> TBR=jbudorick@chromium.org,phoglund@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=none
> 
> Committed: https://crrev.com/932350f8583c875a222df3c3b7d9913418a2fe54
> Cr-Commit-Position: refs/heads/master@{#401595}

TBR=jbudorick@chromium.org,phoglund@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=none
==========

to

==========
Reland #2 Refactor process_resources.py to use aapt's --extra-packages

Without --extra-packages, we now use the same code-path
to generate the current R.java as well as dependent R.java files.
Previously, we were combing aapt's R.java with our own template.

Our custom template no longer generating assignments for non-array
styleable fields within onResourcesLoaded() (matches aapt's behaviour)

Reason for reland:
Fixed GYP issue & verified all webrtc targets compile.

TBR=jbudorick
BUG=none
==========

[agrieve, 2016-06-23 15:26:02]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-06-23 15:26:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2096693002/40001

[agrieve, 2016-06-23 15:27:00]

The CQ bit was unchecked by agrieve@chromium.org

[agrieve, 2016-06-23 15:28:06]

The CQ bit was checked by agrieve@chromium.org

[commit-bot: I haz the power, 2016-06-23 15:28:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2096693002/60001

[commit-bot: I haz the power, 2016-06-23 16:21:52]

Description was changed from

==========
Reland #2 Refactor process_resources.py to use aapt's --extra-packages

Without --extra-packages, we now use the same code-path
to generate the current R.java as well as dependent R.java files.
Previously, we were combing aapt's R.java with our own template.

Our custom template no longer generating assignments for non-array
styleable fields within onResourcesLoaded() (matches aapt's behaviour)

Reason for reland:
Fixed GYP issue & verified all webrtc targets compile.

TBR=jbudorick
BUG=none
==========

to

==========
Reland #2 Refactor process_resources.py to use aapt's --extra-packages

Without --extra-packages, we now use the same code-path
to generate the current R.java as well as dependent R.java files.
Previously, we were combing aapt's R.java with our own template.

Our custom template no longer generating assignments for non-array
styleable fields within onResourcesLoaded() (matches aapt's behaviour)

Reason for reland:
Fixed GYP issue & verified all webrtc targets compile.

TBR=jbudorick
BUG=none
==========

[commit-bot: I haz the power, 2016-06-23 16:21:56]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-06-23 16:27:24]

Description was changed from

==========
Reland #2 Refactor process_resources.py to use aapt's --extra-packages

Without --extra-packages, we now use the same code-path
to generate the current R.java as well as dependent R.java files.
Previously, we were combing aapt's R.java with our own template.

Our custom template no longer generating assignments for non-array
styleable fields within onResourcesLoaded() (matches aapt's behaviour)

Reason for reland:
Fixed GYP issue & verified all webrtc targets compile.

TBR=jbudorick
BUG=none
==========

to

==========
Reland #2 Refactor process_resources.py to use aapt's --extra-packages

Without --extra-packages, we now use the same code-path
to generate the current R.java as well as dependent R.java files.
Previously, we were combing aapt's R.java with our own template.

Our custom template no longer generating assignments for non-array
styleable fields within onResourcesLoaded() (matches aapt's behaviour)

Reason for reland:
Fixed GYP issue & verified all webrtc targets compile.

TBR=jbudorick
BUG=none

Committed: https://crrev.com/f3c8dfe0190ad701a9339436ff1d7f3b1e556e7d
Cr-Commit-Position: refs/heads/master@{#401624}
==========

[commit-bot: I haz the power, 2016-06-23 16:27:25]

Patchset 3 (id:??) landed as
https://crrev.com/f3c8dfe0190ad701a9339436ff1d7f3b1e556e7d
Cr-Commit-Position: refs/heads/master@{#401624}

[agrieve, 2016-06-23 17:18:14]

A revert of this CL (patchset #3 id:60001) has been created in
https://codereview.chromium.org/2097583002/ by agrieve@chromium.org.

The reason for reverting is: Relanded the wrong patchset! Missing code for
excluding styleable..