Reland Reland Reland Switch Chrome Android using v8 arch specific external data name

Reland Reland Reland Switch Chrome Android using v8 arch specific external data name

This patch changed all android GN target which use v8 snapshot files
as of it being prepared.

origin patch
https://codereview.chromium.org/1422793004/

This patch also fix package_resource.py issue that asset's
renaming didn't trigger the packaging.

origin patch
https://codereview.chromium.org/1417753004/

Switch Chrome Android using v8 arch specific external data name

For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.

This patch also removed the existing v8 startup data in
destination directory by using copy_ex to remove directory
first.

BUG=540788, 547235
TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick@chromium.org,mef@chromium.org,rmcilroy@chromium.org,dtrainor@chromium.org,cpu@chromium.org

Committed: https://crrev.com/01630673ae102e70913ac6832fa994c50a1514b5
Cr-Commit-Position: refs/heads/master@{#357577}

[michaelbai, 2015-11-03 05:38:27]

Description was changed from

==========
Reland Reland Reland Switch Chrome Android using v8 arch specific external data
name

BUG=
==========

to

==========
Reland Reland Reland Switch Chrome Android using v8 arch specific external data
name

This patch changed all android GN target which use v8 snapshot files
as of it being prepared.

origin patch
https://codereview.chromium.org/1422793004/

This patch also fix package_resource.py issue that asset's
renaming didn't trigger the packaging.

origin patch
https://codereview.chromium.org/1417753004/

Switch Chrome Android using v8 arch specific external data name

For 32-bit, natives_blob_32.bin and snapshot_blob_32.bin are used.
For 64-bit, natives_blob_64.bin and natives_blob_64.bin are used.

This patch also removed the existing v8 startup data in
destination directory by using copy_ex to remove directory
first.

BUG=540788, 547235
TBR=jochen@chromium.org,dpranke@chromium.org,halliwell@chromium.org,jbudorick...
==========

[michaelbai, 2015-11-03 05:40:44]

michaelbai@chromium.org changed reviewers:
+ jbudorick@chromium.org

[michaelbai, 2015-11-03 05:47:35]

jbudorick@ comparing to reverted patch, this patch change some GN targets which
were added after I prepared this patch.

agrieve@ Dirk, If you happen to know anyone will land GN patch which related to
v8 snapshot, please let me know.

This is no way to try Android GN bot, right?

[michaelbai, 2015-11-03 05:48:41]

On 2015/11/03 05:47:35, michaelbai wrote:
> jbudorick@ comparing to reverted patch, this patch change some GN targets
which
> were added after I prepared this patch.
> 
> agrieve@ Dirk, If you happen to know anyone will land GN patch which related
to
> v8 snapshot, please let me know.
> 
> This is no way to try Android GN bot, right?

jbudorick@ patch #1 is reverted patch.

[michaelbai, 2015-11-03 05:49:11]

The CQ bit was checked by michaelbai@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-11-03 05:49:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1411283009/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1411283009/20001

[commit-bot: I haz the power, 2015-11-03 07:00:46]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-03 07:00:47]

Dry run: Try jobs failed on following builders:
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)

[jbudorick, 2015-11-03 16:56:23]

lgtm

On 2015/11/03 05:47:35, michaelbai wrote:
> jbudorick@ comparing to reverted patch, this patch change some GN targets
which
> were added after I prepared this patch.
> 
> agrieve@ Dirk, If you happen to know anyone will land GN patch which related
to
> v8 snapshot, please let me know.
> 
> This is no way to try Android GN bot, right?

Unfortunately, no, there's no Android GN trybot that runs tests.

[michaelbai, 2015-11-03 18:37:37]

The CQ bit was checked by michaelbai@chromium.org

[commit-bot: I haz the power, 2015-11-03 18:39:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1411283009/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1411283009/20001

[commit-bot: I haz the power, 2015-11-03 19:48:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-03 19:49:10]

Patchset 2 (id:??) landed as
https://crrev.com/01630673ae102e70913ac6832fa994c50a1514b5
Cr-Commit-Position: refs/heads/master@{#357577}