Dry run: Try jobs failed on following builders: linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/423614)
3 years, 8 months ago
(2017-04-05 03:56:33 UTC)
#7
lgtm for code change Please update description as self explained with What, Why, Outcome, References ...
3 years, 8 months ago
(2017-04-05 04:01:24 UTC)
#9
lgtm for code change
Please update description as self explained with What, Why, Outcome, References
for renaming.
In this way, people doesn't need to read discussion in review.
tkent
https://codereview.chromium.org/2802543002/diff/1/third_party/WebKit/Source/core/testing/Internals.cpp File third_party/WebKit/Source/core/testing/Internals.cpp (right): https://codereview.chromium.org/2802543002/diff/1/third_party/WebKit/Source/core/testing/Internals.cpp#newcode1009 third_party/WebKit/Source/core/testing/Internals.cpp:1009: if (equalIgnoringCase(matchStatus, "Active")) Please don't use equalIgnoringCase(). It's unnecessarily ...
3 years, 8 months ago
(2017-04-05 04:16:32 UTC)
#10
Description was changed from ========== Update window.internals.addTextMatchMarker() to take enum instead of bool Suggested by ...
3 years, 8 months ago
(2017-04-05 18:03:40 UTC)
#11
Description was changed from
==========
Update window.internals.addTextMatchMarker() to take enum instead of bool
Suggested by @yosin here:
https://codereview.chromium.org/2801483002#msg20
BUG=707867
==========
to
==========
Update window.internals.addTextMatchMarker() to take enum instead of bool
Suggested by @yosin here:
https://codereview.chromium.org/2801483002#msg20
We are changing DocumentMarkerController::addTextMatchMarker() to take an enum
in https://codereview.chromium.org/2801483002 since it's unclear from looking at
a callsite what "true" and "false" mean. This CL handles the window.internals
changes separately since a bunch of tests have to be updated.
BUG=707867
==========
rlanday
The CQ bit was checked by rlanday@chromium.org to run a CQ dry run
3 years, 8 months ago
(2017-04-05 19:08:05 UTC)
#12
Dry run: Try jobs failed on following builders: mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_ng/builds/422878)
3 years, 8 months ago
(2017-04-05 20:23:34 UTC)
#15
Dry run: Try jobs failed on following builders: linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/424332)
3 years, 8 months ago
(2017-04-05 22:23:13 UTC)
#25
Issue 2802543002: Update window.internals.addTextMatchMarker() to take enum instead of bool
(Closed)
Created 3 years, 8 months ago by rlanday
Modified 3 years, 8 months ago
Reviewers: tkent, Xiaocheng, yosin_UTC9
Base URL:
Comments: 1