Граф коммитов

301 Коммитов

Автор SHA1 Сообщение Дата
Patrick Brosset dff0290b38 Bug 938188 - Make highlighter capable of highlighting only one region, fading out others. r=bgrins
Introduced a new highlighter option that makes each region <path> element in the highlighter only
cover the actual area of the corresponding region, excluding the area of nested regions.
This is useful when used with the existing showOnly region because it lets users see exactly where
a given region is.
This patch makes the layout-view use this new option, so that when users hover over the various
regions in the layout-view, only the corresponding regions are highlighted.
2015-06-20 14:51:50 +02:00
Jeff Walden 8ebc2cc372 Bug 1146136 - Fix most in-tree tests that parenthesize destructuring pattern assignment targets, as ES6 forbids such parenthesization. (The patch making SpiderMonkey reject this syntax will follow shortly.) r=efaust, r=testingonlychange 2015-05-30 00:55:35 -07:00
Léon McGregor 83f6b76b6a Bug 1163332 - Add a 'screenshot this node' button to the inspector popup menu. r=pbrosset 2015-06-09 03:58:00 -04:00
Patrick Brosset 6cfa30710e Bug 1170477 - 3 - Test for the inspector breadcrumbs context menu; r=tromey
--HG--
extra : rebase_source : 47b0157ddbc87c0dc99f7e4df494ca404ab06cd3
extra : histedit_source : 162f6aded07df485d6a8fe3780bfd22c80c722c4
2015-06-03 16:31:23 +02:00
Patrick Brosset fad0f0afdb Bug 1170477 - 2 - Fix undefined inspector.strings accessor in breadcrumbs; r=tromey
The only user of the l10s inspector strings bundle is a string that this
change removes.
The "Siblings" string is supposed to tell users that the section of the
menu that follows contains sibling nodes, but it's disabled (and looks like
an item you should normally be able to click on), and it's separated from
the list of siblings by a horizontal separator (making it look like even
less related to the next section).

Also fixed the fact that the inspector context menu was regenerated for
every sibling found on the current node.

Also fixed the fact that the inspector context menu would not appear when
right clicking on the documentElement breadcrumb node.

--HG--
extra : rebase_source : a398639563d4b68ccd3e8d1bc0b90cdc7ee4580d
extra : histedit_source : 60fc27b626eeedc0fb5c40b75e3bf406e463bd26
2015-06-02 11:49:11 +02:00
Patrick Brosset 4d356f71eb Bug 1170477 - 1 - Quick code cleanup for better eslint compliancy; r=tromey
--HG--
extra : rebase_source : 7a7fe63d3831e92a0a9952cd6ab9d0ed247eeee0
extra : histedit_source : 3d18ed4247202f437aafedb897aea1cdf1b39956
2015-06-02 11:26:24 +02:00
J. Ryan Stinnett f828e44873 Bug 1168689 - Allow highlighter to see XBL anonymous nodes. r=pbrosset 2015-06-01 16:52:13 -07:00
Patrick Brosset 8c2aaa42cf Bug 1166189 - ESLint configuration files for devtools; r=tromey
Adds a global .eslintrc file for all source files in /browser/devtools.
This file defines a set of global variables that are commonly used in
/browser/devtools code.
The files that import modules with Cu.import will need to define their
own globals as /* globals ... */ comments.
This file also defines the configuration for all the rules we want to
use.

This also adds a set of .eslintrc files, one per test directory in
/browser/devtools. Each of these files extend from one of 2 parent config
files: .eslintrc.xpcshell or .eslintrc.mochitest.
The parent config define the set of globals these types of tests have
access to (test runner functions, assertion functions, etc.).

Finally, this also adds .eslintrc files in /toolkit/devtools for code and
tests, which just extend from their counterparts in /browser/devtools.

