Riley Tatum would like Justin Donnelly, Moe Ahmadi, Aviv Kiss, Jiacheng Guo, Duncan Mercer, Chromium LUCI CQ and rubber-...@appspot.gserviceaccount.com to review this change.
Reland "[next] Launch Omnibox and NTP Next features"
This reverts commit 76cca6bde4e5374b0e1d2b5d52aa7f9ff0afb84c.
Reason for revert: Fixing tests
Original change's description:
> Revert "[next] Launch Omnibox and NTP Next features"
>
> This reverts commit 2e92ae12ce9bc226bb996030a4f03e77ad62bb9b.
>
> Reason for revert: Causing multiple CI failures
>
> Failure Link: https://ci.chromium.org/ui/p/chrome/builders/ci/linux64-rel-ready/69253/overview
> https://ci.chromium.org/ui/p/chrome/builders/ci/mac64/296405/overview
> https://ci.chromium.org/ui/p/chromium/builders/ci/Win10%20Tests%20x64/110547/overview
>
> Original change's description:
> > [next] Launch Omnibox and NTP Next features
> >
> > Fixed: 505856800
> > Change-Id: Icaf8358228ba09e0fec80b2dfbec54206d8ceb5c
> > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7791453
> > Commit-Queue: Riley Tatum <rta...@google.com>
> > Reviewed-by: Justin Donnelly <jdon...@chromium.org>
> > Reviewed-by: Moe Ahmadi <mah...@chromium.org>
> > Reviewed-by: Aviv Kiss <aviv...@google.com>
> > Cr-Commit-Position: refs/heads/main@{#1654470}
>
> Bug: 505856800
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Change-Id: Ie15b93b7e00748e0967b444029e14bfaee7922fe
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/8017587
> Auto-Submit: Jiacheng Guo <g...@google.com>
> Owners-Override: Jiacheng Guo <g...@google.com>
> Commit-Queue: Jiacheng Guo <g...@google.com>
> Bot-Commit: rubber-...@appspot.gserviceaccount.com <rubber-...@appspot.gserviceaccount.com>
> Cr-Commit-Position: refs/heads/main@{#1654553}
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Bot-Commit | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Gerrit GetPureRevert API does not mark this CL as a pure revert. Learn more: go/rubber-stamper-user-guide.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
Removing myself from the reviewers since I'm not an owner of these files anymore.
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Code-Review | +1 |
| Code-Review | +1 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
| Commit-Queue | +2 |
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |
LUCI Bisection has identified this change as the cause of a test failure. See the analysis: https://ci.chromium.org/ui/p/chromium/bisection/test-analysis/b/5475346477481984
Sample build with failed test: https://ci.chromium.org/b/8676714509093895633
Affected test(s):
[://chrome/test\:browser_tests!gtest::DevToolsProtocolTest#CreateInDefaultContextById](https://ci.chromium.org/ui/test/chromium/:%2F%2Fchrome%2Ftest%5C:browser_tests%21gtest::DevToolsProtocolTest%23CreateInDefaultContextById?q=VHash%3Abd478c65e18258a9)
[://chrome/test\:browser_tests!gtest::DevToolsProtocolTest#HiddenTargetCanBeClosed](https://ci.chromium.org/ui/test/chromium/:%2F%2Fchrome%2Ftest%5C:browser_tests%21gtest::DevToolsProtocolTest%23HiddenTargetCanBeClosed?q=VHash%3Abd478c65e18258a9)
[://chrome/test\:browser_tests!gtest::DevToolsProtocolTest#HiddenTargetIsNotVisibleInTabStrip](https://ci.chromium.org/ui/test/chromium/:%2F%2Fchrome%2Ftest%5C:browser_tests%21gtest::DevToolsProtocolTest%23HiddenTargetIsNotVisibleInTabStrip?q=VHash%3Abd478c65e18258a9)
[://chrome/test\:browser_tests!gtest::DevToolsProtocolTest#HiddenTargetIsTheLastOne](https://ci.chromium.org/ui/test/chromium/:%2F%2Fchrome%2Ftest%5C:browser_tests%21gtest::DevToolsProtocolTest%23HiddenTargetIsTheLastOne?q=VHash%3Abd478c65e18258a9)
[://chrome/test\:browser_tests!gtest::DevToolsProtocolTest#OpensDevTools_OpensForTabTarget](https://ci.chromium.org/ui/test/chromium/:%2F%2Fchrome%2Ftest%5C:browser_tests%21gtest::DevToolsProtocolTest%23OpensDevTools_OpensForTabTarget?q=VHash%3Abd478c65e18258a9)
and 34 more ...
A revert for this change was not created because the builder that this CL broke is not watched by gardeners, therefore less important. You can consider revert this CL, fix forward or let builder owners resolve it themselves.
If this is a false positive, please report it at http://b.corp.google.com/createIssue?component=1199205&description=Analysis%3A+https%3A%2F%2Fci.chromium.org%2Fui%2Fp%2Fchromium%2Fbisection%2Ftest-analysis%2Fb%2F5475346477481984&format=PLAIN&priority=P3&title=Wrongly+blamed+https%3A%2F%2Fchromium-review.googlesource.com%2Fc%2Fchromium%2Fsrc%2F%2B%2F8027107&type=BUG
| Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. |