Bug 118676 – Web Inspector: Master bug for moving to the new inspector on all ports

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Roland Takacs]

As you know we should use the Safari inspector UI (Source/WebInspectorUI) inspead of the old one (Source/WebCore/inspector/front-end).
In this place all ports can track the bugs that related to the Safari inspector UI integration.

[Radar WebKit Bug Importer]

<rdar://problem/14444151>

[Seokju Kwon]

I wonder whether 'New Web Inspector' can pass LayoutTest related to Inspector or not.

[Timothy Hatcher]

(In reply to comment #2)
> I wonder whether 'New Web Inspector' can pass LayoutTest related to Inspector or not.

It can't. We plan to make a new test harness soon.

[Marcelo Morais]

Hello, now that efl/gtk/mac/win use the new inspector, I would like to start removing the old code (qt was the last port with the old inspector, and is being removed from trunk), is that ok for you guys?

[Timothy Hatcher]

Yes! The only question about removing the old is test coverage. It would be nice to convert some of the old Inspector UI tests over to protocol tests. (We don't plan to do any UI testing of the new Inspector but we want to test the protocol.)

[Marcelo Morais]

(In reply to comment #5)
> Yes! The only question about removing the old is test coverage. It would be nice to convert some of the old Inspector UI tests over to protocol tests. (We don't plan to do any UI testing of the new Inspector but we want to test the protocol.)

I see... have you decided which tests should be converted?

How can we proceed? Maybe creating separated bugs, one to remove the old front end and another to convert these tests and remove the rest? Suggestions?

[Marcelo Morais]

Hi again, I created the bugs 122294 and 122295, one to remove the old front-end and another to check which UI tests can be converted.

I will remove the old front end first and leave the tests untouched.

I don't know if you already thought about which tests should be converted, if you don't maybe I can take a look at this later, I think that would be a good opportunity to me to learn about this. 

Is that ok for you?

[Joseph Pecoraro]

(In reply to comment #7)
> Hi again, I created the bugs 122294 and 122295, one to remove the old front-end and another to check which UI tests can be converted.
> 
> I will remove the old front end first and leave the tests untouched.

If you remove the old frontend you will need to skip all of the LayoutTests/inspector and LayoutTests/http/tests/inspector tests on all platforms, because they rely on running code inside the old frontend.


> I don't know if you already thought about which tests should be converted, if you don't maybe I can take a look at this later, I think that would be a good opportunity to me to learn about this. 
> 
> Is that ok for you?

I think this is fine. It would be nice to have a strategy for converting over tests. We haven't really had much time to think about this.

Ideally we would have at least one test for each protocol method, then perhaps extra tests for cases involving multiple back and forths between a frontend and backend.

In the process of writing the new tests we can remove old tests that tested the same protocol methods. You mention you are interested, it would be nice to talk a small piece of the protocol, maybe the Storage domain, and write new tests for that. I think it would be a nice small task, and we can see from there how easy / difficult it would be to convert the rest.

I started working on the Debugger domain. See LayoutTests/inspector-protocol/debugger as an example on how to structure and write the tests.

[Marcelo Morais]

> If you remove the old frontend you will need to skip all of the LayoutTests/inspector and LayoutTests/http/tests/inspector tests on all platforms, because they rely on running code inside the old frontend.
>

Yes, I'll do this.


> I think this is fine. It would be nice to have a strategy for converting over tests. We haven't really had much time to think about this.
> 
> Ideally we would have at least one test for each protocol method, then perhaps extra tests for cases involving multiple back and forths between a frontend and backend.
> 
> In the process of writing the new tests we can remove old tests that tested the same protocol methods. You mention you are interested, it would be nice to talk a small piece of the protocol, maybe the Storage domain, and write new tests for that. I think it would be a nice small task, and we can see from there how easy / difficult it would be to convert the rest.
> 
> I started working on the Debugger domain. See LayoutTests/inspector-protocol/debugger as an example on how to structure and write the tests.

Nice, I'll take a look at this. Like I said it is a good opportunity to learn and I want to help. :)

[Blaze Burg]

Since this bug was created, I believe all ports have moved to the new inspector or have dropped out of trunk. There are still follow-up bugs for converting skipped inspector UI tests into inspector protocol tests. But, this master bug doesn't seem to 'block' anything now.