--HG--
extra : rebase_source : d7b6f639634b64dc084d0835a5fb36c9720a87e7
2015-05-28 09:30:03 +02:00
Patrick Brosset 6b0f771384 Bug 1167617 - Remove SpiderMonkey specific JS syntax from inspector; r=jryans
--HG--
extra : rebase_source : 95a9daf992adf418831ed362c07ac3402e0c5f5f
extra : histedit_source : 4e2d9d6fadc0da7bb7b29bad6dffd7576df6f16a
2015-05-22 20:50:01 +02:00
Patrick Brosset c0cd389247 Bug 1122605 - Focus selected inspector node after user selection. r=bgrins
--HG--
extra : rebase_source : b1f020a34c3dd3473a19aa2bede724bef53de8ba
2015-05-21 15:41:53 +02:00
Patrick Brosset 6abdd22399 Bug 1160972 - Minor code cleanup in inspector-panel.js for better consistency; r=miker
--HG--
extra : rebase_source : a2eed1ea7adeaab6dcc44836bcb3c8f0c611e909
2015-05-04 14:11:26 +02:00
Patrick Brosset eb37c89c88 Bug 1158822 - Link css url attributes to style-editor and js url attributes to debugger; r=jsantell
The node-attribute-parser now marks uris to css and js files as cssresource and jsresource.
Thanks to this, the inspector can open the corresponding files in the style-editor and
debugger rather than just opening a new tab with the source.
This makes use of 2 new toolbox methods: viewSourceInStyleEditor and viewSourceInDebugger.

--HG--
extra : rebase_source : fcca4fe73c147381c1fcccda4f7956c8d41b4c2b
extra : histedit_source : 84cda2275b42bd6196243a83e9f129f3c66faa6b%2C0ee06929ac85dfeafdd5ee66d842e32fc5395578
2015-05-04 09:23:04 +02:00
Patrick Brosset 7337cb49cd Bug 921102 - 4 - Tests for the open/copy links on markup-view attributes; r=bgrins
--HG--
extra : rebase_source : 7b27344f7a10816649f0a22e3fd3dd111e5d573f
extra : histedit_source : d03061e861ce17e73c9bbca15b9e62fbeed11e9e
2015-05-02 23:00:13 +02:00
Patrick Brosset 3a5e7d4a71 Bug 921102 - 3 - Open/copy markup-view attribute links; r=bgrins
This part adds contextual menu items that become enabled when
the user right clicks on an attribute that has a link.
Depending on the nature of the link, a new tab will be opened or a node
selected.
The user can also choose to copy the link in the clipboard.

--HG--
extra : rebase_source : 00128a076003ebac34096d81d9e326bee6631259
extra : histedit_source : c1d67e84e9d57280040e5831233df523fff904e2
2015-05-02 22:37:45 +02:00
Tooru Fujisawa 0edbeb71c8 Bug 1102219 - Part 5: Replace more `String.prototype.contains` with `String.prototype.includes` in chrome code. r=till 2015-04-30 00:32:05 +09:00
Jordan Santell 2513d09a8a Bug 1134778 - Consolidate usages of view sourcing in tools with a source-utils module. r=vp,jryans 2015-04-24 14:16:34 -07:00
Joe Walker 9cc5a4d6b3 Bug 1128988 - runAt support for commands/converters; r=bgrins
--HG--
rename : browser/devtools/main.js => browser/devtools/definitions.js
2015-04-23 10:24:49 +01:00
Patrick Brosset 57a6320951 Bug 1152279 - Give the inspector sidebar a default size so it can be collapsed; r=miker
--HG--
extra : rebase_source : 5c83ca3af0b6c96cc022b8baf47df9cfd7e471f7
2015-04-21 15:29:49 +02:00
John Kang 0dbd2d45d6 Bug 1143074 - "Inspector search box can't find nodes with IDs/classes that include CSS reserved characters" [r=pbrosset] 2015-04-21 02:56:00 +02:00
Jennifer Fong 7836d45e7a Bug 901250 - Add scroll into view menu item for the inspector. r=bgrins 2015-04-17 11:03:24 -07:00
Patrick Brosset 292adfbff7 Bug 1121528 - Avoid the inspector going blank when quickly navigating; r=bgrins
This prevents the WalkerActor from failing at handling various requests when
those requests are about DeaedNodes. Indeed, the async nature of the devtools
protocol means that the client could be asking for information about a node that
doesn't exist anymore because it was part of a page that was navigated away from.
The WalkerActor should know how to deal with such cases, so the patch adds a
number of early returns and a new test for them.
This helps avoiding the inspector-panel from going blank in some edge cases.

