Issue 11473019: [cros,rlz] Remove RLZ checkbox from OOBE UI.

Issue 11473019: [cros,rlz] Remove RLZ checkbox from OOBE UI. (Closed)

Created:
8 years ago by Ivan Korotkov

Modified:
8 years ago

Reviewers:
dennis_jeffrey, Nikita (slow)

CC:
chromium-reviews, nkostylev+watch_chromium.org, kkania, dyu1, arv (Not doing code reviews), robertshield, oshima+watch_chromium.org, dennis_jeffrey, anantha, stevenjb+watch_chromium.org, davemoore+watch_chromium.org

Base URL:
svn://svn.chromium.org/chrome/trunk/src

Visibility:
Public.

More Reviews

Description

[cros,rlz] Remove RLZ checkbox from OOBE UI. BUG=160059 TBR=nirnimesh@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=172040

Patch Set 1 #

Patch Set 2 : linux_chromeos fix #

Patch Set 3 : Add back old strings #

Created: 8 years ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+20 lines, -80 lines)			Patch
M	chrome/app/chromeos_strings.grdp	View	1 2	1 chunk	+3 lines, -0 lines	0 comments	Download
M	chrome/browser/automation/testing_automation_provider_chromeos.cc	View		2 chunks	+2 lines, -5 lines	0 comments	Download
M	chrome/browser/chromeos/login/eula_screen.h	View		1 chunk	+1 line, -3 lines	0 comments	Download
M	chrome/browser/chromeos/login/eula_screen.cc	View		1 chunk	+1 line, -4 lines	0 comments	Download
M	chrome/browser/chromeos/login/eula_screen_actor.h	View		1 chunk	+1 line, -3 lines	0 comments	Download
M	chrome/browser/chromeos/login/mock_screen_observer.h	View		1 chunk	+0 lines, -2 lines	0 comments	Download
M	chrome/browser/chromeos/login/screen_observer.h	View		1 chunk	+0 lines, -4 lines	0 comments	Download
M	chrome/browser/chromeos/login/wizard_controller.h	View		2 chunks	+0 lines, -5 lines	0 comments	Download
M	chrome/browser/chromeos/login/wizard_controller.cc	View	1 2	4 chunks	+4 lines, -19 lines	0 comments	Download
M	chrome/browser/resources/chromeos/login/oobe.css	View	1 2	1 chunk	+0 lines, -9 lines	0 comments	Download
M	chrome/browser/resources/chromeos/login/oobe.html	View		2 chunks	+1 line, -6 lines	0 comments	Download
M	chrome/browser/resources/chromeos/login/oobe_screen_eula.js	View		2 chunks	+2 lines, -4 lines	0 comments	Download
M	chrome/browser/ui/webui/chromeos/login/eula_screen_handler.cc	View	1 2	3 chunks	+3 lines, -7 lines	0 comments	Download
M	chrome/common/pref_names.h	View	1 2	1 chunk	+0 lines, -1 line	0 comments	Download
M	chrome/common/pref_names.cc	View	1 2	1 chunk	+0 lines, -3 lines	0 comments	Download
M	chrome/test/pyautolib/pyauto.py	View		3 chunks	+2 lines, -5 lines	0 comments	Download

Messages

Total messages: 7 (0 generated)

Expand Messages | Collapse Messages

Ivan Korotkov

PTAL There's almost nothing RLZ-related here, only ChromeOS-related stuff

8 years ago (2012-12-07 12:58:55 UTC) #1

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/ivankr@chromium.org/11473019/8001

8 years ago (2012-12-10 09:07:23 UTC) #5

Message was sent while issue was closed.

pyautolib changes LGTM

Expand Messages | Collapse Messages