OS: Android Move all Android NTP related Java/C++ bindings to a dedicated directory. All files should be move to chrome/browser/android/ntp
This isn't only about Java/C++ bindings, but rather about all NTP-related C++ code, no?
Everything that is in c/b/android
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66 commit ea5c499adf2b5bf93ed158d0d5084fc6ef768c66 Author: markusheintz <markusheintz@google.com> Date: Fri Mar 18 17:27:13 2016 Move the remaining NTP realted files to c/b/android/ntp subdir. BUG=596080 Review URL: https://codereview.chromium.org/1811333003 Cr-Commit-Position: refs/heads/master@{#382003} [modify] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/browser/android/chrome_jni_registrar.cc [rename] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/browser/android/ntp/new_tab_page_prefs.cc [rename] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/browser/android/ntp/new_tab_page_prefs.h [rename] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/browser/android/ntp/new_tab_page_url_handler.cc [rename] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/browser/android/ntp/new_tab_page_url_handler.h [modify] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/browser/chrome_content_browser_client.cc [modify] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/browser/prefs/browser_prefs.cc [modify] https://crrev.com/ea5c499adf2b5bf93ed158d0d5084fc6ef768c66/chrome/chrome_browser.gypi
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8ebb579108e181994403c0fde3f5ae011ea21272 commit 8ebb579108e181994403c0fde3f5ae011ea21272 Author: markusheintz <markusheintz@chromium.org> Date: Fri Mar 18 19:27:46 2016 Add newt@ to chrome/browser/android/ntp/OWNERS after creating the dir and moving all NTP related files there. BUG=596080 Review URL: https://codereview.chromium.org/1814183004 Cr-Commit-Position: refs/heads/master@{#382048} [modify] https://crrev.com/8ebb579108e181994403c0fde3f5ae011ea21272/chrome/browser/android/ntp/OWNERS
Reopen, because most_visited_sites.* are not moved yet. There may be even more files to move.
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/2447cc1b134f127a6f1d956b2d30585b53c58a89 commit 2447cc1b134f127a6f1d956b2d30585b53c58a89 Author: markusheintz <markusheintz@chromium.org> Date: Tue Mar 29 13:52:27 2016 Move most_visited_sites* files to the ntp directory in c/b/a/ BUG=596080 Review URL: https://codereview.chromium.org/1840013002 Cr-Commit-Position: refs/heads/master@{#383715} [modify] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/android/BUILD.gn [modify] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/browser/android/chrome_jni_registrar.cc [rename] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/browser/android/ntp/most_visited_sites.cc [rename] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/browser/android/ntp/most_visited_sites.h [rename] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/browser/android/ntp/most_visited_sites_unittest.cc [modify] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/browser/prefs/browser_prefs.cc [modify] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/chrome_browser.gypi [modify] https://crrev.com/2447cc1b134f127a6f1d956b2d30585b53c58a89/chrome/chrome_tests_unit.gypi
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/48bfbafafb178f8fe624a00d81ced2e0c518d113 commit 48bfbafafb178f8fe624a00d81ced2e0c518d113 Author: markusheintz <markusheintz@chromium.org> Date: Wed Mar 30 13:09:04 2016 Move popular_sites.* files to c/b/a/ntp BUG=596080 Review URL: https://codereview.chromium.org/1840033002 Cr-Commit-Position: refs/heads/master@{#383961} [modify] https://crrev.com/48bfbafafb178f8fe624a00d81ced2e0c518d113/chrome/browser/android/ntp/most_visited_sites.cc [rename] https://crrev.com/48bfbafafb178f8fe624a00d81ced2e0c518d113/chrome/browser/android/ntp/popular_sites.cc [rename] https://crrev.com/48bfbafafb178f8fe624a00d81ced2e0c518d113/chrome/browser/android/ntp/popular_sites.h [modify] https://crrev.com/48bfbafafb178f8fe624a00d81ced2e0c518d113/chrome/browser/prefs/browser_prefs.cc [modify] https://crrev.com/48bfbafafb178f8fe624a00d81ced2e0c518d113/chrome/browser/ui/webui/popular_sites_internals_message_handler.cc [modify] https://crrev.com/48bfbafafb178f8fe624a00d81ced2e0c518d113/chrome/chrome_browser.gypi
Comment 1 by treib@chromium.org
, Mar 18 2016