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

Issue 8440060: Revert 108391 - Broke 'compile' on Mac clobber builder (Closed)

Created:
9 years, 1 month ago by Ryan Sleevi
Modified:
9 years, 1 month ago
Reviewers:
Sheridan Rawlins, scr
CC:
chromium-reviews, pam+watch_chromium.org, Paweł Hajdan Jr., Nico
Visibility:
Public.

Description

Revert 108391 - Broke 'compile' on Mac clobber builder Allow javascript unit tests using webui test_api framework. I moved javascript2webui.js over to chrome/test/base/js2gtest.js and shared it across webui and unit tests with an extra parameter for test type, which governs the few differences, such as what to include/subclass from and the flavor of gtest (TEST_F v. IN_PROC_BROWSER_TEST_F). v8_unit_test implemented 2 main methods to make it work with the generated C++ - AddLibrary - loads the file in the test context - RunJavascriptF - launches the runTest with the testFixture and testName, coordinating with Error and ChromeSend to report results. Helper functions: - Error - watches for console.error, noting failure if seen. - ChromeSend - pulls apart the result from the test_api's call to chrome.send('testResult', [ok, msg]) R=arv@chromium.org BUG=87820 TEST=unit_tests --gtest_filter=FrameworkUnitTest.* Review URL: http://codereview.chromium.org/8418015 TBR=scr@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108407

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -386 lines) Patch
A + chrome/browser/ui/webui/javascript2webui.js View 0 chunks +-1 lines, --1 lines 0 comments Download
M chrome/chrome_tests.gypi View 5 chunks +9 lines, -38 lines 0 comments Download
D chrome/test/base/js2gtest.js View 1 chunk +0 lines, -81 lines 0 comments Download
M chrome/test/base/v8_unit_test.h View 2 chunks +12 lines, -44 lines 0 comments Download
M chrome/test/base/v8_unit_test.cc View 2 chunks +15 lines, -176 lines 0 comments Download
D chrome/test/data/unit/framework_unittest.js View 1 chunk +0 lines, -42 lines 0 comments Download
M tools/gypv8sh.py View 1 chunk +6 lines, -6 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Ryan Sleevi
9 years, 1 month ago (2011-11-03 02:51:41 UTC) #1
Ryan Sleevi
Possibly something with goma? http://build.chromium.org/p/chromium/builders/Mac/builds/8695/steps/compile/logs/stdio ====Building chrome:unit_tests Support (Release) ____PhaseScriptExecution Rule "js2unit" cd /b/build/slave/Mac/build/src/chrome /bin/sh ...
9 years, 1 month ago (2011-11-03 02:53:35 UTC) #2
scr
No, I think it's a missing gyp dependency on v8_shell that didn't foil me or ...
9 years, 1 month ago (2011-11-03 03:04:42 UTC) #3
Sheridan Rawlins
9 years, 1 month ago (2011-11-03 03:18:31 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698