Reload remote tab from about:devtools-toolbox
Categories
(DevTools :: about:debugging, enhancement, P3)
Tracking
(firefox78 fixed)
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: Harald, Assigned: davidwalsh)
References
(Blocks 1 open bug)
Details
User Story
When testing my developer site on my phone, I want to quickly reload to fetch new data or reset state, so that I don't have to open menus on my phone.
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Similar to bug 1617237.
Optional: Can we map F5 and variants to reload the remote page?
Reporter | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
The design for Reload button is provided from Victoria.
https://bugzilla.mozilla.org/attachment.cgi?id=9140285
And also, if possible, let's try to implement Back/Forward buttons as well in here since I think it takes not much time.
Assignee | ||
Comment 2•5 years ago
|
||
:daisuke I look forward to implementing this. Could you provide a bit of info on the technical side? i.e. how we can send the message to reload via remote debugging?
Thank you!
Comment 3•5 years ago
|
||
Hi David! Thank you very much!
Regarding UI:
The component that shows the debuggee's name, URI and so on is DebugTargetInfo.
I think we can add Reload button to the component.
Regarding reloading the debuggee:
When we debug a tab, the target of toolbox that we can refer from DebugTargetInfo component is a front of BrowsingContextTargetActor.
And as the actor has reload function, we can reload using the function.
Regarding Back/Forward feature:
I had thought those functions are in BrowsingContextTargetActor, but it seems we don't have yet.
So, if we realize the feature, need to add and expose the functions in the actor and corresponding front.
(And check the backward compatibility)
So, it might be better to fix it in another bug.
Now, Micah is tackling to fix bug 1617237, but we can refer the code since modifying very similar place.
https://phabricator.services.mozilla.com/D70279
https://phabricator.services.mozilla.com/D70280
Hm, so, it might be better to fix this after bug 1617237 in order to avoid conflicts..
Assignee | ||
Comment 4•5 years ago
|
||
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 6•5 years ago
|
||
bugherder |
Description
•