Url to the flake analysis page: https://chanli-dot-findit-for-me.appspot.com/waterfall/check-flake?master_name=chromium.linux&builder_name=Android%20Tests&build_number=35448&step_name=chrome_public_test_apk%20on%20Android&test_name=org.chromium.chrome.browser.media.router.MediaRouterIntegrationTest#testBasic Other 2 cases: https://chanli-dot-findit-for-me.appspot.com/waterfall/check-flake?master_name=chromium.linux&builder_name=Android%20Tests&build_number=35450&step_name=chrome_public_test_apk%20on%20Android&test_name=org.chromium.chrome.browser.tab.SadTabTest#testSadButtonRevertsBackToReloadAfterSuccessfulLoad https://chanli-dot-findit-for-me.appspot.com/waterfall/check-flake?master_name=chromium.linux&builder_name=Android%20Tests%20(dbg)&build_number=37812&step_name=chrome_public_test_apk%20on%20Android&test_name=org.chromium.chrome.browser.media.remote.CastPositionTransferTest#testLocalToRemotePositionTransfer All 3 cases are for chromium.linux, Android Tests(Or Android Tests (dbg)) chrome_public_test_apk on Android. An analysis showed on the dashboard (either completed or errored_out), but when clicked on the link, it shows: Error: You could schedule an analysis for flaky test only after you login with google.com account. All 3 analyses and their roots are in data store, but MasterFlakeAnalysis.GetVersion failed. To solve this problem, we need to encode the parameters before we send them to handler.
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/28254148d89f27878ff20100bea0024b2558de16 commit 28254148d89f27878ff20100bea0024b2558de16 Author: Chan Li <chanli@chromium.org> Date: Tue Dec 06 21:17:12 2016 [Findit] Urlencoded test_name filter and use key for read-only urls. BUG= 671386 Change-Id: I96b168052801f2b1c857693e9d3902ee44370054 Reviewed-on: https://chromium-review.googlesource.com/417180 Commit-Queue: Chan Li <chanli@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> [modify] https://crrev.com/28254148d89f27878ff20100bea0024b2558de16/appengine/findit/handlers/flake/list_flakes.py [modify] https://crrev.com/28254148d89f27878ff20100bea0024b2558de16/appengine/findit/handlers/flake/test/list_flakes_test.py [modify] https://crrev.com/28254148d89f27878ff20100bea0024b2558de16/appengine/findit/templates/flake/dashboard.html
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/1557ef798f3c5a0b00eb1e9a698adaa5f478b4a8 commit 1557ef798f3c5a0b00eb1e9a698adaa5f478b4a8 Author: Chan Li <chanli@chromium.org> Date: Tue Dec 06 22:07:45 2016 [Findit] Use encodeURIComponent instead of encodeURI to encode special characters. BUG= 671386 Change-Id: Ifcc9c06638ff0b4316d9990565a222d81fbfa231 Reviewed-on: https://chromium-review.googlesource.com/417102 Commit-Queue: Chan Li <chanli@chromium.org> Reviewed-by: Chan Li <chanli@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org> Reviewed-by: Jeffrey Li <lijeffrey@chromium.org> [modify] https://crrev.com/1557ef798f3c5a0b00eb1e9a698adaa5f478b4a8/appengine/findit/templates/flake/dashboard.html
Comment 1 by chanli@chromium.org
, Dec 6 2016