GitLab

Before this patch, we cleared the delegate for web_content
unconditionally. That means we would remove an already existing delegate
from web_content.

This patch introduces a bool in the controller to track whether the delegate
should be cleared in the destructor.

After this patch, we will no longer stomp on other components setting
a delegate in web_contents.

R=gogerald@chromium.org

Bug: 806868
Change-Id: I92ce1ef2dcf4a48f6709bddee0f76871a58c2df4
Reviewed-on: https://chromium-review.googlesource.com/c/1263118Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596978}