[Offline pages] Refactoring: Merge Connectivity Listener into TabObserver

chrome/android/java_sources.gni

Index: chrome/android/java_sources.gni
diff --git a/chrome/android/java_sources.gni b/chrome/android/java_sources.gni
index 95ef7e1317677c9e06a36ad4628378f70df7cd70..3bda9b582210671ef70413cdfb4a2ab17735edee 100644
--- a/chrome/android/java_sources.gni
+++ b/chrome/android/java_sources.gni
@@ -499,7 +499,6 @@ chrome_java_sources = [
   "java/src/org/chromium/chrome/browser/ntp/snippets/SnippetsManager.java",
   "java/src/org/chromium/chrome/browser/offlinepages/ClientId.java",
   "java/src/org/chromium/chrome/browser/offlinepages/OfflinePageBridge.java",
-  "java/src/org/chromium/chrome/browser/offlinepages/OfflinePageConnectivityListener.java",

[Pete Williamson, 2016/03/28 19:52:38]

This looks good, but why is it required?  Do we need to manually update gni
files the same way we used to update  .gyp files every time we add or remove a
file?

[fgorski, 2016/03/28 22:52:32]

Don't know what to tell you, except that we still have to update BUILD.gn files,
which include these. What you don't need is the explicit step generating
projects before running ninja.

[Pete Williamson, 2016/03/29 08:09:36]

OK, I guess that might have happened while I was off working in another code
tree, I missed the announcement that we need to add files to BUILD.gn

[fgorski, 2016/03/29 16:00:43]

Acknowledged.

   "java/src/org/chromium/chrome/browser/offlinepages/OfflinePageFreeUpSpaceCallback.java",
   "java/src/org/chromium/chrome/browser/offlinepages/OfflinePageFreeUpSpaceDialog.java",
   "java/src/org/chromium/chrome/browser/offlinepages/OfflinePageItem.java",