Commit Graph

81 Commits

Author SHA1 Message Date
Jan Odvarko
95f6bd58be Bug 1266419 - Fixing tests; r=bgrins, poirot.alex 2016-05-26 14:51:28 +02:00
Ryan VanderMeulen
43398fbb69 Backed out changeset 9cfc40ea2dbd (bug 1269226) for browser_inspector_delete-selected-node-02.js timeouts. 2016-05-26 12:53:04 -04:00
Steve Melia
32fa6f3a81 Bug 1269226 - Explicitly handle node deletion in inspector breadcrumbs. r=pbro 2016-05-25 00:06:28 +01:00
Nicolas Chevobbe
5c07ed3c81 Bug 1273323 - Add integration tests for namespaced elements. r=jdescottes
Add tests for :
- markup display
- add rule
- breadcrumb
- highlighter info bar
- markup search
- webconsole element output

Move webconsole dom output test to head.js as similar things are done
in several tests.


MozReview-Commit-ID: 6gclZhzd7sD

--HG--
extra : transplant_source : %8F%9C%B5%E5%04S%BC%5B%C9%E8QV%8F%E8i%C7%E8%9A%08%A6
2016-05-22 00:03:42 +02:00
Joel Maher
a6fcbcf0a3 Bug 1270962 - move tests which access the clipboard to subsuite 'clipboard'. r=bgrins,ryanvm,armenzg a=merge
MozReview-Commit-ID: IZziPmwFtHj

--HG--
extra : source : a50249d48b1e86a3749bccc51ece4d1a827a621c
2016-05-25 15:28:24 -04:00
Carsten "Tomcat" Book
3cab03a461 Merge mozilla-central to fx-team 2016-05-24 15:15:55 +02:00
Patrick Brosset
4f864f4ffd Bug 1274964 - ESLint cleanup of devtools/client/inspector; r=jryans
MozReview-Commit-ID: HCRVzU6NdM4

--HG--
extra : rebase_source : 375d55c7688599d57615a04f68ed7230cab856c2
extra : amend_source : a0fa5104ef3fbd7ba98512979cfb2396645c07e9
2016-05-23 10:41:02 +02:00
Wes Kocher
fa48509cda Merge fx-team to central, a=merge 2016-05-20 14:14:00 -07:00
Nicolas Chevobbe
62400e0cd7 Bug 1270215 - Ensure we display properly cased node names all across the devtools. r=jdescottes
Add a displayName property on the NodeActor, which compute from Element.prefix + Element.localName.
The computation is made by a getNodeDisplayName function which can be imported wherever needed.
Edit some tests to ensure we correctly display node names.

MozReview-Commit-ID: 6z0G3ynbMoU

--HG--
extra : transplant_source : %E0%AFM%88D%BC%AD%08%1D%A4%FB%F2%5D%9E%D3%90%DE%94%EC%CD
2016-05-12 07:18:58 +02:00
J. Ryan Stinnett
30b2b7ce44 Bug 1271084 - Apply ESLint autofixes to ignored /devtools files. r=tromey
For simple rules like function spacing, we can auto-fix these across the code
base so they are followed in a consistent way.

To generate this patch, I ran:

./mach eslint devtools --no-ignore --fix

After this, I reverted any changes to third party files that we really do want
to ignore.

MozReview-Commit-ID: 6Q8BApkAW20
2016-05-18 12:49:23 -05:00
Patrick Brosset
044be2e517 Bug 1210671 - Avoid CPOW to create the test page in browser_inspector_initialization.js; r=miker
MozReview-Commit-ID: Jjq5SUWZTXa

