feat: support grammarly

hugomano commented 5 years ago

Objective

Support Grammarly extension

What this PR does

Added chrome.cookies methods and events support
Added chrome.tabs.query basic support
Stub chrome.tabs.insertCSS
Remove the background page persistence partition to be iso with Chromium: https://cs.chromium.org/chromium/src/chrome/browser/background/background_contents_service.cc?dr=CSs&g=0&l=678
Changed stubs for browser actions: call callback even if we do nothing
Fixed Port
Add API calls logger to see what is doing in BG and CS
Add ECx.sendEvent() for forward chrome API event from app
Add readme pieces of stuff
Refactor chrome.windows
Add framework for simplifying API additions

Acceptance tests

UI spellcheck feedback
Popup Editor
Extension onboarding
Textarea focus / unfocus
Tab switching
Tab reloading

alexstrat commented 5 years ago

@hugomano d8c4c86 seems to have unblocked your issue.

hugomano commented 5 years ago

24/04 - 18h00

Verify that emitBackground("content-script-inited") works well

Why createAndObserve takes L?

var F = i.createAndObserve(L).actions;

function L(e) {

createMirrorStore is not very well synchronized (@alexstrat hypothesis)

hugomano commented 5 years ago

todo: emit correctly chrome.tabs events

hugomano commented 5 years ago

tabs.onUpdatedevents are missing

chrome.tabs.onUpdated.addListener(function(e,t){n.getActiveTab().then(function(n){n&&n.id===e&&(t.url||t.favIconUrl||"complete"===t.status)&&chrome.tabs.get(e,function(e){return o(e)})})})

hugomano commented 5 years ago

same with tabs.onActivated

alexstrat commented 5 years ago

For the record, can you detail what was broken with Port please?

hugomano commented 5 years ago

Regarding https://github.com/getstation/electron-chrome-extension/pull/42#issuecomment-498296642, the MessageSender#url embedded in the Port was missing.

getstation / electron-chrome-extension