-
-
Notifications
You must be signed in to change notification settings - Fork 153
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Chrome browser event in database during recording #744
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @KrishPatel13 ! I left a few comments, happy to chat about any of it if you like.
If you could show some console output and/or screenshots in the PR description that would be very much appreciated!
An update for Chrome Extension Issue: After spending about 4-5 hours debugging nativeMessaging, I was not able to fix the issue. I dropped the idea of using nativeMessaging. Reasons:
Hence, After an hour of research, I recalled that I had used WebSockets in my assignment in CSC309 to communicate between an web server and my java running process. The task there was that whenever the user click submit I had to sent that api request to my java running client and then, connect to db and save some info about the user (registering). I thought this might relevant for the task that we are working on too. Hence, I quickly made a web_socket_extension example to demo this idea for constant communication between extension and a desktop running process (in our case a .py file). I made a dumpy Commit for the Web Socket "Ping" Example: |
My next step would be to log the dom changes instead of "ping" message. |
My goal is to at least, make a prototype by tonight: record.py (dummy): that runs and save all the dom changes in a sqlite db and once this is done, I will then manipulate the actual record.py next week so that it records the dom changes in our actual db. (our main goal). I hope this sounds like a good plan to tackle this issue :-) |
Update: Now I am able to log DOM changes Commit: d9f9792 |
Now, I can save all the DOM messages sent by the extension into a db synchronously. Commit: 1b606fb Some Screenshots for the work: and This commit proves a working prototype for the chrome extension issue. Now my next steps shall be to integrate this into our actual application. |
04d4658
to
1b606fb
Compare
FIxed Visualization here: b3d54eb I just ran visualize on a browser_event record recording. Here are some screenshots to support: |
8a20800
to
c3d1275
Compare
c3d1275
to
925db61
Compare
What kind of change does this PR introduce?
This PR implements the chrome extension to write DOM and DOM events to the database.
Summary
Motivation: Related Issue: #51
Checklist
Other information
Supercedes #364