--HG--
extra : rebase_source : 885ecd284c4dc3c82ab8229b677384af80781900
2016-05-17 12:51:44 +02:00
Patrick Brosset
45c01647ea Bug 1264907 - Don't show the firstChild of the current selection in breadcrumbs; r=jdescottes
The breadcrumbs widget used to have a feature where it would show the first
child of the current selection even the DOM tree hadn't been expanded that
far yet.
This was to allow keyboard navigating the DOM through the breadcrumbs.
The breadcrumbs is a very rarely used widget and this code was unnecessarily
making things complex.
It was decided that this feature would be removed.
Instead, the breadcrumbs now act as simple linear elements in a toolbar and
you can keyboard navigate them with LEFT/RIGHT only. TAB/shift-TAB simply go
in/out of the breadcrumbs widget.

MozReview-Commit-ID: BmcaLnVBOBn

--HG--
extra : rebase_source : 1673a6c9da02cf8b3b542e4ce905ccb239250aa7
2016-04-15 12:03:33 +02:00
Alexandre Poirot
62793e8163 Bug 1268441 - Convert inspector key shortcut to stop using XUL. r=bgrins 2016-05-12 03:07:56 -07:00
J. Ryan Stinnett
e1b9ca77b0 Bug 1270619 - Add missing imports instead of using globals. r=pbro
MozReview-Commit-ID: 28Xn7N9FSZr
2016-05-09 13:35:09 -05:00
Julian Descottes
87fe331949 Bug 1270462 - part1: extract devtools tooltip toggle logic to separate file;r=bgrins,jsnajdr
The code used to make the tooltip appear/disappear when hovering targets
has been extracted to a separate class that can be shared between the
current Tooltip.js implementation and the upcoming HTMLTooltip.

MozReview-Commit-ID: UYSjPFeMYK

--HG--
extra : rebase_source : 5dcca2d5887ffc98fec621092640073a0909c13f
2016-05-06 14:54:30 +02:00
Steve Melia
0fb164e885 Bug 1269915 - Remove right click menu from breadcrumbs;r=bgrins
MozReview-Commit-ID: 4mD2zcW7Awe
2016-05-04 22:48:15 +01:00
Brian Grinstead
c77eeb7c73 Bug 1269497 - Provide toolbox.win getter to make lines shorter in tests;r=jdescottes
MozReview-Commit-ID: 9KuWzsyHQGF
2016-05-03 07:41:20 -07:00
Nicolas Chevobbe
c6eac9c8b5 Bug 1269034 - Fix getSuggestionForQuery in inspector actor for id queries. r=bgrins
Make sure that result node id is not empty before appending it to the results

MozReview-Commit-ID: 5SuTrNf4t8v

--HG--
extra : transplant_source : E%AF%D7%A9W%DC%C6u%8B%9F%FF%00%A7%FEY%1A%7E%CA%AE%95
2016-04-30 00:36:17 +02:00
Nicolas Chevobbe
40d57bef03 Bug 1245365 - Fix markup view search with attribute selector. r=pbro
Add an ATTRIBUTE search state to better handle attribute's search.
Edit test to make sure we handle it right

MozReview-Commit-ID: ADcIJRbhjO

--HG--
extra : transplant_source : %C7R%AC%EE%2A%BA%C1%FE%B4%E4%D3%B0%D6c%B1%AB%B5%04%C6%CB
2016-04-21 21:54:03 +02:00
Julian Descottes
8e94f2fd48 Bug 1245776 - part2: remove custom openInspector methods;r=gl
shared/test/head had custom methods to open the Computed & Rule view.
They did not register any test actor and could be called several times
in the same test.

In order to reuse the openInspector method from inspector/test/head, the
tests have been modified.
We are moving from
openRuleView() -> do rule view tests -> openComputedView() -> do computed view tests
to
openInspector() -> select rule view -> do rule view tests -> select computed view -> do computed view tests

A bit more verbose for the tests themselves, but this will be useful when
migrating the shared tests to using the testActor.

MozReview-Commit-ID: 94cOqhPICyI

