[Extensions] Implement chrome.runtime.openOptionsPage.

[Extensions] Implement chrome.runtime.openOptionsPage.

BUG=430054
R=rockot@chromium.org

Committed: https://crrev.com/ce22c477cd86c9edfe564083dc244c44e18b0caa
Cr-Commit-Position: refs/heads/master@{#316975}

[not at google - send to devlin, 2015-02-18 17:06:31]

Well this was more effort than I expected.

[Ken Rockot(use gerrit already), 2015-02-18 20:47:47]

Doesn't seem too bad. Maybe we should (not now) let the delegate specify default
implementations to avoid the tedious duplication of effort with every change
that just needs stubs in shell or test impls.

Anyway, LGTM

[Ken Rockot(use gerrit already), 2015-02-18 20:48:26]

On 2015/02/18 20:47:47, Ken Rockot wrote:
> Doesn't seem too bad. Maybe we should (not now) let the delegate specify
default
> implementations to avoid the tedious duplication of effort with every change
> that just needs stubs in shell or test impls.
> 
> Anyway, LGTM

Also the (not now) only means "not now, unless you want to" because I'd be fine
with that here.

[not at google - send to devlin, 2015-02-18 21:05:08]

On 2015/02/18 20:48:26, Ken Rockot wrote:
> On 2015/02/18 20:47:47, Ken Rockot wrote:
> > Doesn't seem too bad. Maybe we should (not now) let the delegate specify
> default
> > implementations to avoid the tedious duplication of effort with every change
> > that just needs stubs in shell or test impls.
> > 
> > Anyway, LGTM
> 
> Also the (not now) only means "not now, unless you want to" because I'd be
fine
> with that here.

That's a better idea, I'll do that now.

[not at google - send to devlin, 2015-02-18 21:54:11]

New patchsets have been uploaded after l-g-t-m from rockot@chromium.org

[not at google - send to devlin, 2015-02-18 21:54:52]

kalman@chromium.org changed reviewers:
+ isherman@chromium.org

[not at google - send to devlin, 2015-02-18 21:54:52]

isherman for histogramx.

[Ilya Sherman, 2015-02-19 00:20:02]

histogramx lgtm

[not at google - send to devlin, 2015-02-19 00:35:27]

The CQ bit was checked by kalman@chromium.org

[commit-bot: I haz the power, 2015-02-19 00:37:00]

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

[commit-bot: I haz the power, 2015-02-19 02:31:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-02-19 02:32:47]

Patchset 3 (id:??) landed as
https://crrev.com/ce22c477cd86c9edfe564083dc244c44e18b0caa
Cr-Commit-Position: refs/heads/master@{#316975}