Compile cr.ui.Table and cr.ui.table.* in gyp v2

Compile cr.ui.Table and cr.ui.table.* in gyp v2

- Replaced all occurrences of cr.ui.Table in cr/ui/table/* to Element.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2779663002
Cr-Commit-Position: refs/heads/master@{#460309}
Committed: https://chromium.googlesource.com/chromium/src/+/068669c24287ba5fbd0b541649ecc65fa08019ee

[oka, 2017-03-27 14:41:42]

Description was changed from

==========
Compile cr.ui.Table and cr.ui.table.* .

BUG=705527
TEST=run_compiler
==========

to

==========
Compile cr.ui.Table and cr.ui.table.* .

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2017-03-28 01:25:19]

.

[oka, 2017-03-28 01:25:21]

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

[commit-bot: I haz the power, 2017-03-28 01:25:44]

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, 2017-03-28 01:38:34]

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

[commit-bot: I haz the power, 2017-03-28 01:38:35]

Dry run: Try jobs failed on following builders:
  closure_compilation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/closure_compila...)

[oka, 2017-03-28 05:38:50]

.

[oka, 2017-03-28 05:43:19]

Description was changed from

==========
Compile cr.ui.Table and cr.ui.table.* .

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Compile cr.ui.Table and cr.ui.table.* .

- Replaced all occurrences of cr.ui.Table in cr/ui/table/*.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2017-03-28 05:44:26]

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

[commit-bot: I haz the power, 2017-03-28 05:44:39]

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

[oka, 2017-03-28 05:44:57]

Description was changed from

==========
Compile cr.ui.Table and cr.ui.table.* .

- Replaced all occurrences of cr.ui.Table in cr/ui/table/*.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Compile cr.ui.Table and cr.ui.table.* in gyp v2

- Replaced all occurrences of cr.ui.Table in cr/ui/table/*.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[oka, 2017-03-28 05:50:06]

oka@chromium.org changed reviewers:
+ dbeam@chromium.org

[oka, 2017-03-28 05:50:07]

PTAL.

[commit-bot: I haz the power, 2017-03-28 08:05:28]

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

[commit-bot: I haz the power, 2017-03-28 08:05:30]

Dry run: This issue passed the CQ dry run.

[Dan Beam, 2017-03-28 13:05:56]

lgtm

https://codereview.chromium.org/2779663002/diff/40001/ui/webui/resources/js/c...
File ui/webui/resources/js/cr/ui/table.js (right):

https://codereview.chromium.org/2779663002/diff/40001/ui/webui/resources/js/c...
ui/webui/resources/js/cr/ui/table.js:134: 
@private

[oka, 2017-03-28 13:09:26]

.

[oka, 2017-03-28 13:12:35]

Thanks!

https://codereview.chromium.org/2779663002/diff/40001/ui/webui/resources/js/c...
File ui/webui/resources/js/cr/ui/table.js (right):

https://codereview.chromium.org/2779663002/diff/40001/ui/webui/resources/js/c...
ui/webui/resources/js/cr/ui/table.js:134: 
On 2017/03/28 13:05:56, Dan Beam (slow) wrote:
> @private

Done.

[oka, 2017-03-28 13:12:49]

The CQ bit was checked by oka@chromium.org

[oka, 2017-03-28 13:12:50]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/2779663002/#ps60001
(title: ".")

[oka, 2017-03-28 13:13:05]

Description was changed from

==========
Compile cr.ui.Table and cr.ui.table.* in gyp v2

- Replaced all occurrences of cr.ui.Table in cr/ui/table/*.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Compile cr.ui.Table and cr.ui.table.* in gyp v2

- Replaced all occurrences of cr.ui.Table in cr/ui/table/* to Element.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2017-03-28 13:13:08]

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

[commit-bot: I haz the power, 2017-03-28 13:13:09]

This CL has an open dependency (Issue 2781533002 Patch 20001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[oka, 2017-03-28 23:53:49]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2017-03-28 23:54:47]

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, 2017-03-29 01:40:41]

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

[commit-bot: I haz the power, 2017-03-29 01:40:42]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[oka, 2017-03-29 05:08:34]

The CQ bit was checked by oka@chromium.org

[commit-bot: I haz the power, 2017-03-29 05:09:09]

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, 2017-03-29 07:09:00]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490764114713900,
"parent_rev": "efa081e7cade69a78de92f0548ad80d440cec7e6", "commit_rev":
"068669c24287ba5fbd0b541649ecc65fa08019ee"}

[commit-bot: I haz the power, 2017-03-29 07:09:46]

Description was changed from

==========
Compile cr.ui.Table and cr.ui.table.* in gyp v2

- Replaced all occurrences of cr.ui.Table in cr/ui/table/* to Element.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Compile cr.ui.Table and cr.ui.table.* in gyp v2

- Replaced all occurrences of cr.ui.Table in cr/ui/table/* to Element.
- Moved renderFunction_ in TableList, which uses the knowledge that
cr.ui.Table has columnModel, to cr.ui.Table.

BUG=705527
TEST=run_compiler
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2779663002
Cr-Commit-Position: refs/heads/master@{#460309}
Committed:
https://chromium.googlesource.com/chromium/src/+/068669c24287ba5fbd0b541649ec...
==========

[commit-bot: I haz the power, 2017-03-29 07:09:47]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/068669c24287ba5fbd0b541649ec...