Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(82)

Issue 535883002: Use JavaScript to detect if Selection.modify is available. (Closed)

Created:
6 years, 3 months ago by ahe
Modified:
6 years, 3 months ago
Reviewers:
aam-me, kasperl
CC:
reviews_dartlang.org, lukechurch, Johnni Winther
Visibility:
Public.

Description

Use JavaScript to detect if Selection.modify is available. R=kasperl@google.com Committed: https://code.google.com/p/dart/source/detail?r=39806

Patch Set 1 #

Total comments: 2

Patch Set 2 : Merged with r39804 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+88 lines, -9 lines) Patch
M dart/site/try/index.html View 1 chunk +8 lines, -0 lines 0 comments Download
M dart/site/try/src/interaction_manager.dart View 2 chunks +2 lines, -7 lines 0 comments Download
M dart/site/try/src/messages.dart View 1 chunk +4 lines, -1 line 0 comments Download
M dart/site/try/src/settings.dart View 1 2 chunks +4 lines, -0 lines 0 comments Download
M dart/tests/try/firefox.applescript View 1 chunk +70 lines, -1 line 0 comments Download

Messages

Total messages: 8 (1 generated)
ahe
6 years, 3 months ago (2014-09-03 09:55:37 UTC) #2
kasperl
LGTM. https://codereview.chromium.org/535883002/diff/1/dart/tests/try/firefox.applescript File dart/tests/try/firefox.applescript (right): https://codereview.chromium.org/535883002/diff/1/dart/tests/try/firefox.applescript#newcode54 dart/tests/try/firefox.applescript:54: delay 0.1 I trust this does what you ...
6 years, 3 months ago (2014-09-03 12:55:18 UTC) #3
ahe
Thank you, Kasper! https://codereview.chromium.org/535883002/diff/1/dart/tests/try/firefox.applescript File dart/tests/try/firefox.applescript (right): https://codereview.chromium.org/535883002/diff/1/dart/tests/try/firefox.applescript#newcode54 dart/tests/try/firefox.applescript:54: delay 0.1 On 2014/09/03 12:55:18, kasperl ...
6 years, 3 months ago (2014-09-03 13:10:23 UTC) #4
ahe
Committed patchset #2 (id:20001) manually as 39806 (presubmit successful).
6 years, 3 months ago (2014-09-03 13:44:40 UTC) #5
aam-me
On 2014/09/03 13:10:23, ahe wrote: > Thank you, Kasper! > > https://codereview.chromium.org/535883002/diff/1/dart/tests/try/firefox.applescript > File dart/tests/try/firefox.applescript ...
6 years, 3 months ago (2014-09-04 11:17:24 UTC) #6
ahe
On 2014/09/04 11:17:24, aam wrote: > Hi Peter, did you have to do some special ...
6 years, 3 months ago (2014-09-04 11:36:09 UTC) #7
aam-me
6 years, 3 months ago (2014-09-04 11:49:13 UTC) #8
Message was sent while issue was closed.
On 2014/09/04 11:36:09, ahe wrote:
> On 2014/09/04 11:17:24, aam wrote:
> > Hi Peter, did you have to do some special set up to make those Firefox,
Safari
> > scripts work?
> 
> Not besides running them on my Mac. Did you run them on a Mac?

Yes.

> > I tried both unsuccessfully. I found that with Firerfox 31 I need to have
more
> > tabs(9 vs 4) to get to the main editor div and then still it doesn't quite
> > work(delete deletes curly brace after "void main()" in Hello World example),
> > with Safari bookmark tab gets in the way of tabbing to main editor div as
> well.
> 
> Yeah. Tabbing doesn't work. I wanted to see if I could use the same approach
to
> test IE11 via Remote Desktop, but it doesn't work.
> 
> So I've considered using accesskey attributes to make them more reliable.

Ah, okay.

Powered by Google App Engine
This is Rietveld 408576698