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

Issue 2546583004: [NTP] Do not force offline version when opening non-download snippets. (Closed)

Created:
4 years ago by Marc Treib
Modified:
4 years ago
Reviewers:
CC:
chromium-reviews
Target Ref:
refs/pending/branch-heads/2924
Project:
chromium
Visibility:
Public.

Description

[NTP] Do not force offline version when opening non-download snippets. Previously we always opened an offline page, if there was one. This CL simply opens the URL for non-download snippets and then Offline Pages decide whether to open its offline page. For Downloads nothing changed. BUG=668485 Review-Url: https://codereview.chromium.org/2525323003 Cr-Commit-Position: refs/heads/master@{#434445} (cherry picked from commit 6f538029f3659bf667470deca0b22abdcd7cafbd) Committed: https://chromium.googlesource.com/chromium/src/+/c9a10dcad458b7a05ca295a25ae621c30f264f05

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -4 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java View 1 chunk +8 lines, -4 lines 0 comments Download

Messages

Total messages: 3 (1 generated)
Marc Treib
Committed patchset #1 (id:1) manually as c9a10dcad458b7a05ca295a25ae621c30f264f05.
4 years ago (2016-12-01 14:31:28 UTC) #2
whywhat
4 years ago (2016-12-02 07:28:16 UTC) #3
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2541123005/ by avayvod@chromium.org.

The reason for reverting is: Broke the official 56 build:
https://bugs.chromium.org/p/chromium/issues/detail?id=670559

Seems like https://codereview.chromium.org/2514343003 has to be merged before
(doesn't seem to be in the 56 branch according to
https://chromium.googlesource.com/chromium/src/+/56.0.2924.14/chrome/android/...).

Powered by Google App Engine
This is Rietveld 408576698