Remove reference to OpenTabOptions in chrome.browser.openTab docs.

chrome/common/extensions/api/browser.idl

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Use the <code>chrome.browser</code> API to interact with the Chrome browser
 // associated with the current application and Chrome profile.
 namespace browser {
   // Options for the $(ref:openTab) function.
   dictionary OpenTabOptions {
     // The URL to navigate to when the new tab is initially opened.
     DOMString url;
   };
 
   callback Callback = void();
 
   interface Functions {
     // Opens a new tab in a browser window associated with the current
     // application and Chrome profile. If no browser window for the Chrome
-    // profile is opened, a new one is opened prior to creating the new tab. The
-    // initial URL of the new tab is specified in |options|.
-    // |options|  : The $(ref:OpenTabOptions) for this function.

[not at google - send to devlin, 2015/03/16 22:21:16]

Tautological, since docs will generate this already - and it'll inline
OpenTabOptions, which is why the link results in a broken link.

[rpaquay, 2015/03/16 22:29:23]

Acknowledged.

-    // |callback| : Called to indicate success or failure.

[not at google - send to devlin, 2015/03/16 22:21:16]

Not true - it's a void callback.

[rpaquay, 2015/03/16 22:29:23]

The intent was to communicate that success/failure should be checked through
"chrome.runtime.lastError" when |callback| is invoked.

[not at google - send to devlin, 2015/03/16 22:33:51]

That sounds like a good thing to do.

+    // profile is opened, a new one is opened prior to creating the new tab.
     static void openTab(OpenTabOptions options,
                         optional Callback callback);
   };
 };