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

Issue 2593093002: Instant Apps: Add instrumentation tests. (Closed)

Created:
4 years ago by Maria
Modified:
4 years ago
Reviewers:
gone
CC:
chromium-reviews, skanuj+watch_chromium.org, melevin+watch_chromium.org, dhollowa+watch_chromium.org, dougw+watch_chromium.org, donnd+watch_chromium.org, jfweitz+watch_chromium.org, David Black, samarth+watch_chromium.org, kmadhusu+watch_chromium.org, Jered
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

Instant Apps: Add instrumentation tests. Adds some of the missing tests for instant apps. BUG=676369, 675757 Review-Url: https://codereview.chromium.org/2550413007 Cr-Commit-Position: refs/heads/master@{#439604} (cherry picked from commit 916f90ce1912767f926d0389f988f49e8b1799c2) R=dfalcantara@chromium.org Committed: https://chromium.googlesource.com/chromium/src/+/b7e00b43872a25473a2e785119673c67cc0849ff

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+125 lines, -7 lines) Patch
M chrome/android/javatests/src/org/chromium/chrome/browser/externalnav/ExternalNavigationHandlerTest.java View 8 chunks +16 lines, -4 lines 0 comments Download
M chrome/android/javatests/src/org/chromium/chrome/browser/instantapps/InstantAppsHandlerTest.java View 3 chunks +109 lines, -3 lines 0 comments Download

Messages

Total messages: 6 (3 generated)
Maria
4 years ago (2016-12-21 18:29:38 UTC) #3
gone
lgtm
4 years ago (2016-12-21 18:31:33 UTC) #4
Maria
4 years ago (2016-12-21 18:36:25 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
b7e00b43872a25473a2e785119673c67cc0849ff (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698