Chromium Code Reviews Chromium Code Reviews
Issue 12319108:
Prevent querying of restricted query values. (Closed)
Base URL: svn://svn.chromium.org/chrome/trunk/src| OLD | NEW |
|---|---|
| 1 <html> | 1 <html> |
| 2 <head> | 2 <head> |
| 3 <script> | 3 <script> |
| 4 | 4 |
| 5 var apiHandle; | 5 var apiHandle; |
| 6 var newTabPageHandle; | 6 var newTabPageHandle; |
| 7 var savedUserText = null; | 7 var savedUserText = null; |
| 8 var suggestionIndex = -1; | 8 var suggestionIndex = -1; |
| 9 var suggestions = ["result 1", "result 2", "result 3"]; | 9 var suggestions = ["result 1", "result 2", "result 3"]; |
| 10 var suggestion; | 10 var suggestion; |
| (...skipping 14 matching lines...) Expand all Loading... | |
| 25 if (window.navigator && window.navigator.embeddedSearch && | 25 if (window.navigator && window.navigator.embeddedSearch && |
| 26 window.navigator.embeddedSearch.newTabPage) | 26 window.navigator.embeddedSearch.newTabPage) |
| 27 return window.navigator.embeddedSearch.newTabPage; | 27 return window.navigator.embeddedSearch.newTabPage; |
| 28 if (window.chrome && window.chrome.embeddedSearch && | 28 if (window.chrome && window.chrome.embeddedSearch && |
| 29 window.chrome.embeddedSearch.newTabPage) | 29 window.chrome.embeddedSearch.newTabPage) |
| 30 return window.chrome.embeddedSearch.newTabPage; | 30 return window.chrome.embeddedSearch.newTabPage; |
| 31 return null; | 31 return null; |
| 32 } | 32 } |
| 33 | 33 |
| 34 function handleNativeSuggestions() { | 34 function handleNativeSuggestions() { |
| 35 // Apparently, one needs to access nativeSuggestions before | |
| 36 // apiHandle.setRestrictedValue can work. | |
|
Jered
2013/03/05 17:47:44
Can you explain a bit more? We should document or
robertshield
2013/03/05 18:19:04
apiHandle.setRestrictedValue ends up in SearchBoxE
| |
| 37 var foo = apiHandle.nativeSuggestions; | |
| 38 | |
| 35 // Showing the loader at 100% height. | 39 // Showing the loader at 100% height. |
| 36 apiHandle.show(2 /* QUERY_SUGGESTIONS */); | 40 apiHandle.show(2 /* QUERY_SUGGESTIONS */); |
| 37 } | 41 } |
| 38 | 42 |
| 39 function handleSubmit() { | 43 function handleSubmit() { |
| 40 // Currently a NOOP in this testing framework, but needs to be defined | 44 // Currently a NOOP in this testing framework, but needs to be defined |
| 41 // so Chrome can correctly determine Instant support for the page. | 45 // so Chrome can correctly determine Instant support for the page. |
| 42 } | 46 } |
| 43 | 47 |
| 44 function handleOnChange() { | 48 function handleOnChange() { |
| (...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 107 } | 111 } |
| 108 | 112 |
| 109 setUp(); | 113 setUp(); |
| 110 | 114 |
| 111 </script> | 115 </script> |
| 112 </head> | 116 </head> |
| 113 <body> | 117 <body> |
| 114 <h1>Instant</h1> | 118 <h1>Instant</h1> |
| 115 </body> | 119 </body> |
| 116 </html> | 120 </html> |
| OLD | NEW |