--HG--
extra : rebase_source : f7892af19b78ecdbb759fb21d17920f0f6e0f4c6
2016-04-20 15:47:22 +02:00
Julian Descottes
fb695be31f Bug 1245776 - part1: include inspector/test/head in shared/test/head;r=gl
Include inspector/test/head in shared/test/head and remove the redundant functions.
Comparing with markup/test/head and computed/test/head, some functions were also
duplicated and have been moved up to inspector/test/head or shared-head.

Logic for opening the inspector will be moved in part2.

MozReview-Commit-ID: 5EIalpbf3cf

--HG--
extra : rebase_source : 83fced672cf2da515b32fa5f1f85b3b12a4ffe5c
2016-04-20 11:40:02 +02:00
Yura Zenevich
bae1923a44 Bug 1242694 - improving inspector markup view accessibility (semantics and keyboard). r=gl
MozReview-Commit-ID: 4EvLALR4NIv

--HG--
extra : rebase_source : 98323a24c41f2753dd6a39f0fe30bb8adca235ca
2016-05-19 16:50:23 -04:00
Patrick Brosset
9d131c2b58 Bug 1262491 - Focus the newly inserted node in the markup-view when clicking the add button. r=jdescottes
MozReview-Commit-ID: FOyFBXT20Du

--HG--
extra : transplant_source : %BD%D7%3D%0B%C1h%3B%BC%95%D0f%C4%10%A0G%B4%E4%DF%99Y
2016-04-14 13:06:41 +02:00
Yura Zenevich
5d862fb3dc Bug 1242852 - (part 2) making inspector toolbar keyboard accessible. r=gl
MozReview-Commit-ID: BmLtydkQao7
---
 devtools/client/inspector/breadcrumbs.js           | 41 ++++++++++
 devtools/client/inspector/inspector-search.js      |  6 ++
 devtools/client/inspector/test/browser.ini         |  3 +
 .../browser_inspector_breadcrumbs_keyboard_trap.js | 79 ++++++++++++++++++
 .../test/browser_inspector_search_keyboard_trap.js | 93 ++++++++++++++++++++++
 devtools/client/inspector/test/head.js             | 17 ++++
 6 files changed, 239 insertions(+)
 create mode 100644 devtools/client/inspector/test/browser_inspector_breadcrumbs_keyboard_trap.js
 create mode 100644 devtools/client/inspector/test/browser_inspector_search_keyboard_trap.js

--HG--
extra : rebase_source : 5399de18ce0f5385088e17c247ffe251ccdf7c63
2016-04-12 11:53:54 -04:00
Ryan VanderMeulen
a90e9d792e Merge m-c to inbound. a=merge CLOSED TREE 2016-05-19 12:49:49 -04:00
Nancy Pang
761dd96791 Bug 1242715 - Added labels to controls and added test cases. r=pbro
MozReview-Commit-ID: HGGTeshLBDs

--HG--
extra : rebase_source : 02310a5784f9b763f81e93ea8edbcb0ad0e830a8
2016-05-19 10:03:58 -04:00
Julian Descottes
4ea6d79805 Bug 1260053 - prevent sidebar resize when panel is collapsed;r=bgrins
The collapsible sidebar has min-height:35vh and max-height:75vh
when viewport-width < 700px (ie vertical layout).

These bounds should not apply when the panel is collapsed. The
collapsing relies on negative margins to hide the panel, which
can only work if the computed height remains constant once the
panel has been collapsed.

Added a new mochitest checking that the height and margin-bottom
are constant through a layout switch.

MozReview-Commit-ID: K6EGiBtJ11S

--HG--
extra : rebase_source : 5141f4610b586c32158d0b3ee2f50fda230f3c05
2016-03-30 12:11:35 +02:00
Patrick Brosset
4cff2a6c2d Bug 1261781 - Inspector toolbar button for inserting new elements; r=jdescottes
MozReview-Commit-ID: BoD6MEMQb3b

--HG--
extra : rebase_source : 15da3c9f2a99da2a97b449b462a8afbccf01997e
2016-04-05 11:08:21 +02:00
Chris Manchester
f7a1b3fb60 Bug 1242051 - Add inter-directory test support file dependencies to ini manifests. r=gps
Previously, every test and support file would be synced to the objdir
when running any test. Now that only those support files and tests requested
are synced, we note support files required beyond those in a test's
directory in ini manifests.

