Add frameId to chrome.tabs.executeScript/insertCSS

extensions/browser/script_executor.h

Index: extensions/browser/script_executor.h
diff --git a/extensions/browser/script_executor.h b/extensions/browser/script_executor.h
index 4f81b78fb62dfe17964aa2600e57b5cbf0c4c6fe..0164003c4e87cfd51813c89e8213617957df6344 100644
--- a/extensions/browser/script_executor.h
+++ b/extensions/browser/script_executor.h
@@ -44,7 +44,7 @@ class ScriptExecutor {
 
   // The scope of the script injection across the frames.
   enum FrameScope {
-    TOP_FRAME,
+    SINGLE_FRAME,
     ALL_FRAMES,
   };
 
@@ -89,6 +89,7 @@ class ScriptExecutor {
                      ScriptType script_type,
                      const std::string& code,
                      FrameScope frame_scope,
+                     int frame_id,

[Devlin, 2016/01/25 19:36:40]

Man, this function makes me so sad.  At the same time, if we moved it to a
Params struct, then no one would ever update the callers because we wouldn't get
a compile error.  Ah well.

Complaints aside, please document the |frame_id| argument briefly (and mention
it's behavior with FrameScope).

[robwu, 2016/01/26 11:03:02]

Done.

                      MatchAboutBlank match_about_blank,
                      UserScript::RunLocation run_at,
                      WorldType world_type,