Remove "list item" from Android NTP class names.

Remove "list item" from Android NTP class names.

This is in preparation for more refactorings that will get rid of having an explicit list of NTP items.

BUG=616090
Committed: https://crrev.com/3323db9ee62d0bc21278d904823413a7f2cff3a6
Cr-Commit-Position: refs/heads/master@{#412087}

[Bernhard Bauer, 2016-08-15 20:12:28]

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

[commit-bot: I haz the power, 2016-08-15 20:12:43]

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-15 20:32:19]

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

[commit-bot: I haz the power, 2016-08-15 20:32:20]

Dry run: 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...)

[Bernhard Bauer, 2016-08-15 20:55:44]

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

[commit-bot: I haz the power, 2016-08-15 20:56:08]

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

[Bernhard Bauer, 2016-08-15 21:15:12]

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

[commit-bot: I haz the power, 2016-08-15 21:15:24]

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

[Bernhard Bauer, 2016-08-15 21:17:31]

bauerb@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org, peconn@chromium.org

[Bernhard Bauer, 2016-08-15 21:17:32]

Please review. Thanks!

[commit-bot: I haz the power, 2016-08-15 22:12:35]

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

[commit-bot: I haz the power, 2016-08-15 22:12:36]

Dry run: This issue passed the CQ dry run.

[PEConn, 2016-08-15 22:56:34]

LGTM - thanks for this, it'll make everything more readable!.

[Bernhard Bauer, 2016-08-15 23:10:05]

The CQ bit was checked by bauerb@chromium.org

[commit-bot: I haz the power, 2016-08-15 23:10:41]

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-15 23:14:54]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-15 23:16:37]

Description was changed from

==========
Remove "list item" from Android NTP class names.

This is in preparation for more refactorings that will get rid of having an
explicit list of NTP items.

BUG=616090
==========

to

==========
Remove "list item" from Android NTP class names.

This is in preparation for more refactorings that will get rid of having an
explicit list of NTP items.

BUG=616090

Committed: https://crrev.com/3323db9ee62d0bc21278d904823413a7f2cff3a6
Cr-Commit-Position: refs/heads/master@{#412087}
==========

[commit-bot: I haz the power, 2016-08-15 23:16:38]

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

[Michael van Ouwerkerk, 2016-08-16 09:20:51]

lgtm

I would have preferred to keep 'Item' in Header and Article though, to
distinguish them more clearly from the View and ViewHolder types.

[Bernhard Bauer, 2016-08-17 20:47:05]

On 2016/08/16 09:20:51, Michael van Ouwerkerk wrote:
> lgtm
> 
> I would have preferred to keep 'Item' in Header and Article though, to
> distinguish them more clearly from the View and ViewHolder types.

What I was trying to do was actually to go the other direction and remove as
much suffix as possible, but for other things (StatusItem, AboveTheFoldItem)
leaving the Item out wouldn't really have worked :)