Expose Physical Web data

components/physical_web/data_source/physical_web_data_source.h

Index: components/physical_web/data_source/physical_web_data_source.h
diff --git a/components/physical_web/data_source/physical_web_data_source.h b/components/physical_web/data_source/physical_web_data_source.h
index 1fa11407fae736e3333edaa026697b54c9d1a824..f19a464f9379e956d67427c1d080d38239bff55b 100644
--- a/components/physical_web/data_source/physical_web_data_source.h
+++ b/components/physical_web/data_source/physical_web_data_source.h
@@ -7,6 +7,8 @@
 
 #include <memory>
 
+#include physical_web_listener.h

[Olivier, 2016/08/22 07:45:09]

All include path must be full path starting from src.
Here
#include "components/physical_web/data_source/physical_web_listener.h"

We also tend to include the minimal amount of files in headers (and in source
file too).
You don't need the actual header here and can just forward declare the
PhysicalWebListener class.

[hayesjordan, 2016/08/22 17:06:21]

Done.

+
 namespace base {
 class ListValue;
 }
@@ -35,6 +37,9 @@ class PhysicalWebDataSource {
   // If discovery is inactive or network requests are enabled, it will always
   // return false.
   virtual bool HasUnresolvedDiscoveries() = 0;
+
+  // Register for changes to Physical Web URLs and associated page metadata.
+  virtual void RegisterListener(PhysicalWebListener* physical_web_listener) = 0;

[Olivier, 2016/08/22 07:45:09]

Please add UnregisterListener (or similar).

[hayesjordan, 2016/08/22 17:06:21]

Done.

 };
 
 #endif  // COMPONENTS_PHYSICAL_WEB_DATA_SOURCE_PHYSICAL_WEB_DATA_SOURCE_H_