--HG--
extra : rebase_source : 26ef72eee71334f6fe467d46a486768296035b8a
extra : histedit_source : 697ddbd179ec2992706f20ca7baccbd4f90f9175%2C6e1130e38ed538809f3ee16fd9a5291fe60a85c1
2015-04-17 12:09:58 +02:00
Brian Grinstead 964f9f6d53 Bug 1153903 - Get rid of logspam during devtools talos test runs;r=pbrosset 2015-04-16 09:54:04 -07:00
Carsten "Tomcat" Book 6fc008ccfc Backed out changeset 145f6347bb11 (bug 1137285) depending on the backout of bug 1145049
--HG--
extra : rebase_source : 3910ece62d2aafe466ce3aff65ebd164052d11cf
2015-04-15 17:24:06 +02:00
Carsten "Tomcat" Book 26799a7307 Backed out changeset 120d272971a6 (bug 1137285)
--HG--
extra : rebase_source : 890cba751909152d0e7d7f51db43c10672831d6e
2015-04-15 17:23:18 +02:00
Carsten "Tomcat" Book 2c95d28b6b Backed out changeset 3570dbae06e2 (bug 1137285)
--HG--
extra : rebase_source : 1f3f448a438812c7e5810ebc8d735755936fdb72
2015-04-15 17:23:10 +02:00
Patrick Brosset 4ae0fca079 Bug 1149346 - First word in selector-search also matches classes and ids; r=harth 2015-04-15 08:15:20 -07:00
Alexandre Poirot 6469c6796e Bug 1137285 - Convert inspector test to be remote friendly r=pbrosset 2015-04-15 13:55:30 +02:00
Alexandre Poirot 21c1b20bb3 Bug 1137285 - Replace content.location = ... by a navigateTo request r=pbrosset 2015-04-15 13:55:29 +02:00
Alexandre Poirot 30fa572d38 Bug 1137285 - Convert addTab+openInspector to openInspectorForURL r=pbrosset 2015-04-15 13:55:29 +02:00
Patrick Brosset fd53c00637 Bug 988278 - Fixes ESCape keypress mess in the inspector to make sure the split console opens; r=miker
This fixes 2 problems related to the split console not opening when it should.
1 - After the inspector selection mode (pick mode) was canceled with ESC, pressing ESC once
again did not open the split console, because the toolbox did not have the focus.
2 - The markup-view tooltip (used to preview images) was eating the first ESC keypress when
the markup-view was focused, even though the tooltip was hidden. This was forcing users to
press ESC twice to open the split console.

--HG--
extra : rebase_source : 503e6f0a8fa2ef153f714af1361e85ad844a49d2
2015-04-13 10:51:49 +02:00
Patrick Brosset e6e0278825 Bug 966787 - 2 - Skip inspector breadcrumb updates when the output doesn't change; r=bgrins
Whenever something changed on the selected element (pseudo, attribute), the
breadcrumbs widget used to loop over all breadcrumbs buttons and re-create the
markup for each.
Now, we cache a string version of the text displayed in a button and compare
the new value to that in the loop, to skip DOM updates.

Additionally, the breadcrumbs widget used to update itself after all markup mutations
in the DOM tree displayed in the inspector. The update method now looks at the mutations
array and early return if none of them actually impact the displayed breadcrumbs.

--HG--
extra : rebase_source : 4945c42d03e5be856c92919aa271811e66f3d188
2015-04-13 10:22:05 +02:00
Patrick Brosset b97af70f4e Bug 966787 - 1 - Code cleanup in breadcrumbs.js; r=bgrins
For a better consistency with the rest of the DevTools codebase.

--HG--
extra : rebase_source : 7bf01b3c7bc42809fbc8c0aacbceccc23e311c77
2015-04-08 12:16:25 +02:00
Patrick Brosset 4aa03199c3 Bug 873443 - Make the inspector search suggest and select elements in frames; r=bgrins
This makes the inspector search input show suggestions for nodes in all documents in
the content page. Selecting a suggestion also now selects the corresponding nodes
even when they are inside nested frames.

--HG--
extra : rebase_source : e7d3ac809ea8f223acfb5d04dd72a9e6a292796d
2015-04-09 17:11:06 +02:00
Patrick Brosset e0255d2129 Bug 994055 - Part 2: Add tests for the inspector sidebar toggle button. r=miker
--HG--
extra : transplant_source : %25O%DE%DD%CA%AB%3Cb3%82%F2%82N%E4HV%C1%27%E72
2015-04-03 17:11:15 +02:00
Patrick Brosset c0f73f550b Bug 994055 - Part 1: Add a toggle sidebar panel button to the inspector. r=miker
This adds a new icon, next to the searchbox (like in the debugger) to expand
or collapse the sidebar panel in the inspector.
The state is *not* persisted yet when the toolbox is closed.
The button is hidden when the toolbox host switches to 'side'.

