Issue 837011: Fix issue where search did not select the search tree item. This also fixes a...

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 837011: Fix issue where search did not select the search tree item. This also fixes a... (Closed)

Created:
10 years, 9 months ago by arv (Not doing code reviews)

Modified:
9 years, 3 months ago

Reviewers:
feldstein

CC:
chromium-reviews, ben+cc_chromium.org

Base URL:
svn://chrome-svn/chrome/trunk/src/

Visibility:
Public.

Description

Fix issue where search did not select the search tree item. This also fixes an issue with reloading after a search. BUG=4890 TEST=Enter some search terms. Hitting enter should select the search tree item and show the search results. Noew, hit F5 to reload. The search term should be filled in and the search tree item should be selected and the search results should be shown. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=41656

Patch Set 1 #

Created: 10 years, 9 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+12 lines, -8 lines)			Patch
	M	chrome/browser/resources/bookmark_manager/main.html	View		3 chunks	+12 lines, -8 lines	0 comments	Download

Messages

Total messages: 2 (0 generated)

Expand Messages | Collapse Messages

LGTM

Expand Messages | Collapse Messages