DescriptionAlternate NTP: Fix bug where shadow is not removed
Opening a link in a new tab from instant search results would cause the instant search result shadow to stay visible.
The problem was that there were three ways the shadow could be hidden:
(1) -showPreview:..:drawDropShadow:NO
(2) - hidePreview
(3) - onActivateTabWithContents:
(1) and (2) correctly hid the shadow but (3) didn't.
Fix was consolidate all the code into (1) and have everyone call that method.
BUG=173778
TEST=See comment 7 in bug 173778 for an easy way to reproduce this bug. Verified that without my fix the shadow is not removed and with my fix the shadow is removed.
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=181865
Patch Set 1 #Patch Set 2 : " #Patch Set 3 : fix test #Patch Set 4 : fix tests #
Messages
Total messages: 12 (0 generated)
|