[subresource_filter] Site Details UI for default state

chrome/android/java/res/xml/single_website_preferences.xml

Index: chrome/android/java/res/xml/single_website_preferences.xml
diff --git a/chrome/android/java/res/xml/single_website_preferences.xml b/chrome/android/java/res/xml/single_website_preferences.xml
index 0d171442f2f2cdf6ca0a7ef0f11c181deca750fe..564ff32baabe9aaa3d3c0ef6f0a6742932baa070 100644
--- a/chrome/android/java/res/xml/single_website_preferences.xml
+++ b/chrome/android/java/res/xml/single_website_preferences.xml
@@ -11,6 +11,12 @@
     <Preference
         android:key="os_permissions_warning_divider"
         android:layout="@layout/divider_preference" />
+    <org.chromium.chrome.browser.preferences.ChromeBasePreference
+        android:key="website_settings_extra_info"
+        android:icon="@drawable/btn_info" />
+    <Preference
+        android:key="website_settings_extra_info_divider"
+        android:layout="@layout/divider_preference" />

[Finnur, 2017/05/23 15:37:13]

I think it would lessen the cognitive overload for the user (and potentially
code complexity for us devs also) if this was a simple subtitle under the
subresource_filter_permission_list preference. I think that was the original
intent of the designer and we should explore that option, unless someone
objects.

A secondary, and perhaps moot concern, is that a preference might not be the
best widget to use to show a static information message, as it has certain
behavior on-click, which indicate that an action takes place on clicking (which
would then need to be disabled to avoid confusion). We should either find a way
to disable these behaviors or find a replacement. I'm not sure what is a good
replacement is, if we end up going that route, but maybe twellington@ knows.

[Charlie Harrison, 2017/05/23 17:12:18]

I'm fine with adding a second subtitle which we did discuss before. Not
implemented in the current PS though.

raymes: Can you confirm that this is an OK solution?
Acknowledged.

[raymes, 2017/05/23 22:29:01]

I chatted with Charlie offline. I'm ok not showing a banner (I don't think
that's great either) but my concern was that afaik nothing else shows a subtitle
here, and I'm not sure that it fits in well with the existing design of the UI.
In the long term I think we want to show subtitles but we should come up with a
good design.

My preference as a short term, simple solution would be just to modify the text
that's displayed in the option for the setting. There's a precedent for doing
this with the default search engine geolocation setting. 

In any case, it may be good to land a simple solution and iron out the details
with a UX designer. 

     <PreferenceCategory
         android:key="site_heading"
         android:title="@string/website_settings_site_category" />