gecko-dev/devtools
Patrick Brosset fc07e38ecd Bug 1295008 - Hide the previous eyedropper when we request to show a new one; r=miker
The eyedropper can be shown in 2 distinct ways:
- the 'eyedropper' gcli command can be called (from the dev toolboar
  or from the browser devtools menu),
- or the inspector's 'pickColorFromPage' method can be called (from
  the inspector toolbar or from the color-picker in the ruleview).

Before this change, it was possible to show several eyedropper because
these 2 codepaths didn't know about each other.
Now, when executing the gcli command, the inspector's
'cancelPickColorFromPage' method is called to hide it first.
And, when the 'pickColorFromPage' method is called, the 'eyedropper --hide'
gcli command is called too.

This way, there's only one eyedropper shown.

There was also a problem where the gcli command would create a new
eyedropper everytime it was called. This was fixed too, by maintaining a
WeakMap of all eyedroppers opened so far.

MozReview-Commit-ID: F6fBP5R7ZTJ

--HG--
extra : rebase_source : f04bbf224ced7b93b4e5b0c42662731ecc58cb58
2016-08-24 10:29:49 +02:00
..
client Bug 1295008 - Hide the previous eyedropper when we request to show a new one; r=miker 2016-08-24 10:29:49 +02:00
docs Bug 1268591 - Make devtools icons black by default and invert them for dark theme. r=bgrins a=kwierso CLOSED TREE 2016-07-22 00:44:11 +02:00
server Bug 1295008 - Hide the previous eyedropper when we request to show a new one; r=miker 2016-08-24 10:29:49 +02:00
shared Bug 1295008 - Hide the previous eyedropper when we request to show a new one; r=miker 2016-08-24 10:29:49 +02:00
.eslintrc Bug 1292253 - replace some Ci uses in view-helpers and TableWidget; r=gregtatum 2016-08-09 09:43:58 -06:00
.eslintrc.mochitests Bug 1264649 - enable reject-some-requires rule for inspector; r=pbro 2016-07-21 09:04:47 -06:00
.eslintrc.xpcshell Bug 1264649 - enable reject-some-requires rule for inspector; r=pbro 2016-07-21 09:04:47 -06:00
bootstrap.js Bug 1271048 - Fix eslint issues in devtools/bootstrap.js. r=pbro 2016-05-10 00:15:03 +02:00
chrome.manifest Bug 1236905 - Use locale files from sources when using devtools addon. r=jryans 2016-01-08 06:59:35 -08:00
install.rdf Bug 1225108 - Introduce an addon to easily work on devtools codebase. r=jryans 2016-01-05 02:41:49 -08:00
moz.build Bug 1286877 - do not set c-basic-offset for python-mode; r=gps 2016-07-14 10:16:42 -06:00
templates.mozbuild Bug 1286877 - do not set c-basic-offset for python-mode; r=gps 2016-07-14 10:16:42 -06:00