| commit | ccd2087f6b000a771e82c28002ad6fcb4a2930f1 | |
| author | Thomas Wolf <thomas.wolf@paranor.ch> | |
| Tue, 7 Jun 2016 20:17:33 +0000 (7 22:17 +0200) | ||
| committer | Thomas Wolf <thomas.wolf@paranor.ch> | |
| Tue, 7 Jun 2016 20:17:33 +0000 (7 22:17 +0200) | ||
| tree | 59b1caaabd2f1eeb344098847b69ea3bcbb2c2a7 | treesnapshot (tar.gz zip) |
| parent | b969c04c6ad48b202c9fc30cecd17ffb79cdc617 | commitdiff |
Paste in repo view need not log error
The PasteCommand shows an alert to the user _and_ logs an error. That's
overkill here. Just showing an alert is good enough. Besides, it's not
really an error.
Even better would be to deactivate the handler when the clipboard
doesn't contain a git repo location. But I suspect a property tester for
this would be rather expensive, so I opted for this simpler solution.
Also add a missing clipboard.dispose() in the FetchGerritChangePage.
Bug: 495629
Change-Id: I8d0dbf14ad168e38fe722827a67fcd58ef9f4cf8
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>
The PasteCommand shows an alert to the user _and_ logs an error. That's
overkill here. Just showing an alert is good enough. Besides, it's not
really an error.
Even better would be to deactivate the handler when the clipboard
doesn't contain a git repo location. But I suspect a property tester for
this would be rather expensive, so I opted for this simpler solution.
Also add a missing clipboard.dispose() in the FetchGerritChangePage.
Bug: 495629
Change-Id: I8d0dbf14ad168e38fe722827a67fcd58ef9f4cf8
Signed-off-by: Thomas Wolf <thomas.wolf@paranor.ch>