Commit a4440342 authored by Mark Cogan's avatar Mark Cogan Committed by Commit Bot

[ios] Fix Share button in tools menu.

CL 563307 didn't add taget/action for the share button on the tools
menu, which meant that the share button (and thus: printing and reading
list) didn't work.

This CL fixes that, by adding the two add/remove target/action calls
that should have been there in the first place.

Bug: 73881
Change-Id: Ia3c90b8c6175cb3038f321636ef467efecc3cbfb
Reviewed-on: https://chromium-review.googlesource.com/565498
Commit-Queue: Mark Cogan <marq@chromium.org>
Reviewed-by: default avatarJean-François Geyelin <jif@chromium.org>
Cr-Commit-Position: refs/heads/master@{#485288}
parent b757c63d
......@@ -472,6 +472,9 @@ NS_INLINE void AnimateInViews(NSArray* views,
[toolsCell.reloadButton removeTarget:self.dispatcher
action:@selector(reload)
forControlEvents:UIControlEventTouchUpInside];
[toolsCell.shareButton removeTarget:self.dispatcher
action:@selector(sharePage)
forControlEvents:UIControlEventTouchUpInside];
[toolsCell.starButton removeTarget:self.dispatcher
action:@selector(bookmarkPage)
forControlEvents:UIControlEventTouchUpInside];
......@@ -595,6 +598,9 @@ NS_INLINE void AnimateInViews(NSArray* views,
[cell.reloadButton addTarget:self.dispatcher
action:@selector(reload)
forControlEvents:UIControlEventTouchUpInside];
[cell.shareButton addTarget:self.dispatcher
action:@selector(sharePage)
forControlEvents:UIControlEventTouchUpInside];
[cell.starButton addTarget:self.dispatcher
action:@selector(bookmarkPage)
forControlEvents:UIControlEventTouchUpInside];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment