[modules] Split imports into regular and special, store regular ones in a map.

[modules] Split imports into regular and special, store regular ones in a map.

BUG=v8:1569
Committed: https://crrev.com/29e85e36f363203e3e1735c648b0f78837d57343
Cr-Commit-Position: refs/heads/master@{#38532}

[neis, 2016-08-09 13:29:36]

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

[commit-bot: I haz the power, 2016-08-09 13:29:47]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-09 13:53:47]

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

[commit-bot: I haz the power, 2016-08-09 13:53:48]

Dry run: This issue passed the CQ dry run.

[neis, 2016-08-09 15:08:23]

neis@chromium.org changed reviewers:
+ adamk@chromium.org

[neis, 2016-08-09 15:08:23]

ptal

[adamk, 2016-08-09 16:47:48]

lgtm

I don't love the names "special" or "regular", but there's not enough code yet
for me to think of better ones for now :)

[adamk, 2016-08-09 16:48:20]

Description was changed from

==========
[modules] Split imports into regular and special, store regular ones in a map.

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

to

==========
[modules] Split imports into regular and special, store regular ones in a map.

BUG=v8:1569
==========

[adamk, 2016-08-09 16:48:44]

Also, I added the modules tracking bug, I think it'd be good to attach that to
all modules CLs.

[neis, 2016-08-10 11:40:17]

The CQ bit was checked by neis@chromium.org

[commit-bot: I haz the power, 2016-08-10 11:40:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-10 11:42:03]

Description was changed from

==========
[modules] Split imports into regular and special, store regular ones in a map.

BUG=v8:1569
==========

to

==========
[modules] Split imports into regular and special, store regular ones in a map.

BUG=v8:1569
==========

[commit-bot: I haz the power, 2016-08-10 11:42:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 11:42:23]

Description was changed from

==========
[modules] Split imports into regular and special, store regular ones in a map.

BUG=v8:1569
==========

to

==========
[modules] Split imports into regular and special, store regular ones in a map.

BUG=v8:1569

Committed: https://crrev.com/29e85e36f363203e3e1735c648b0f78837d57343
Cr-Commit-Position: refs/heads/master@{#38532}
==========

[commit-bot: I haz the power, 2016-08-10 11:42:24]

Patchset 1 (id:??) landed as
https://crrev.com/29e85e36f363203e3e1735c648b0f78837d57343
Cr-Commit-Position: refs/heads/master@{#38532}

[neis, 2016-08-10 11:43:02]

On 2016/08/09 16:47:48, adamk wrote:
> lgtm
> 
> I don't love the names "special" or "regular", but there's not enough code yet
> for me to think of better ones for now :)

I'm not very happy with them either.