[I-Spy] Add support for rebaselining expectations from the web UI.

[I-Spy] Add support for rebaselining expectations from the web UI.

BUG=318865
TEST=unittests included
NOTRY=True

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=240226

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=243140

[craigdh, 2013-12-06 02:42:23]

Corresponding downstream change will be up shortly.

[frankf, 2013-12-07 00:23:35]

On 2013/12/06 02:42:23, craigdh wrote:
> Corresponding downstream change will be up shortly.

Please provide a link to the running instace

[craigdh, 2013-12-10 19:18:24]

On 2013/12/07 00:23:35, frankf wrote:
> On 2013/12/06 02:42:23, craigdh wrote:
> > Corresponding downstream change will be up shortly.
> 
> Please provide a link to the running instace

https://ispy.googleplex.com/?test_run=test3

[frankf, 2013-12-11 00:34:41]

On 2013/12/10 19:18:24, craigdh wrote:
> On 2013/12/07 00:23:35, frankf wrote:
> > On 2013/12/06 02:42:23, craigdh wrote:
> > > Corresponding downstream change will be up shortly.
> > 
> > Please provide a link to the running instace
> 
> https://ispy.googleplex.com/?test_run=test3

You shouldn't show the button if there's no diff. You also get an error if you
press the button twice.

[frankf, 2013-12-11 00:40:19]

On 2013/12/11 00:34:41, frankf wrote:
> On 2013/12/10 19:18:24, craigdh wrote:
> > On 2013/12/07 00:23:35, frankf wrote:
> > > On 2013/12/06 02:42:23, craigdh wrote:
> > > > Corresponding downstream change will be up shortly.
> > > 
> > > Please provide a link to the running instace
> > 
> > https://ispy.googleplex.com/?test_run=test3
> 
> You shouldn't show the button if there's no diff. You also get an error if you
> press the button twice.

Also, I would reword the button to:

"Chrome version x.x.x.x looks good!" with a popup explaining the consequence.

[frankf, 2013-12-11 00:40:31]

https://codereview.chromium.org/106523003/diff/1/chrome/test/functional/ispy/...
File chrome/test/functional/ispy/client/dom.py (right):

https://codereview.chromium.org/106523003/diff/1/chrome/test/functional/ispy/...
chrome/test/functional/ispy/client/dom.py:17: return _script
just return the string

[craigdh, 2013-12-11 21:56:38]

ptal. https://ispy.googleplex.com/?test_run=test5

The button issues were already fixed in the last patch, it just hadn't been
pushed to the live site.

https://codereview.chromium.org/106523003/diff/1/chrome/test/functional/ispy/...
File chrome/test/functional/ispy/client/dom.py (right):

https://codereview.chromium.org/106523003/diff/1/chrome/test/functional/ispy/...
chrome/test/functional/ispy/client/dom.py:17: return _script
On 2013/12/11 00:40:31, frankf wrote:
> just return the string

Done.

[frankf, 2013-12-11 22:09:15]

lgtm

[commit-bot: I haz the power, 2013-12-12 00:09:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/craigdh@chromium.org/106523003/10001

[commit-bot: I haz the power, 2013-12-12 00:11:32]

Change committed as 240226

[craigdh, 2014-01-04 02:27:12]

On 2013/12/12 00:11:32, I haz the power (commit-bot) wrote:
> Change committed as 240226

Need to reland this, it was missing dom.py.

[frankf, 2014-01-06 19:15:59]

lgtm

[commit-bot: I haz the power, 2014-01-06 19:29:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/craigdh@chromium.org/106523003/30001

[commit-bot: I haz the power, 2014-01-06 19:31:38]

Change committed as 243140