[DocServer] Fixed Sample Links

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

Index: chrome/common/extensions/docs/static/js/samples.js
diff --git a/chrome/common/extensions/docs/static/js/samples.js b/chrome/common/extensions/docs/static/js/samples.js
index 6388205733b3ab368ca2e94556200a7eb642afce..66da941f3bb80042bf9f9578ba7000ae1bfc1746 100644
--- a/chrome/common/extensions/docs/static/js/samples.js
+++ b/chrome/common/extensions/docs/static/js/samples.js
@@ -9,8 +9,10 @@
   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;

[not at google - send to devlin, 2013/09/24 18:16:30]

nice, I like it. nits:
- s/sample_title/sampleTitle/ or just |title|.
  - oops all of those other variable names should be camel case as well... hm.
- use |sample| not |samples[i]|.
- use textContent not innerText.

[François Beaufort, 2013/09/25 10:10:30]

Done.

+      if (sample.getAttribute('tags').toLowerCase().indexOf(search_text) < 0 &&
+          sample_title.toLowerCase().indexOf(search_text) < 0)
         sample.style.display = 'none';
       else
         sample.style.display = '';