DevTools: Fix the way search works when switching between editor tabs.

DevTools: Fix the way search works when switching between editor tabs.

We should perform the search in the newly selected editor after switching tabs.
We should also postpone setting selection in the editor to the moment where it has synced its sized, otherwise we mess up the scroll position.

BUG=413153
R=lushnikov@chromium.org, lushnikov

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184257

[vsevik, 2014-10-22 07:46:05]

PTAL

[lushnikov, 2014-10-22 11:12:37]

https://codereview.chromium.org/669043002/diff/20001/Source/devtools/front_en...
File Source/devtools/front_end/components/SearchableView.js (right):

https://codereview.chromium.org/669043002/diff/20001/Source/devtools/front_en...
Source/devtools/front_end/components/SearchableView.js:372: this._clearSearch();
these three lines are "resetSearch()" essentially

[lushnikov, 2014-10-22 11:14:30]

On 2014/10/22 11:12:37, lushnikov wrote:
>
https://codereview.chromium.org/669043002/diff/20001/Source/devtools/front_en...
> File Source/devtools/front_end/components/SearchableView.js (right):
> 
>
https://codereview.chromium.org/669043002/diff/20001/Source/devtools/front_en...
> Source/devtools/front_end/components/SearchableView.js:372:
this._clearSearch();
> these three lines are "resetSearch()" essentially

Can we also add a test for this? This all looks very fragile.

[lushnikov, 2014-10-22 11:31:24]

lgtm

[vsevik, 2014-10-23 07:27:38]

The CQ bit was checked by vsevik@chromium.org

[commit-bot: I haz the power, 2014-10-23 07:28:18]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/669043002/40001

[commit-bot: I haz the power, 2014-10-23 09:21:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-23 09:21:27]

Try jobs failed on following builders:
  win_blink_rel on tryserver.blink
(http://build.chromium.org/p/tryserver.blink/builders/win_blink_rel/builds/33224)

[vsevik, 2014-10-23 10:52:16]

The CQ bit was checked by vsevik@chromium.org

[commit-bot: I haz the power, 2014-10-23 10:52:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/669043002/40001

[vsevik, 2014-10-23 11:53:12]

Committed patchset #3 (id:40001) manually as 184257 (presubmit successful).