Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(212)

Unified Diff: chrome/browser/resources/snippets_internals.html

Issue 2145563002: Add ContentSuggestionsService to SnippetsInternals (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@articleprovider2
Patch Set: Bernhard's and Marc's comments Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/snippets_internals.html
diff --git a/chrome/browser/resources/snippets_internals.html b/chrome/browser/resources/snippets_internals.html
index 0f8fdfc340c5697720b43f93d4fa22ac7caee790..68874eb5e8c7749d9ebc20585b8d25a7c0844821 100644
--- a/chrome/browser/resources/snippets_internals.html
+++ b/chrome/browser/resources/snippets_internals.html
@@ -37,11 +37,13 @@ found in the LICENSE file.
</div>
<div id="snippets">
- <h2>Snippets <span class="detail">(click for details)</span></h2>
+ <h2>Snippets directly from NTPSnippetsService
+ <span class="detail">(click for details)</span>
+ </h2>
<table class="section-details">
<tr jsselect="list" style="display:none">
<td class="title-link">
- <span class="snippet-title" jsvalues="snippet-id:id">
+ <span class="snippet-title" jsvalues="hidden-id:id">
<span jscontent="title"></span>&gt;&gt;</span>
<div jsvalues="id:id" class="snippet-detail hidden">
<table>
@@ -106,7 +108,7 @@ found in the LICENSE file.
<table class="section-details">
<tr jsselect="list" style="display:none">
<td class="title-link">
- <span class="discarded-snippet-title" jsvalues="snippet-id:id">
+ <span class="discarded-snippet-title" jsvalues="hidden-id:id">
<span jscontent="title"></span> &gt;&gt;</span>
<div jsvalues="id:id" class="snippet-detail hidden">
<table>
@@ -149,4 +151,58 @@ found in the LICENSE file.
</table>
<div class="detail" id="hosts-empty"></div>
</div>
+
+ <div id="content-suggestions">
+ <h2>Content suggestions by category
+ <span class="detail">(click for details)</span>
+ </h2>
+ <div jsselect="list" style="display:none">
+ <h3>
+ <span jscontent="name"></span>
+ <span class="detail">
+ (<span jscontent="status"></span>)
+ </span>
+ </h3>
+
+ <table class="section-details">
+ <tr jsselect="suggestions" style="display:none">
+ <td class="title-link">
+ <span class="content-suggestion-title"
+ jsvalues="hidden-id:id">
+ <span jscontent="title"></span>&gt;&gt;</span>
Bernhard Bauer 2016/07/12 14:44:51 Move the last closing tag to a new line so it alig
Philipp Keck 2016/07/12 15:19:56 Good idea. I get confused by this HTML style where
Bernhard Bauer 2016/07/12 15:50:31 Because HTML5 doesn't require closing tags for cer
Philipp Keck 2016/07/12 16:05:37 So minimize HTML to reduce the binary size? Removi
Bernhard Bauer 2016/07/12 16:16:27 Coincidentally, aberent@ is working on exactly tha
jkrcal 2016/07/13 11:35:23 Indeed, minimizing the code was not my primary goa
+ <div jsvalues="id:id" class="content-suggestion-detail hidden">
+ <table>
+ <tr>
+ <td>ID
+ <td jscontent="suggestionId">
+ <tr>
+ <td>URL
+ <td><a class="url" jsvalues="href:url" jscontent="url"></a>
+ <tr>
+ <td>AMP URL
+ <td>
+ <a class="amp-url" jsvalues="href:ampUrl"
+ jscontent="ampUrl"></a>
+ <tr>
+ <td>Snippet text
+ <td jscontent="snippetText">
+ <tr>
+ <td>Publish date
+ <td class="expiry">
+ <span class="date" jscontent="publishDate"></span>
+ <tr>
+ <td>Publisher name
+ <td jscontent="publisherName">
+ </table>
+ </div>
+ </table>
+ </div>
+ <div class="detail" id="content-suggestions-empty"></div>
+ <div class="forms">
+ <input id="submit-clear-cached-suggestions" type="submit"
+ value="Clear cached suggestions">
+ <input id="submit-clear-discarded-suggestions" type="submit"
+ value="Clear discarded suggestions">
+ </div>
+ </div>
</div>

Powered by Google App Engine
This is Rietveld 408576698