MozReview-Commit-ID: EmlDz9d4lqt
2016-04-04 14:56:52 -07:00
Jared Wein
8955f79d9b Backout the backout because follow-up fix got pushed. Bug 1259859 - Add a test that looks for misused characters in user-facing strings. r=dolske,jryans
MozReview-Commit-ID: IBHYB76bvsk
2016-04-02 10:50:50 +02:00
Sebastian Hengst
370ca8535c Backed out changeset a877338b4740 (bug 1259859) for failures in devtools mochitest browser_cmd_highlight_03.js. r=backout 2016-04-02 10:45:08 +02:00
Jared Wein
4dbcfae693 Bug 1259859 - Add a test that looks for misused characters in user-facing strings. r=dolske,jryans
MozReview-Commit-ID: IBHYB76bvsk
2016-04-02 02:27:36 -04:00
Matteo Ferretti
4373495447 Bug 1139187 - Unit Test for Geometry Editor; r=pbro
MozReview-Commit-ID: 9smoBjNt1St

--HG--
extra : rebase_source : 57b0e4873193728fa25b74619b8e4a6e19138189
2016-03-17 10:59:39 -04:00
Matteo Ferretti
7d401d4ae4 Bug 1139187 - Allow moving and resizing elements in content; r=pbro
MozReview-Commit-ID: EmmFBXW22dk

--HG--
extra : rebase_source : 78e58f1a61dfafb61cf89498881c40d42c7a6c18
2016-03-17 10:59:03 -04:00
Steve Melia
5c6d9164b2 Bug 1256658 - Make inspector search next/previous based on ctrl-g by default, and cmd-g on osx;r=bgrins
MozReview-Commit-ID: F5NsNGmoeL8
2016-03-26 15:21:23 +00:00
Steve Melia
184e838496 Bug 1259060 - Add shift+enter to navigate to previous result in inspector search;r=bgrins
MozReview-Commit-ID: 6Y8tz7dML5s
2016-03-25 13:58:27 +00:00
Ryan VanderMeulen
5a7da8152e No bug - Request longer timeout for browser_inspector_search-01.js.
--HG--
extra : rebase_source : 8ab46228754e21a4bd5327adaeb146c1522b1cd9
2016-03-18 17:07:46 -04:00
Patrick Brosset
4c08a64bcd Bug 1246088 - Safely access contentWindow in iframe and embed; r=bgrins
MozReview-Commit-ID: 6GRsMsJPn5x

--HG--
extra : rebase_source : e2a69eead5a64287fcf5575ca6f09464edf121ca
2016-03-09 11:54:12 +01:00
Ryan VanderMeulen
e41e37efae Bug 1250058 - Skip browser_inspector_initialization.js on Linux debug e10s as well. 2016-03-05 22:31:02 -05:00
Ryan VanderMeulen
496caa4fef Bug 1250058 - Skip a couple leak-prone inspector tests on OSX e10s as well. 2016-02-29 18:14:56 -05:00
Julian Descottes
d16c1e7cfb Bug 1189492 - part3: show inspector toggle panel button in SIDE host;r=tromey
MozReview-Commit-ID: B7IY7n1EzUq

--HG--
extra : rebase_source : de69df71e5a3b0fe193ffebb0c3fcf93c142b846
2016-02-25 10:17:59 +01:00
Patrick Brosset
a31b063cce Bug 1246677 - 7 - Clean remaining ruleview and tests eslint warnings; r=jdescottes
MozReview-Commit-ID: BcyEv4fUWJZ

--HG--
extra : rebase_source : d589e9df51222df10db1e2095f5018d8c44bcb5a
2016-02-23 17:09:01 +01:00
Patrick Brosset
fb434e2e1e Bug 1035661 - Remove useless test browser_inspector_scrolling.js; r=miker
MozReview-Commit-ID: 5PU2iXMFcIZ

