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

Issue 10827285: Revert 150041 to re-enable Chrome to Mobile by default on trunk (after M22 branch). (Closed)

Created:
8 years, 4 months ago by msw
Modified:
8 years, 4 months ago
Reviewers:
msw, sky, nyquist
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 150041 to re-enable Chrome to Mobile by default on trunk (after M22 branch). Change 150041 - Revert 143421 to disable Chrome to Mobile by default on trunk (for M22 branch). Change 143421 - Revert 142809 to re-enable Chrome to Mobile by default on trunk (after M21 branch). Change 142809 - Revert 135840 to disable Chrome to Mobile by default on trunk (for M21 branch). Change 135840 - Revert 135442 to re-enable Chrome to Mobile by default on trunk. We passed the M20 branch point with Chrome to Mobile disabled, as intended. Change 135442 - Disable Chrome To Mobile by default; enable via chrome://flags. This feature should be disabled-by-default on M19; for immediate merge. Don't revert http://crrev.com/130312 to keep the AboutFlags_disable-chrome-to-mobile UMA bin. Note: Mac's ViewIDTests don't yet support VIEW_ID_CHROME_TO_MOBILE_BUTTON. BUG=102709, 120905 TEST=Chrome To Mobile is enabled on trunk (after M22 branch point). TBR=msw@chromium.org,sky@chromium.org,nyquist@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=151120

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -9 lines) Patch
M chrome/browser/about_flags.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M chrome/browser/chrome_to_mobile_service.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/gtk/view_id_util_browsertest.cc View 1 chunk +0 lines, -4 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
msw
8 years, 4 months ago (2012-08-10 20:42:08 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698