Route browser reload command to DevTools if present.

Route browser reload command to DevTools if present.

DevTools augments reloading with extra actions like recording the timeline.
Let reload go through DevTools if browser command triggered it, similarly to
how it already goes for Ctrl+R shortcut.

BUG=521057
Committed: https://crrev.com/3030f314d43c2ca51cff507f92911e03067dbecf
Cr-Commit-Position: refs/heads/master@{#343608}

[dgozman, 2015-08-14 18:37:21]

dgozman@chromium.org changed reviewers:
+ pfeldman@chromium.org, sky@chromium.org

[dgozman, 2015-08-14 18:37:21]

Could you please take a look?

[pfeldman, 2015-08-14 18:40:40]

lgtm

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/devtools/dev...
File chrome/browser/devtools/devtools_window.cc (right):

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/devtools/dev...
chrome/browser/devtools/devtools_window.cc:1245:
bindings_->CallClientFunction("DevToolsAPI.reload",
requestReload? reloadInspectedPage?

[sky, 2015-08-14 20:04:31]

LGTM

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/ui/browser_c...
File chrome/browser/ui/browser_commands.cc (right):

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/ui/browser_c...
chrome/browser/ui/browser_commands.cc:222:
devtools->ReloadInspectedWebContents(ignore_cache);
nit: spacing is off.

[dgozman, 2015-08-14 23:57:34]

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/devtools/dev...
File chrome/browser/devtools/devtools_window.cc (right):

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/devtools/dev...
chrome/browser/devtools/devtools_window.cc:1245:
bindings_->CallClientFunction("DevToolsAPI.reload",
On 2015/08/14 18:40:40, pfeldman wrote:
> requestReload? reloadInspectedPage?

reloadInspectedPage.

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/ui/browser_c...
File chrome/browser/ui/browser_commands.cc (right):

https://codereview.chromium.org/1292403004/diff/1/chrome/browser/ui/browser_c...
chrome/browser/ui/browser_commands.cc:222:
devtools->ReloadInspectedWebContents(ignore_cache);
On 2015/08/14 20:04:31, sky wrote:
> nit: spacing is off.

Done.

[dgozman, 2015-08-16 17:46:42]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2015-08-16 17:46:42]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org,
pfeldman@chromium.org
Link to the patchset: https://codereview.chromium.org/1292403004/#ps20001
(title: "Fixed review comments")

[commit-bot: I haz the power, 2015-08-16 17:46:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1292403004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1292403004/20001

[commit-bot: I haz the power, 2015-08-16 18:59:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-16 18:59:44]

Patchset 2 (id:??) landed as
https://crrev.com/3030f314d43c2ca51cff507f92911e03067dbecf
Cr-Commit-Position: refs/heads/master@{#343608}