LGTM thanks, with trivial
https://codereview.appspot.com/8940043/diff/3001/app/subapps/browser/browser.js File app/subapps/browser/browser.js (right):
https://codereview.appspot.com/8940043/diff/3001/app/subapps/browser/browser.js#newcode24 app/subapps/browser/browser.js:24: * Show or hide the details panel. We decided against the *'s in a Friday call.
https://codereview.appspot.com/8940043/diff/3001/test/test_browser_app.js File test/test_browser_app.js (right):
https://codereview.appspot.com/8940043/diff/3001/test/test_browser_app.js#newcode514 test/test_browser_app.js:514: it('no querystring still searched', function() { non descriptive test name.
https://codereview.appspot.com/8940043/
« Back to merge proposal
LGTM thanks, with trivial
https:/ /codereview. appspot. com/8940043/ diff/3001/ app/subapps/ browser/ browser. js browser/ browser. js (right):
File app/subapps/
https:/ /codereview. appspot. com/8940043/ diff/3001/ app/subapps/ browser/ browser. js#newcode24 browser/ browser. js:24: * Show or hide the details panel.
app/subapps/
We decided against the *'s in a Friday call.
https:/ /codereview. appspot. com/8940043/ diff/3001/ test/test_ browser_ app.js browser_ app.js (right):
File test/test_
https:/ /codereview. appspot. com/8940043/ diff/3001/ test/test_ browser_ app.js# newcode514 browser_ app.js: 514: it('no querystring still searched',
test/test_
function() {
non descriptive test name.
https:/ /codereview. appspot. com/8940043/