[DocServer] Fixed Sample Links

chrome/common/extensions/docs/static/js/samples.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 (function() {
   var search_box = document.getElementById('search_input');
   var samples = document.getElementsByClassName('sample');
 
   function filterSamples() {
     var search_text = search_box.value.toLowerCase();
     for (var i = 0; i < samples.length; ++i) {
-      var sample = samples[i]
-      if (sample.getAttribute('tags').toLowerCase().indexOf(search_text) < 0)
+      var sample = samples[i];
+      var sample_title = samples[i].getElementsByTagName('h2')[0].innerText;
+      if (sample.getAttribute('tags').toLowerCase().indexOf(search_text) < 0 &&
+          sample_title.toLowerCase().indexOf(search_text) < 0)
         sample.style.display = 'none';
       else
         sample.style.display = '';
     }
   }
   search_box.addEventListener('search', filterSamples);
   search_box.addEventListener('keyup', filterSamples);
 
   var api_filter_items = document.getElementById('api_filter_items');
   api_filter_items.addEventListener('click', function(event) {
     if (event.target instanceof HTMLAnchorElement) {
       search_box.value = event.target.innerText;
       filterSamples();
     }
   });
+
+  function onLoad() {
+    var search = window.location.hash.substr(1).replace(/-/g, ' ');
+    if (search) {
+      search_box.value = search;
+      filterSamples();
+    }
+  }
+
+  window.addEventListener('DOMContentLoaded', onLoad);

[not at google - send to devlin, 2013/09/23 15:59:51]

the onLoad is hopefully unnecessary since the relevant DOM will have loaded by
the time this script executes.

however, I don't think we should hide the samples this way anyway: I don't like
executing JS on the client (keep it to an absolute minimum) and if we're loading
all this content anyway we might as well just show it. anchor is fine. If you
want to only show a single sample (and that sounds good to me actually) then we
should make it part of the actual URL in some way, so that the server only need
to render that particular sample.

could you file a but on that? there are a couple of different ways to do it and
a quick look on a bug would resolve it rather than doing it here.

[François Beaufort, 2013/09/24 09:24:49]

Done.

 })();