machenbach
36ebaf21c6
Revert of [inspector] fix timestamp formatting with non C locales (patchset #7 id:120001 of https://codereview.chromium.org/2410933002/ )
...
Reason for revert:
Breaks layout tests:
https://build.chromium.org/p/client.v8.fyi/builders/V8-Blink%20Linux%2064/builds/10548
See also:
https://github.com/v8/v8/wiki/Blink-layout-tests
Original issue's description:
> [inspector] fix timestamp formatting with non C locales
>
> If current locale has "," as decimal separator then message for consoleAPICalled will be corrupted.
>
> BUG=chromium:653424
> R=dgozman@chromium.org
>
> Committed: https://crrev.com/dde5ef75cbac1eb7e2dae59b246e4a0d0ba6a0f4
> Cr-Commit-Position: refs/heads/master@{#40190}
TBR=dgozman@chromium.org ,kozyatinskiy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=chromium:653424
Review-Url: https://codereview.chromium.org/2419453002
Cr-Commit-Position: refs/heads/master@{#40200}
2016-10-12 08:18:36 +00:00
kozyatinskiy
dde5ef75cb
[inspector] fix timestamp formatting with non C locales
...
If current locale has "," as decimal separator then message for consoleAPICalled will be corrupted.
BUG=chromium:653424
R=dgozman@chromium.org
Review-Url: https://codereview.chromium.org/2410933002
Cr-Commit-Position: refs/heads/master@{#40190}
2016-10-11 23:22:07 +00:00
kozyatinskiy
3b6c7f04a9
[inspector] fixed one more size_t -> int conversion
...
This problem was detected only on linux64_gyp bot.
It's safe to convert length string into int from size_t.
R=dgozman@chromium.org
Review-Url: https://codereview.chromium.org/2402583004
Cr-Commit-Position: refs/heads/master@{#40101}
2016-10-07 22:16:04 +00:00
kozyatinskiy
2d5cc49b72
[inspector] don't use String16 in inspector test runner
...
String16 is not public part of src/inspector. All usage are replaced with vector of char/unit16_t to avoid potential linker problems.
BUG=chromium:635948
R=dgozman@chromium.org ,machenbach@chromium.org
Review-Url: https://codereview.chromium.org/2403493002
Cr-Commit-Position: refs/heads/master@{#40098}
2016-10-07 21:16:55 +00:00
kozyatinskiy
186e7db8dd
[inspector] fix compilation on win and linux
...
BUG=chromium:635948
R=dgozman@chromium.org ,machenbach@chromium.org
Review-Url: https://codereview.chromium.org/2389133004
Cr-Commit-Position: refs/heads/master@{#40003}
2016-10-05 17:07:46 +00:00
kozyatinskiy
e90a79c637
[inspector] introduced exceptionThrown support in test runner
...
BUG=chromium:635948
R=dgozman@chromium.org
Review-Url: https://codereview.chromium.org/2384373002
Cr-Commit-Position: refs/heads/master@{#39998}
2016-10-05 15:08:14 +00:00
kozyatinskiy
24beac30ee
[inspector] Make InspectorTest.sendCommand* private
...
Introduce Protocol.Domain.method(args) and Protocol.Domain.onEventName() instead.
Renamed InspectorTest.evaluateInPage -> InspectorTest.addScript.
Improved InspectorTest.logMessage.
BUG=chromium:635948
R=dgozman@chromium.org ,alph@chromium.org
Review-Url: https://codereview.chromium.org/2390733002
Cr-Commit-Position: refs/heads/master@{#39942}
2016-10-03 23:33:07 +00:00
kozyatinskiy
f0649c8f08
[inspector] added inspector test runner [part 3]
...
- added test runner, that takes file names and V8 flags as arguments and run scripts from passed files with passed flags in frontend context
BUG=chromium:635948
R=dgozman@chromium.org ,alph@chromium.org
Committed: https://crrev.com/f3f9f4448dfa533d768878245a9bdbb57b4d941b
Review-Url: https://codereview.chromium.org/2372793002
Cr-Original-Commit-Position: refs/heads/master@{#39891}
Cr-Commit-Position: refs/heads/master@{#39929}
2016-10-02 18:10:39 +00:00
machenbach
ee0d69910b
Revert "[inspector] added inspector test runner [part 3-5]"
...
Revert "[inspector] added inspector test runner [part 3]"
This reverts commit f3f9f4448d4a5f5d09919ddbdab195https://codereview.chromium.org/2379053003/
BUG=chromium:635948
TBR=kozyatinskiy@chromium.org ,
NOTRY=true
Review-Url: https://codereview.chromium.org/2379303002
Cr-Commit-Position: refs/heads/master@{#39908}
2016-09-30 09:25:31 +00:00
kozyatinskiy
f3f9f4448d
[inspector] added inspector test runner [part 3]
...
- added test runner, that takes file names and V8 flags as arguments and run scripts from passed files with passed flags in frontend context
BUG=chromium:635948
R=dgozman@chromium.org ,alph@chromium.org
Review-Url: https://codereview.chromium.org/2372793002
Cr-Commit-Position: refs/heads/master@{#39891}
2016-09-29 22:23:47 +00:00