--HG--
extra : transplant_source : %D0%09%C2%0E%23%94%B0%A9Ym%BF%1B%60%3C%A5%D6D%C3%92%DE
2015-04-03 16:39:33 +02:00
Matteo Ferretti 39806788c1 Bug 1144163 - Add a rulers highlighter; added unit test. r=pbrosset
- Added unit test for Rulers Highlighter
 - Added `Test:ScrollWindow` function to doc_frame_script.js
2015-03-28 00:25:18 +01:00
Panos Astithas e4b9084401 Add a devtools tag to all devtools tests (bug 1149907). r=pbrosset 2015-04-01 14:09:33 +03:00
Sami Jaktholm 9b756b2add Bug 1035124 - Move browser_inspector_update-on-navigation.js from inspector to markup view tests. r=pbrosset
--HG--
rename : browser/devtools/inspector/test/browser_inspector_update-on-navigation.js => browser/devtools/markupview/test/browser_markupview_update-on-navigtion.js
2015-03-13 23:23:01 +02:00
Patrick Brosset 0140de65b4 Bug 1139925 - Make the BoxModelHighlighter highlight all quads and draw guides around the outer-most rect
LayoutHelpers.getAdjustedBoxQuads now returns all quads that el.getBoxQuads returns.
The BoxModelHighlighter calculates an outer rect based on these to draw the guides.
And if the element has more than 1 quad (inline element that spans line breaks), then all quads are
highlighted.
Also all related tests were modified and a couple of new tests were added.

--HG--
extra : rebase_source : 78a7890ac547a95561ac917c1721be6ed32c5426
2015-03-13 15:54:10 +01:00
Sami Jaktholm ffa08e1726 Bug 1035140 - Split browser_inspector_menu-0?.js into multiple tests. r=pbrosset
--HG--
rename : browser/devtools/inspector/test/browser_inspector_menu-02.js => browser/devtools/inspector/test/browser_inspector_menu-03-paste-items.js
rename : browser/devtools/inspector/test/browser_inspector_menu-01.js => browser/devtools/inspector/test/browser_inspector_menu-04-other.js
rename : browser/devtools/inspector/test/doc_inspector_menu-02.html => browser/devtools/inspector/test/doc_inspector_menu.html
extra : rebase_source : 7d4fb3527bd0103859ce9aaee40974aa98b26ac5
2015-03-14 13:53:32 +02:00
Abhishek Bhattacharya 45edc26f1b Bug 1120111 - Tests for the highlighter pick mode key listeners; r=pbrosset 2015-03-10 07:04:00 +01:00
Patrick Brosset b6f31b8647 Bug 1123851 - 2 - Tests for the element geometry highlighter; r=bgrins 2015-03-09 17:33:24 +01:00
Patrick Brosset 7f57c217fd Bug 1074836 - 8 - Remove the gloal skip-if=e10s on browser/devtools/inspector tests; r=me 2015-02-09 10:03:07 +01:00
Patrick Brosset 8d93385c33 Bug 1074836 - 7 - Fix the inspector node reselection test with e10s; r=jwalker 2015-02-09 10:03:06 +01:00
Patrick Brosset 155f70201e Bug 1074836 - 6 - Fix the inspector search field test with e10s; r=jwalker 2015-02-09 10:03:06 +01:00
Patrick Brosset 306859f127 Bug 1074836 - 5 - Disable iframe scrolling inspector test with e10s; r=me 2015-02-09 10:03:06 +01:00
Patrick Brosset fa1773640a Bug 1074836 - 4 - Fix the iframe deletion inspector test with e10s; r=bgrins 2015-02-09 10:03:06 +01:00
Patrick Brosset 25bcad199c Bug 1074836 - 3 - Fix the browser ctx-menu inspector test with e10s; r=bgrins 2015-02-09 10:03:06 +01:00
Patrick Brosset cf7d250aca Bug 1074836 - 2 - Disable the gcli related test in browser/devtools/inspector with e10s; r=me 2015-02-09 10:03:06 +01:00
Patrick Brosset b0464366bf Bug 1074836 - 1 - Use both actorID and connPrefix to retrieve the right highlighter actors during inspector tests; r=miker 2015-02-09 10:03:06 +01:00