--HG--
extra : rebase_source : 7af9eaf6b9f5500511ec5ea85bea3c3528ce38b4
2016-02-24 17:29:22 +01:00
Patrick Brosset
79edff1525 Bug 1250925 - Enable browser_inspector_gcli-inspect-command.js with e10s; r=miker
MozReview-Commit-ID: I9yvp7JEsef

--HG--
extra : rebase_source : 7609f80929faf916686a109d531f9726de2697f5
2016-02-24 17:20:36 +01:00
Ryan VanderMeulen
b92db8e975 Backed out 10 changesets (bug 1246677) for causing frequent OSX browser_rules_completion-existing-property_01.js failures (10.6 and 10.10).
Backed out changeset a6d3a7a5e4ea (bug 1246677)
Backed out changeset 5dcb51fcef15 (bug 1246677)
Backed out changeset e450c5329a06 (bug 1246677)
Backed out changeset a65e8badc4c4 (bug 1246677)
Backed out changeset 0cdde48161bb (bug 1246677)
Backed out changeset bab3de413d37 (bug 1246677)
Backed out changeset e4f6cc6197cb (bug 1246677)
Backed out changeset 34e6b56e419d (bug 1246677)
Backed out changeset 5067508e5db4 (bug 1246677)
Backed out changeset 913b4129466e (bug 1246677)
2016-02-24 21:04:31 -05:00
Patrick Brosset
e8e084446b Bug 1246677 - 7 - Clean remaining ruleview and tests eslint warnings; r=jdescottes
MozReview-Commit-ID: BcyEv4fUWJZ

--HG--
extra : rebase_source : cb63ed3c966567fe298bf67cafdbf08e36118712
2016-02-23 17:09:01 +01:00
Patrick Brosset
ce1b99b254 Bug 1244120 - Enable browser_rules_content_02.js with e10s; r=bgrins
Extracted a shared helper to open the browser context menu and choose
the 'inspect element' item. This helper works with e10s.
Adapted it a little bit so it waits for the right events in order to
make sure the inspector is ready.

This also involved modifying inspectNode in nsContextMenu.js to make it
wait until the node was selected and the node was ready.

Used this in browser_inspector_initialization.js,
browser_rules_content_02.js and browser_markup_keybindings_04.js

Also removed a now useless inspector-updated event that was trigger from
the animation-inspector panel in some situation. This was left behind
from a long time ago but didn't serve any purpose anymore.

MozReview-Commit-ID: 5bFHxxrwxhL

--HG--
extra : rebase_source : cb30c7e47605f59674f2344dba10af814172333d
2016-02-22 10:02:54 +01:00
Patrick Brosset
25793ecb39 Bug 1245654 - Make the inspector shortcut start the pick mode again; r=bgrins
MozReview-Commit-ID: 9qImivhKhKf

--HG--
extra : rebase_source : c65c314c0195205a062c6d7260a6e001044eea0b
extra : histedit_source : ddad40ea60d5abdbd81c4f3286a2d300de588572%2Ccd82c42337de918b928be10ad26a94b910613bf1
2016-02-18 17:52:33 +01:00
Patrick Brosset
47facd047c Bug 1244755 - 1 - Enable browser_layout.js with e10s by using the testActor; r=miker
--HG--
extra : commitid : 8leAzjkOtri
extra : rebase_source : fc651e0af92495cae1b835dbf7b89d8c7ac6fe28
extra : histedit_source : 859047cf767b5a4809d4466eac03ceed2f71e9a5
2016-02-01 17:26:08 +01:00
Wes Kocher
4722a4bfe8 Backed out changeset 3d2eace770d1 (bug 1244120) for very frequent leaks on Win7 CLOSED TREE
--HG--
extra : commitid : AiLlcy3cWsb
2016-02-04 12:51:14 -08:00