Move NTP related classes to ntp package.

Move NTP related classes to ntp package.

This CL moves the following classes to org.chromium.chrome.browser.ntp:
- ForeignSessionHelper
- LogoBridge
- NewTabPagePrefs
- RecentlyClosedBridge

BUG=None

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

[aurimas (slooooooooow), 2015-07-29 21:15:44]

aurimas@chromium.org changed reviewers:
+ newt@chromium.org

[aurimas (slooooooooow), 2015-07-29 21:15:44]

PTAL

[newt (away), 2015-07-29 22:35:32]

I'm OK with making the various JNI classes package-private, but if you're going
to do that, be sure to remove all the occurrences of "public" from them.

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
File
chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java:26:
public interface ForeignSessionCallback {
remove all instances of "public" from this file, since the class itself is
package-private

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
File chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoBridge.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoBridge.java:20:
public static class Logo {
remove all instances of "public" from this file, since the class itself is
package-private

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
File
chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentlyClosedBridge.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentlyClosedBridge.java:23:
public interface RecentlyClosedCallback {
remove all instances of "public" from this file, since the class itself can be
made package-private

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/javat...
File
chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeRecentlyClosedTest.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/javat...
chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeRecentlyClosedTest.java:1:
// Copyright 2015 The Chromium Authors. All rights reserved.
this class should probably also be in the ntp package.

[aurimas (slooooooooow), 2015-07-29 23:03:41]

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
File
chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/ntp/ForeignSessionHelper.java:26:
public interface ForeignSessionCallback {
On 2015/07/29 at 22:35:32, newt wrote:
> remove all instances of "public" from this file, since the class itself is
package-private

Done

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
File chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoBridge.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/ntp/LogoBridge.java:20:
public static class Logo {
On 2015/07/29 at 22:35:32, newt wrote:
> remove all instances of "public" from this file, since the class itself is
package-private

Done

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
File
chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentlyClosedBridge.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/java/...
chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentlyClosedBridge.java:23:
public interface RecentlyClosedCallback {
On 2015/07/29 at 22:35:32, newt wrote:
> remove all instances of "public" from this file, since the class itself can be
made package-private

Done

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/javat...
File
chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeRecentlyClosedTest.java
(right):

https://chromiumcodereview.appspot.com/1263573004/diff/1/chrome/android/javat...
chrome/android/javatests/src/org/chromium/chrome/browser/document/DocumentModeRecentlyClosedTest.java:1:
// Copyright 2015 The Chromium Authors. All rights reserved.
On 2015/07/29 at 22:35:32, newt wrote:
> this class should probably also be in the ntp package.

Done

[newt (away), 2015-07-29 23:21:19]

lgtm

[aurimas (slooooooooow), 2015-07-29 23:23:17]

The CQ bit was checked by aurimas@chromium.org

[commit-bot: I haz the power, 2015-07-29 23:24:15]

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

[commit-bot: I haz the power, 2015-07-30 02:12:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-30 02:13:39]

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