Skip to content

Commit

Permalink
- Fixed linting changes
Browse files Browse the repository at this point in the history
  • Loading branch information
akshaydeo committed Jul 25, 2022
1 parent ab3f7d3 commit c2472b4
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 53 deletions.
19 changes: 2 additions & 17 deletions docs/api/context-bridge.md
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ contextBridge.exposeInMainWorld(
)
```

An example of complex API exposed in an isolated world is shown below:
An example of API exposed in an isolated world is shown below:

```javascript

Expand All @@ -104,22 +104,7 @@ contextBridge.exposeInIsolatedWorld(
1005,
'electron',
{
doThing: () => ipcRenderer.send('do-a-thing'),
myPromises: [Promise.resolve(), Promise.reject(new Error('whoops'))],
anAsyncFunction: async () => 123,
data: {
myFlags: ['a', 'b', 'c'],
bootTime: 1234
},
nestedAPI: {
evenDeeper: {
youCanDoThisAsMuchAsYouWant: {
fn: () => ({
returnData: 123
})
}
}
}
doThing: () => ipcRenderer.send('do-a-thing')
}
)

Expand Down
23 changes: 4 additions & 19 deletions lib/renderer/api/context-bridge.ts
Original file line number Diff line number Diff line change
@@ -1,11 +1,7 @@
const binding = process._linkedBinding('electron_renderer_context_bridge');

const checkContextIsolationEnabled = () => {
if (!process.contextIsolated) {
throw new Error(
'contextBridge API can only be used when contextIsolation is enabled'
);
}
if (!process.contextIsolated) throw new Error('contextBridge API can only be used when contextIsolation is enabled');
};

const contextBridge: Electron.ContextBridge = {
Expand All @@ -26,22 +22,11 @@ export const internalContextBridge = {
overrideGlobalValueFromIsolatedWorld: (keys: string[], value: any) => {
return binding._overrideGlobalValueFromIsolatedWorld(keys, value, false);
},
overrideGlobalValueWithDynamicPropsFromIsolatedWorld: (
keys: string[],
value: any
) => {
overrideGlobalValueWithDynamicPropsFromIsolatedWorld: (keys: string[], value: any) => {
return binding._overrideGlobalValueFromIsolatedWorld(keys, value, true);
},
overrideGlobalPropertyFromIsolatedWorld: (
keys: string[],
getter: Function,
setter?: Function
) => {
return binding._overrideGlobalPropertyFromIsolatedWorld(
keys,
getter,
setter || null
);
overrideGlobalPropertyFromIsolatedWorld: (keys: string[], getter: Function, setter?: Function) => {
return binding._overrideGlobalPropertyFromIsolatedWorld(keys, getter, setter || null);
},
isInMainWorld: () => binding._isCalledFromMainWorld() as boolean
};
Expand Down
28 changes: 11 additions & 17 deletions spec-main/api-context-bridge-spec.ts
Original file line number Diff line number Diff line change
@@ -1,23 +1,17 @@
import * as cp from 'child_process';
import { BrowserWindow, ipcMain } from 'electron/main';
import { contextBridge } from 'electron/renderer';
import { expect } from 'chai';
import * as fs from 'fs-extra';
import * as http from 'http';
import * as os from 'os';
import * as path from 'path';
import * as cp from 'child_process';

import { BrowserWindow, ipcMain } from 'electron/main';

import { AddressInfo } from 'net';
import { closeWindow } from './window-helpers';
import { contextBridge } from 'electron/renderer';
import { emittedOnce } from './events-helpers';
import { expect } from 'chai';
import { AddressInfo } from 'net';

const fixturesPath = path.resolve(
__dirname,
'fixtures',
'api',
'context-bridge'
);
const fixturesPath = path.resolve(__dirname, 'fixtures', 'api', 'context-bridge');

describe('contextBridge', () => {
let w: BrowserWindow;
Expand All @@ -29,9 +23,7 @@ describe('contextBridge', () => {
res.setHeader('Content-Type', 'text/html');
res.end('');
});
await new Promise<void>((resolve) =>
server.listen(0, '127.0.0.1', resolve)
);
await new Promise<void>((resolve) => server.listen(0, '127.0.0.1', resolve));
});

after(async () => {
Expand Down Expand Up @@ -862,7 +854,8 @@ describe('contextBridge', () => {
throwNotClonable: () => {
return Object(Symbol('foo'));
},
argumentConvert: () => {}
argumentConvert: () => {
}
});
});
const result = await callWithBindings((root: any) => {
Expand Down Expand Up @@ -1988,7 +1981,8 @@ describe('contextBridge', () => {
throwNotClonable: () => {
return Object(Symbol('foo'));
},
argumentConvert: () => {}
argumentConvert: () => {
}
});
});
const result = await callWithBindings((root: any) => {
Expand Down

0 comments on commit c2472b4

Please sign in to comment.