Remove innerHTML usage in chrome://signin-internals

This change removes innerHTML usage by changing it to textContent.
This make the page compatible with Trusted Types.

Bug: 1125965
Change-Id: I0f01d25ef3e001be1c1aa6d056b0a5b72da99592
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2399821
Commit-Queue: David Roger <droger@chromium.org>
Reviewed-by: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#805229}

components/signin/core/browser/about_signin_internals.cc

@@ -526,7 +526,7 @@ AboutSigninInternals::TokenInfo::ToValue() const {
 
   std::string scopes_str;
   for (auto it = scopes.begin(); it != scopes.end(); ++it) {
-    scopes_str += *it + "<br/>";
+    scopes_str += *it + "\n";
   }
   token_info->SetString("scopes", scopes_str);
   token_info->SetString("request_time", base::TimeToISO8601(request_time));

components/signin/core/browser/resources/signin_index.css

@@ -36,6 +36,7 @@ div#cookie-info {
 }
 
 table.signin-details {
+  white-space: pre;
   width: 100%;
 }
 

components/signin/core/browser/resources/signin_index.html

@@ -48,9 +48,9 @@
                       [this.children[2].innerText, request_time],
                       [this.children[3].innerText, status]])">
           <td jscontent="service"></td>
-          <td jsvalues=".innerHTML: scopes"></td>
+          <td jsvalues=".textContent: scopes"></td>
           <td jscontent="request_time"></td>
-          <td jsvalues=".innerHTML: status"></td>
+          <td jsvalues=".textContent: status"></td>
         </tr>
       </table>
     </div>