Rename ignore_cache and ignoreCache to bypass* unless it affects API

third_party/WebKit/Source/devtools/front_end/externs.js

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 277 matching lines @@
  */
 function ExtensionDescriptor() {
     this.startPage = "";
     this.name = "";
 }
 
 /**
  * @constructor
  */
 function ExtensionReloadOptions() {
+    // TODO(toyoshim): Check if this ignoreCache shouldn't be changed for
+    // DevTools protocol. Also, how this prototype is used actually?

[Takashi Toyoshima, 2016/04/06 09:36:23]

I can not find any caller that uses this function to instantiate new
ExtensionReloadOptions.
But this file is externs.js, so I guess this is exposed to external users in
some way?

[dgozman, 2016/04/06 15:44:19]

This is for type checking. It's mentioned in ExtensionServer.js, and we check
that fields accessed are as defined here. Should be left as is.

[Takashi Toyoshima, 2016/04/08 04:50:55]

Thanks!

     this.ignoreCache = false;
     this.injectedScript = "";
     this.userAgent = "";
 }
 
 var Adb = {};
 /** @typedef {{id: string, name: string, url: string, adbAttachedForeign: boolean}} */
 Adb.Page;
 /** @typedef {{id: string, adbBrowserChromeVersion: string, compatibleVersion: boolean, adbBrowserName: string, source: string, adbBrowserVersion: string, pages: !Array<!Adb.Page>}} */
 Adb.Browser;
@@ skipping 381 matching lines @@
  * @extends {ESTree.Node}
  * @constructor
  */
 ESTree.TemplateLiteralNode = function()
 {
     /** @type {!Array.<!ESTree.Node>} */
     this.quasis;
     /** @type {!Array.<!ESTree.Node>} */
     this.expressions;
 }