foolip
2017/04/17 08:46:04
Pleas exclude "test" from the filename, just metho
Pleas exclude "test" from the filename, just methodName.https.html to match the
method under test would be OK. Or navigator-methodName.https.html perhaps.
On 2017/04/17 08:46:04, foolip_UTC7 wrote:
> Pleas exclude "test" from the filename, just methodName.https.html to match
the
> method under test would be OK. Or navigator-methodName.https.html perhaps.
foolip
2017/04/17 08:46:04
https://garykac.github.io/system-keyboard-lock/ do
https://garykac.github.io/system-keyboard-lock/ doesn't have a cancelKeyLock()
method, can you update the spec if you've all agreed to the name change?
You could also assert that the return value is undefined so that if it's changed
to return a promise, this test also has to be updated.
Hzj_jie
2017/04/18 02:26:07
Yes, Gary is working on changing the spec. There a
On 2017/04/17 08:46:04, foolip_UTC7 wrote:
> https://garykac.github.io/system-keyboard-lock/ doesn't have a cancelKeyLock()
> method, can you update the spec if you've all agreed to the name change?
>
> You could also assert that the return value is undefined so that if it's
changed
> to return a promise, this test also has to be updated.
Yes, Gary is working on changing the spec. There are also several other changes
required.
10 }, 'Keyboard Lock cancelKeyLock test');
foolip
2017/04/17 08:46:04
Omit "test" from titles as well. Since this is the
Omit "test" from titles as well. Since this is the only test in the file, you
can move the title into <title>. Or omit <title>, in any case two titles aren't
useful.
On 2017/04/17 08:46:04, foolip_UTC7 wrote:
> Omit "test" from titles as well. Since this is the only test in the file, you
> can move the title into <title>. Or omit <title>, in any case two titles
aren't
> useful.
Issue 2805763004: [System-Keyboard-Lock] Forward navigator functions to RenderFrameHost
(Closed)
Created 3 years, 8 months ago by Hzj_jie
Modified 3 years, 7 months ago
Reviewers: whywhat, Avi (use Gerrit), dcheng, foolip, haraken, esprehn
Base URL:
Comments: 209